Fix a memleak in cli_qfilename (only used in smbtorture)
authorVolker Lendecke <vl@sernet.de>
Mon, 8 Sep 2008 20:39:16 +0000 (22:39 +0200)
committerVolker Lendecke <vl@samba.org>
Tue, 9 Sep 2008 07:54:36 +0000 (09:54 +0200)
(This used to be commit 7e0cca19fec078c0b46807492a7a035f4fab612b)

source3/libsmb/clirap.c

index 61e2fb7f1a3ded9fec27cf39f2cd519fdf79bb96..3b4db2e26da67c8b896f4d1c7f7b967d8efbeea1 100644 (file)
@@ -970,11 +970,16 @@ bool cli_qfilename(struct cli_state *cli, int fnum, char *name, size_t namelen)
        }
 
        if (!rdata || data_len < 4) {
+               SAFE_FREE(rparam);
+               SAFE_FREE(rdata);
                return False;
        }
 
        clistr_pull(cli, name, rdata+4, namelen, IVAL(rdata, 0), STR_UNICODE);
 
+       SAFE_FREE(rparam);
+       SAFE_FREE(rdata);
+
        return True;
 }