diff options
author | Jeremy Allison <jra@samba.org> | 2003-03-07 00:34:43 +0000 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2003-03-07 00:34:43 +0000 |
commit | 1cbd53dc3cd2d56f5e62d42fed5d589efcecb8f8 (patch) | |
tree | 8837bdc723c842911c8d5e5683d5c85bb31b044f /source/smbd/ipc.c | |
parent | 172f415c9eaa4d310a52794e3dc1a6bc92206ce4 (diff) | |
download | samba-1cbd53dc3cd2d56f5e62d42fed5d589efcecb8f8.tar.gz samba-1cbd53dc3cd2d56f5e62d42fed5d589efcecb8f8.tar.xz samba-1cbd53dc3cd2d56f5e62d42fed5d589efcecb8f8.zip |
Fix noticed by Corny.Bondad@hp.com. Ensure we free up memory on error exit.
Jeremy.
Diffstat (limited to 'source/smbd/ipc.c')
-rw-r--r-- | source/smbd/ipc.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/source/smbd/ipc.c b/source/smbd/ipc.c index 08337c2901a..67c5da01f33 100644 --- a/source/smbd/ipc.c +++ b/source/smbd/ipc.c @@ -397,6 +397,7 @@ int reply_trans(connection_struct *conn, char *inbuf,char *outbuf, int size, int if (tpscnt) { if((params = (char *)malloc(tpscnt)) == NULL) { DEBUG(0,("reply_trans: param malloc fail for %u bytes !\n", tpscnt)); + SAFE_FREE(data); END_PROFILE(SMBtrans); return(ERROR_DOS(ERRDOS,ERRnomem)); } @@ -412,6 +413,8 @@ int reply_trans(connection_struct *conn, char *inbuf,char *outbuf, int size, int int i; if((setup = (uint16 *)malloc(suwcnt*sizeof(uint16))) == NULL) { DEBUG(0,("reply_trans: setup malloc fail for %u bytes !\n", (unsigned int)(suwcnt * sizeof(uint16)))); + SAFE_FREE(data); + SAFE_FREE(params); END_PROFILE(SMBtrans); return(ERROR_DOS(ERRDOS,ERRnomem)); } |