Changeset 985 for trunk/dll/avv.c

Timestamp:

Mar 1, 2008, 2:37:14 AM (17 years ago)

Author:

Gregg Young

Message:

Update sizes dialog (ticket 44); Make max command line length user settable (ticket 199); use xfree for free in most cases (ticket 212); initial code to check for valid ini file (ticket 102); Some additional refactoring and structure rework; Some documentation updates;

File:

• trunk/dll/avv.c (modified) (3 diffs)

trunk/dll/avv.c

@@ -24 +24 @@
   20 Aug 07 GKY Move #pragma alloc_text to end for OpenWatcom compat
   06 Jan 08 GKY Use NormalizeCmdLine to check program strings on entry
+  29 Feb 08 GKY Changes to enable user settable command line length
 
 ***********************************************************************/
@@ -93 +94 @@
 static PSZ free_and_strdup_quoted_from_window(HWND hwnd, USHORT id, PSZ pszDest)
 { // fixme for command line limit
-  CHAR szCmdLine[MAXCOMLINESTRG];
-
+  CHAR *szCmdLine;
+
+  szCmdLine = xmalloc(MaxComLineStrg, pszSrcFile, __LINE__);
+        if (!szCmdLine)
+          return NULL; //already complained
   xfree(pszDest);
   WinQueryDlgItemText(hwnd, id, sizeof(szCmdLine), szCmdLine);
   if (*szCmdLine){
     PSZ pszWorkBuf;
-    pszWorkBuf = xmalloc(MAXCOMLINESTRG, pszSrcFile, __LINE__);
+    pszWorkBuf = xmalloc(MaxComLineStrg, pszSrcFile, __LINE__);
     if (pszWorkBuf) {
       NormalizeCmdLine(pszWorkBuf, szCmdLine);
@@ -110 +114 @@
   else
     pszDest = NULL;
+  xfree(szCmdLine);
   return pszDest;
 }