replacing free() with SAFE_FREE() where possible
authorGerald Carter <jerry@samba.org>
Mon, 24 Jun 2002 21:14:30 +0000 (21:14 +0000)
committerGerald Carter <jerry@samba.org>
Mon, 24 Jun 2002 21:14:30 +0000 (21:14 +0000)
(This used to be commit 5a4a7e5a88f4fdc5891436b05e44b05d1e7ad3f1)

source3/printing/pcap.c

index 920c6f354e27daa0ba0f4ac2c012dc880eb6d428..4bca63fffb1fd52745e33a9a4e0c359f38135e88 100644 (file)
@@ -111,7 +111,7 @@ static void ScanQconfig_fn(char *psz,void (*fn)(char *, char *))
 
        iEtat = 0;
        /* scan qconfig file for searching <printername>:       */
-       for (;(line = fgets_slash(NULL,sizeof(pstring),pfile)); free(line))
+       for (;(line = fgets_slash(NULL,sizeof(pstring),pfile)); safe_free(line))
        {
                if (*line == '*' || *line == 0)
                continue;
@@ -181,7 +181,7 @@ static BOOL ScanQconfig(char *psz,char *pszPrintername)
        if ((pfile = x_fopen(psz, O_RDONLY, 0)) == NULL)
        {
              DEBUG(0,( "Unable to open qconfig file %s for read!\n", psz));
-             free(pName);
+             SAFE_FREE(pName);
              return(False);
        }
        slprintf(pName, iLg + 9, "%s:",pszPrintername);
@@ -189,7 +189,7 @@ static BOOL ScanQconfig(char *psz,char *pszPrintername)
        /*DEBUG(3,( " Looking for entry %s\n",pName));*/
        iEtat = 0;
        /* scan qconfig file for searching <printername>:       */
-       for (;(line = fgets_slash(NULL,sizeof(pstring),pfile)); free(line))
+       for (;(line = fgets_slash(NULL,sizeof(pstring),pfile)); safe_free(line))
        {
                if (*line == '*' || *line == 0)
                continue;
@@ -209,7 +209,7 @@ static BOOL ScanQconfig(char *psz,char *pszPrintername)
                           /* name is found without stanza device  */
                           /* probably a good printer ???               */
                           free (line);
-                          free(pName);
+                          SAFE_FREE(pName);
                           fclose(pfile);
                           return(True);
                          }
@@ -223,7 +223,7 @@ static BOOL ScanQconfig(char *psz,char *pszPrintername)
                          {
                                /* it's a good virtual printer */
                                free (line);
-                               free(pName);
+                               SAFE_FREE(pName);
                                fclose(pfile);
                                return(True);
                          }
@@ -288,7 +288,7 @@ BOOL pcap_printername_ok(char *pszPrintername, char *pszPrintcapname)
       return(False);
     }
 
-  for (;(line = fgets_slash(NULL,sizeof(pstring),pfile)); free(line))
+  for (;(line = fgets_slash(NULL,sizeof(pstring),pfile)); safe_free(line))
     {
       if (*line == '#' || *line == 0)
        continue;
@@ -307,7 +307,7 @@ BOOL pcap_printername_ok(char *pszPrintername, char *pszPrintcapname)
            {
              /* normalise the case */
              pstrcpy(pszPrintername,p);
-             free(line);
+             SAFE_FREE(line);
              x_fclose(pfile);
              return(True);           
            }
@@ -369,7 +369,7 @@ void pcap_printer_fn(void (*fn)(char *, char *))
       return;
     }
 
-  for (;(line = fgets_slash(NULL,sizeof(pstring),pfile)); free(line))
+  for (;(line = fgets_slash(NULL,sizeof(pstring),pfile)); safe_free(line))
     {
       if (*line == '#' || *line == 0)
        continue;