Ticket #1900: 1900-replacing-free-by-g_free.patch

File 1900-replacing-free-by-g_free.patch, 8.5 KB (added by vit_r, 10 years ago)
  • src/man2hlp.c

    From ab92944576d829805cb171f9e59e6b138a70da3d Mon Sep 17 00:00:00 2001
    From: Vit Rosin <vit_r@list.ru>
    Date: Fri, 25 Dec 2009 11:30:10 +0000
    Subject: [PATCH]  replacing 'free()' or 'free ()' by 'g_free ()'
    
    ---
     src/man2hlp.c                |   12 ++++++------
     src/pipethrough.c            |    4 ++--
     vfs/mcfsutil.c               |    2 +-
     vfs/samba/lib/charset.c      |    4 ++--
     vfs/samba/lib/username.c     |    2 +-
     vfs/samba/lib/util.c         |   20 ++++++++++----------
     vfs/samba/lib/util_file.c    |    2 +-
     vfs/samba/lib/util_str.c     |    2 +-
     vfs/samba/libsmb/clientgen.c |   34 +++++++++++++++++-----------------
     9 files changed, 41 insertions(+), 41 deletions(-)
    
    diff --git a/src/man2hlp.c b/src/man2hlp.c
    index aa95df6..1dd858f 100644
    a b main (int argc, char **argv) 
    822822            in_row = current_link->line; 
    823823            print_error (buffer); 
    824824        } 
    825         free (current_link->linkname); 
     825        g_free (current_link->linkname); 
    826826        if (current_link != &links) 
    827             free (current_link); 
     827            g_free (current_link); 
    828828        current_link = next; 
    829829    } 
    830830 
    main (int argc, char **argv) 
    837837                     "", cnode->lname ? cnode->lname : lc_node, lc_node); 
    838838        fprintf (f_out, "\n"); 
    839839 
    840         free (cnode->node); 
     840        g_free (cnode->node); 
    841841        if (cnode->lname) 
    842             free (cnode->lname); 
     842            g_free (cnode->lname); 
    843843        if (cnode != &nodes) 
    844             free (cnode); 
     844            g_free (cnode); 
    845845        cnode = next; 
    846846    } 
    847847 
    main (int argc, char **argv) 
    891891        return 1; 
    892892    } 
    893893 
    894     free (outfile_buffer); 
     894    g_free (outfile_buffer); 
    895895    fclose_check (f_out); 
    896896    /* Closing everything */ 
    897897 
  • src/pipethrough.c

    diff --git a/src/pipethrough.c b/src/pipethrough.c
    index e31f816..bc6528c 100644
    a b static void reader_buffer_finalize(/*@special@*/ struct reader_buffer *buf) 
    156156        /*@releases buf->data; @*/ 
    157157{ 
    158158        if (buf->data != NULL) { 
    159                 free(buf->data); 
     159                g_free (buf->data); 
    160160                buf->data = NULL; 
    161161        } 
    162162        buf->size = 0; 
    extern void pipe_outbuffer_finalize(struct pipe_outbuffer *buf) 
    450450        /*@ensures isnull buf->data; @*/ 
    451451{ 
    452452        if (buf->data != NULL) { 
    453                 free(buf->data); 
     453                g_free (buf->data); 
    454454                buf->data = NULL; 
    455455        } 
    456456        buf->size = 0; 
  • vfs/mcfsutil.c

    diff --git a/vfs/mcfsutil.c b/vfs/mcfsutil.c
    index d3c2e04..be7bf5a 100644
    a b rpc_get (int sock, ...) 
    186186            /* Don't use glib functions here - this code is used by mcserv */ 
    187187            text = malloc (len + 1); 
    188188            if (socket_read_block (sock, text, len) == 0) { 
    189                 free (text); 
     189                g_free (text); 
    190190                va_end (ap); 
    191191                return 0; 
    192192            } 
  • vfs/samba/lib/charset.c

    diff --git a/vfs/samba/lib/charset.c b/vfs/samba/lib/charset.c
    index 35f9126..f65b950 100644
    a b clean_and_exit: 
    315315  if(fp != NULL) 
    316316    fclose(fp); 
    317317  if(cp_p) 
    318     free((char *)cp_p); 
     318    g_free ((char *)cp_p); 
    319319  return NULL; 
    320320} 
    321321 
    void add_char_string(const char *s) 
    392392    } 
    393393  } 
    394394 
    395   free(extra_chars); 
     395  g_free (extra_chars); 
    396396} 
  • vfs/samba/lib/username.c

    diff --git a/vfs/samba/lib/username.c b/vfs/samba/lib/username.c
    index aaec3c7..abeb2ce 100644
    a b static struct passwd *_Get_Pwnam(const char *s) 
    160160      pwret = getpwanam(s); 
    161161      if (pwret) 
    162162        { 
    163           free(ret->pw_passwd); 
     163          g_free (ret->pw_passwd); 
    164164          ret->pw_passwd = pwret->pwa_passwd; 
    165165        } 
    166166#endif 
  • vfs/samba/lib/util.c

    diff --git a/vfs/samba/lib/util.c b/vfs/samba/lib/util.c
    index da2fa83..adda79c 100644
    a b void *Realloc(void *p,size_t size) 
    16421642  void *ret=NULL; 
    16431643 
    16441644  if (size == 0) { 
    1645     if (p) free(p); 
     1645    if (p) g_free (p); 
    16461646    DEBUG(5,("Realloc asked for 0 bytes\n")); 
    16471647    return NULL; 
    16481648  } 
    struct hostent *Get_Hostbyname(const char *name) 
    22512251#if 0 
    22522252  if (!isalnum(*name2)) 
    22532253    { 
    2254       free(name2); 
     2254      g_free (name2); 
    22552255      return(NULL); 
    22562256    } 
    22572257#endif /* 0 */ 
    struct hostent *Get_Hostbyname(const char *name) 
    22592259  ret = sys_gethostbyname(name2); 
    22602260  if (ret != NULL) 
    22612261    { 
    2262       free(name2); 
     2262      g_free (name2); 
    22632263      return(ret); 
    22642264    } 
    22652265 
    struct hostent *Get_Hostbyname(const char *name) 
    22682268  ret = sys_gethostbyname(name2); 
    22692269  if (ret != NULL) 
    22702270    { 
    2271       free(name2); 
     2271      g_free (name2); 
    22722272      return(ret); 
    22732273    } 
    22742274 
    struct hostent *Get_Hostbyname(const char *name) 
    22772277  ret = sys_gethostbyname(name2); 
    22782278  if (ret != NULL) 
    22792279    { 
    2280       free(name2); 
     2280      g_free (name2); 
    22812281      return(ret); 
    22822282    } 
    22832283   
    22842284  /* nothing works :-( */ 
    2285   free(name2); 
     2285  g_free (name2); 
    22862286  return(NULL); 
    22872287} 
    22882288 
    void free_namearray(name_compare_entry *name_array) 
    25332533    return; 
    25342534 
    25352535  if(name_array->name != NULL) 
    2536     free(name_array->name); 
     2536    g_free (name_array->name); 
    25372537 
    2538   free((char *)name_array); 
     2538  g_free ((char *)name_array); 
    25392539} 
    25402540 
    25412541/**************************************************************************** 
    int str_checksum(const char *s) 
    28412841/***************************************************************** 
    28422842zero a memory area then free it. Used to catch bugs faster 
    28432843*****************************************************************/   
    2844 void zero_free(void *p, size_t size) 
     2844void zero_free (void *p, size_t size) 
    28452845{ 
    28462846        memset(p, 0, size); 
    2847         free(p); 
     2847        g_free (p); 
    28482848} 
    28492849 
    28502850 
  • vfs/samba/lib/util_file.c

    diff --git a/vfs/samba/lib/util_file.c b/vfs/samba/lib/util_file.c
    index 502f8c7..9db0be2 100644
    a b char *fgets_slash(char *s2,int maxlen,FILE *f) 
    308308          return(s); 
    309309        case EOF: 
    310310          if (len <= 0 && !s2)  
    311             free(s); 
     311            g_free (s); 
    312312          return(len>0?s:NULL); 
    313313        case ' ': 
    314314          if (start_of_line) 
  • vfs/samba/lib/util_str.c

    diff --git a/vfs/samba/lib/util_str.c b/vfs/samba/lib/util_str.c
    index 74b5303..fd83067 100644
    a b void string_free(char **s) 
    980980  if (!s || !(*s)) return; 
    981981  if (*s == null_string) 
    982982    *s = NULL; 
    983   if (*s) free(*s); 
     983  if (*s) g_free (*s); 
    984984  *s = NULL; 
    985985} 
    986986 
  • vfs/samba/libsmb/clientgen.c

    diff --git a/vfs/samba/libsmb/clientgen.c b/vfs/samba/libsmb/clientgen.c
    index 0dd0710..05b7be3 100644
    a b BOOL cli_NetWkstaUserLogon(struct cli_state *cli,char *user, char *workstation) 
    528528        } 
    529529         
    530530        if (rparam) 
    531       free(rparam); 
     531      g_free (rparam); 
    532532        if (rdata) 
    533       free(rdata); 
     533      g_free (rdata); 
    534534        return (cli->rap_error == 0); 
    535535} 
    536536#endif /*0 */ 
    int cli_RNetShareEnum(struct cli_state *cli, void (*fn)(const char *, uint32, co 
    592592    } 
    593593   
    594594  if (rparam) 
    595     free(rparam); 
     595    g_free (rparam); 
    596596  if (rdata) 
    597     free(rdata); 
     597    g_free (rdata); 
    598598 
    599599  return count; 
    600600} 
    BOOL cli_NetServerEnum(struct cli_state *cli, char *workgroup, uint32 stype, 
    665665        } 
    666666   
    667667        if (rparam) 
    668       free(rparam); 
     668      g_free (rparam); 
    669669        if (rdata) 
    670       free(rdata); 
     670      g_free (rdata); 
    671671         
    672672        return(count > 0); 
    673673} 
    BOOL cli_qpathinfo(struct cli_state *cli, const char *fname, 
    17451745                *mode = SVAL(rdata,l1_attrFile); 
    17461746        } 
    17471747 
    1748         if (rdata) free(rdata); 
    1749         if (rparam) free(rparam); 
     1748        if (rdata) g_free (rdata); 
     1749        if (rparam) g_free (rparam); 
    17501750        return True; 
    17511751} 
    17521752 
    BOOL cli_qpathinfo2(struct cli_state *cli, const char *fname, 
    18121812                *ino = IVAL(rdata, 64); 
    18131813        } 
    18141814 
    1815         if (rdata) free(rdata); 
    1816         if (rparam) free(rparam); 
     1815        if (rdata) g_free (rdata); 
     1816        if (rparam) g_free (rparam); 
    18171817        return True; 
    18181818} 
    18191819#endif  /* 0 */ 
    BOOL cli_qfileinfo(struct cli_state *cli, int fnum, 
    18841884                *ino = IVAL(rdata, 64); 
    18851885        } 
    18861886 
    1887         if (rdata) free(rdata); 
    1888         if (rparam) free(rparam); 
     1887        if (rdata) g_free (rdata); 
     1888        if (rparam) g_free (rparam); 
    18891889        return True; 
    18901890} 
    18911891 
    int cli_list(struct cli_state *cli,const char *Mask,uint16 attribute, 
    21412141 
    21422142                total_received += ff_searchcount; 
    21432143 
    2144                 if (rdata) free(rdata); rdata = NULL; 
    2145                 if (rparam) free(rparam); rparam = NULL; 
     2144                if (rdata) g_free (rdata); rdata = NULL; 
     2145                if (rparam) g_free (rparam); rparam = NULL; 
    21462146                 
    21472147                DEBUG(3,("received %d entries (eos=%d resume=%d)\n", 
    21482148                         ff_searchcount,ff_eos,ff_resume_key)); 
    int cli_list(struct cli_state *cli,const char *Mask,uint16 attribute, 
    21562156        } 
    21572157 
    21582158        /* free up the dirlist buffer */ 
    2159         if (dirlist) free(dirlist); 
     2159        if (dirlist) g_free (dirlist); 
    21602160        return(total_received); 
    21612161} 
    21622162 
    void cli_shutdown(struct cli_state *cli) 
    24052405{ 
    24062406        if (cli->outbuf) 
    24072407        { 
    2408                 free(cli->outbuf); 
     2408                g_free (cli->outbuf); 
    24092409        } 
    24102410        if (cli->inbuf) 
    24112411        { 
    2412                 free(cli->inbuf); 
     2412                g_free (cli->inbuf); 
    24132413        } 
    24142414#ifdef WITH_SSL 
    24152415    if (cli->fd != -1)