diff options
| author | Volker Lendecke <vl@sernet.de> | 2008-09-08 22:39:16 +0200 |
|---|---|---|
| committer | Volker Lendecke <vl@samba.org> | 2008-09-09 09:54:36 +0200 |
| commit | 7fafa9756a273e200fa251c1dea25e884729628c (patch) | |
| tree | 444743dc2b0c89bc639961ce50d746a14555bc56 /source3/libsmb/clirap.c | |
| parent | d39e9e05ed485f93bb2709ec2494dd2c42ea1300 (diff) | |
| download | samba-7fafa9756a273e200fa251c1dea25e884729628c.tar.gz samba-7fafa9756a273e200fa251c1dea25e884729628c.tar.bz2 samba-7fafa9756a273e200fa251c1dea25e884729628c.zip | |
Fix a memleak in cli_qfilename (only used in smbtorture)
(This used to be commit 7e0cca19fec078c0b46807492a7a035f4fab612b)
Diffstat (limited to 'source3/libsmb/clirap.c')
| -rw-r--r-- | source3/libsmb/clirap.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/source3/libsmb/clirap.c b/source3/libsmb/clirap.c index 61e2fb7f1a..3b4db2e26d 100644 --- a/source3/libsmb/clirap.c +++ b/source3/libsmb/clirap.c @@ -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; } |