Changeset 1039 for trunk/dll/command.c


Ignore:
Timestamp:
Jul 6, 2008, 12:16:21 AM (17 years ago)
Author:
Gregg Young
Message:

Removed unnecessary xfrees and included fortify.h where needed; moved several misplaced (x)frees;

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/dll/command.c

    r1009 r1039  
    4444#include "pathutil.h"                   // NormalizeCmdLine
    4545#include "command.h"
     46#include "fortify.h"
    4647
    4748typedef struct
    … …  
    310311    xfree(info->title, pszSrcFile, __LINE__);
    311312    xfree(info->pszCmdLine, pszSrcFile, __LINE__);
    312     xfree(info, pszSrcFile, __LINE__);
     313    free(info);
    313314    info = next;
    314315  }
    … …  
    360361            xfree(info->pszCmdLine, pszSrcFile, __LINE__);
    361362            xfree(info->title, pszSrcFile, __LINE__);
    362             xfree(info, pszSrcFile, __LINE__);
     363            free(info);
    363364            break;
    364365          }
    … …  
    372373        }
    373374      }
    374       xfree(pszCmdLine, pszSrcFile, __LINE__);
     375      free(pszCmdLine);
    375376      fclose(fp);
    376377    }
    … …  
    429430    xfree(info->pszCmdLine, pszSrcFile, __LINE__);
    430431    xfree(info->title, pszSrcFile, __LINE__);
    431     xfree(info, pszSrcFile, __LINE__);
     432    free(info);
    432433    return NULL;
    433434  }
    … …  
    466467        xfree(info->pszCmdLine, pszSrcFile, __LINE__);
    467468        xfree(info->title, pszSrcFile, __LINE__);
    468         xfree(info, pszSrcFile, __LINE__);
     469        free(info);
    469470        return TRUE;
    470471      }
    … …  
    635636        pszWorkBuf = xmalloc(MaxComLineStrg, pszSrcFile, __LINE__);
    636637        if (!pszWorkBuf) {
    637           xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     638          free(temp.pszCmdLine);
    638639          break; //already complained
    639640        }
    … …  
    641642        NormalizeCmdLine(pszWorkBuf, temp.pszCmdLine);
    642643        memcpy(temp.pszCmdLine, pszWorkBuf, strlen(pszWorkBuf) + 1);
    643         xfree(pszWorkBuf, pszSrcFile, __LINE__);
     644        free(pszWorkBuf);
    644645        if (!strchr(temp.pszCmdLine, '%')){
    645646          ret = saymsg(MB_YESNO,
    … …  
    669670        if (fCancelAction){
    670671          fCancelAction = FALSE;
    671           xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     672          free(temp.pszCmdLine);
    672673          break;
    673674        }
    … …  
    708709          }
    709710        }
    710         xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     711        free(temp.pszCmdLine);
    711712      }
    712713      x = (SHORT) WinSendDlgItemMsg(hwnd,
    … …  
    740741        pszWorkBuf = xmalloc(MaxComLineStrg, pszSrcFile, __LINE__);
    741742        if (!pszWorkBuf) {
    742           xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     743          free(temp.pszCmdLine);
    743744          break; //already complained
    744745        }
    … …  
    746747        NormalizeCmdLine(pszWorkBuf, temp.pszCmdLine);
    747748        memcpy(temp.pszCmdLine, pszWorkBuf, strlen(pszWorkBuf) + 1);
    748         xfree(pszWorkBuf, pszSrcFile, __LINE__);
     749        free(pszWorkBuf);
    749750        if (!strchr(temp.pszCmdLine, '%')){
    750751          ret = saymsg(MB_YESNO,
    … …  
    774775        if (fCancelAction){
    775776          fCancelAction = FALSE;
    776           xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     777          free(temp.pszCmdLine);
    777778          break;
    778779        }
    … …  
    809810          }
    810811        }
    811         xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     812        free(temp.pszCmdLine);
    812813      }
    813814      break;
    … …  
    853854        temp.pszCmdLine = xmallocz(MaxComLineStrg, pszSrcFile, __LINE__);
    854855        if (!temp.pszCmdLine) {
    855           xfree(pszWorkBuf, pszSrcFile, __LINE__);
     856          free(pszWorkBuf);
    856857          break; //already complained
    857858        }
    … …  
    859860        NormalizeCmdLine(pszWorkBuf, temp.pszCmdLine);
    860861        memcpy(temp.pszCmdLine, pszWorkBuf, strlen(pszWorkBuf) + 1);
    861         xfree(pszWorkBuf, pszSrcFile, __LINE__);
     862        free(pszWorkBuf);
    862863        if (fCancelAction){
    863864          fCancelAction = FALSE;
    864           xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     865          free(temp.pszCmdLine);
    865866          break;
    866867        }
    … …  
    976977          }
    977978        }
    978         xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     979        free(temp.pszCmdLine);
    979980      }
    980981      break;
Note: See TracChangeset for help on using the changeset viewer.