8d22a576e16342266ce80501318803e7b0fa466b
galt
  Fri Sep 16 11:48:02 2016 -0700
redundant va_end called is not needed. now it follows the standard. va_end was getting called twice incorrectly even though we saw no errors from it.

diff --git src/hg/lib/jksql.c src/hg/lib/jksql.c
index ebdc621..5844467 100644
--- src/hg/lib/jksql.c
+++ src/hg/lib/jksql.c
@@ -3936,31 +3936,30 @@
 	    // copy string to its final destination.
 	    memmove(buffer, tempBuf2, sz+1); // +1 for terminating 0;
 	    }
 	freeMem(tempBuf2);
 	}
     freeMem(tempBuf);
     }
 else
     {
     sz = vsnprintf(buffer, bufSize, newFormat, orig_args);
     /* note that some version return -1 if too small */
     }
 
 freeMem(newFormat);
 va_end(orig_args);
-va_end(args);
 
 return sz;
 
 }
 
 
 
 
 
 int vaSqlSafef(char* buffer, int bufSize, char *format, va_list args)
 /* VarArgs Format string to buffer, vsprintf style, only with buffer overflow
  * checking.  The resulting string is always terminated with zero byte. */
 {
 int sz = vaSqlSafefNoAbort(buffer, bufSize, TRUE, format, args);
 if ((sz < 0) || (sz >= bufSize))