5157fdf6389da69cc64f5c0f222f9f6bc8358fec
hiram
  Fri May 10 14:23:21 2019 -0700
better to centralize listing of legal arguments in one location refs #23437 #18869

diff --git src/hg/hubApi/apiUtils.c src/hg/hubApi/apiUtils.c
index 3aec27e..b4b8b84 100644
--- src/hg/hubApi/apiUtils.c
+++ src/hg/hubApi/apiUtils.c
@@ -379,31 +379,31 @@
     }
 if (NULL == hubGenome)
     apiErrAbort(err400, err400Msg, "failed to find specified genome=%s for endpoint '%s'  given hubUrl '%s'", genome, endpoint, hubUrl);
 return hubGenome;
 }
 
 char *verifyLegalArgs(char *validArgList)
 /* validArgList is a semicolon;separated;list;of;valid;arguments
  * returning string of any other arguments not on that list found in
  * cgiVarList(), NULL when none found.
  */
 {
 struct hash *validHash = NULL;
 char *words[32];
 int wordCount = 0;
-if (validArgList)
+if (isNotEmpty(validArgList))
     {
     wordCount = chopByChar(cloneString(validArgList), ';', words, 32);
     }
 
 if (wordCount)
     {
     validHash = hashNew(0);
     int i;
     for (i = 0; i < wordCount; ++i)
 	hashAddInt(validHash, words[i], 1);
     }
 
 int extrasFound = 0;
 struct dyString *extras = newDyString(128);
 struct cgiVar *varList = cgiVarList();