Changeset 959 for trunk/dll/avv.c

Timestamp:

Feb 18, 2008, 1:47:31 AM (18 years ago)

Author:

Gregg Young

Message:

Use xfree where appropriate. Check that buffer exists following all xmallocs. Stopped at eas.c with xfree checking. One remaining xmalloc without test in dirsize.c

File:

• trunk/dll/avv.c (modified) (1 diff)

trunk/dll/avv.c

@@ -100 +100 @@
     PSZ pszWorkBuf;
     pszWorkBuf = xmalloc(MAXCOMLINESTRG, pszSrcFile, __LINE__);
-    NormalizeCmdLine(pszWorkBuf, szCmdLine);
-    pszDest = xstrdup(szCmdLine, pszSrcFile, __LINE__);
-    xfree(pszWorkBuf);
+    if (pszWorkBuf) {
+      NormalizeCmdLine(pszWorkBuf, szCmdLine);
+      pszDest = xstrdup(pszWorkBuf, pszSrcFile, __LINE__);
+      xfree(pszWorkBuf);
+    }
+    else
+      pszDest = xstrdup(szCmdLine, pszSrcFile, __LINE__); // return the unexamined input on failure
   }
   else