Changeset 1039 for trunk/dll/assoc.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/assoc.c

    r1029 r1039  
    3737#include "strutil.h"                    // GetPString
    3838#include "fm3dll.h"
     39#include "fortify.h"
    3940
    4041#pragma data_seg(DATA1)
    … …  
    122123    xfree(info->pszCmdLine, pszSrcFile, __LINE__);
    123124    xfree(info->sig, pszSrcFile, __LINE__);
    124     xfree(info, pszSrcFile, __LINE__);
     125    free(info);
    125126    info = next;
    126127  }
    … …  
    185186          xfree(info->pszCmdLine, pszSrcFile, __LINE__);
    186187          xfree(info->mask, pszSrcFile, __LINE__);
    187           xfree(info, pszSrcFile, __LINE__);
     188          free(info);
    188189          break;
    189190        }
    … …  
    234235                        MPFROMSHORT(x), MPFROMSHORT(TRUE));
    235236    }
    236     xfree(pszDisplayStr, pszSrcFile, __LINE__);
     237    free(pszDisplayStr);
    237238  }
    238239}
    … …  
    316317          xfree(info->pszCmdLine, pszSrcFile, __LINE__);
    317318          xfree(info->mask, pszSrcFile, __LINE__);
    318           xfree(info, pszSrcFile, __LINE__);
     319          free(info);
    319320        }
    320321        else {
    … …  
    363364        xfree(info->mask, pszSrcFile, __LINE__);
    364365        xfree(info->sig, pszSrcFile, __LINE__);
    365         xfree(info, pszSrcFile, __LINE__);
     366        free(info);
    366367        return TRUE;
    367368      }
    … …  
    696697        pszWorkBuf = xmalloc(MaxComLineStrg, pszSrcFile, __LINE__);
    697698        if (!pszWorkBuf) {
    698           xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     699          free(temp.pszCmdLine);
    699700          break; //already complained
    700701        }
    … …  
    705706          memcpy(temp.pszCmdLine, pszWorkBuf, strlen(pszWorkBuf) + 1);
    706707        }
    707         xfree(pszWorkBuf, pszSrcFile, __LINE__);
     708        free(pszWorkBuf);
    708709        WinQueryDlgItemText(hwnd, ASS_SIG, sizeof(temp.sig), temp.sig);
    709710        rstrip(temp.sig);
    … …  
    732733        if (fCancelAction){
    733734          fCancelAction = FALSE;
    734           xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     735          free(temp.pszCmdLine);
    735736          break;
    736737        }
    … …  
    743744          save_associations();
    744745        }
    745         xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     746        free(temp.pszCmdLine);
    746747      }
    747748      WinDismissDlg(hwnd, 1);
    … …  
    771772        pszWorkBuf = xmalloc(MaxComLineStrg, pszSrcFile, __LINE__);
    772773        if (!pszWorkBuf) {
    773           xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     774          free(temp.pszCmdLine);
    774775          break; //already complained
    775776        }
    … …  
    780781          memcpy(temp.pszCmdLine, pszWorkBuf, strlen(pszWorkBuf) + 1);
    781782        }
    782         xfree(pszWorkBuf, pszSrcFile, __LINE__);
     783        free(pszWorkBuf);
    783784        WinQueryDlgItemText(hwnd, ASS_SIG, sizeof(temp.sig), temp.sig);
    784785        rstrip(temp.sig);
    … …  
    807808        if (fCancelAction){
    808809          fCancelAction = FALSE;
    809           xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     810          free(temp.pszCmdLine);
    810811          break;
    811812        }
    … …  
    817818          save_associations();
    818819        }
    819         xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     820        free(temp.pszCmdLine);
    820821      }
    821822      break;
    … …  
    853854          save_associations();
    854855        }
    855         xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     856        free(temp.pszCmdLine);
    856857      }
    857858
    … …  
    875876        pszWorkBuf = xmalloc(MaxComLineStrg, pszSrcFile, __LINE__);
    876877        if (!pszWorkBuf) {
    877           xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     878          free(temp.pszCmdLine);
    878879          break; //already complained
    879880        }
    … …  
    884885          memcpy(temp.pszCmdLine, pszWorkBuf, strlen(pszWorkBuf) + 1);
    885886        }
    886         xfree(pszWorkBuf, pszSrcFile, __LINE__);
     887        free(pszWorkBuf);
    887888        WinQueryDlgItemText(hwnd, ASS_SIG, sizeof(temp.sig), temp.sig);
    888889        rstrip(temp.sig);
    … …  
    911912        if (fCancelAction){
    912913          fCancelAction = FALSE;
    913           xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     914          free(temp.pszCmdLine);
    914915          break;
    915916        }
    … …  
    920921          save_associations();
    921922        }
    922         xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     923        free(temp.pszCmdLine);
    923924      }
    924925      {
    … …  
    957958          save_associations();
    958959        }
    959         xfree(temp.pszCmdLine, pszSrcFile, __LINE__);
     960        free(temp.pszCmdLine);
    960961      }
    961962      break;
Note: See TracChangeset for help on using the changeset viewer.