Fix various minor memory leaks on errors
This commit is contained in:
parent
7454f7165e
commit
029f8c2148
|
@ -1,4 +1,4 @@
|
|||
/* $NetBSD: xsess.c,v 1.7 2013/06/28 15:04:35 joerg Exp $ */
|
||||
/* $NetBSD: xsess.c,v 1.8 2015/08/08 10:38:35 shm Exp $ */
|
||||
|
||||
/*
|
||||
* Copyright (c) 2010 The NetBSD Foundation, Inc.
|
||||
|
@ -36,7 +36,7 @@
|
|||
* POSSIBILITY OF SUCH DAMAGE.
|
||||
*/
|
||||
#include <sys/cdefs.h>
|
||||
__RCSID("$NetBSD: xsess.c,v 1.7 2013/06/28 15:04:35 joerg Exp $");
|
||||
__RCSID("$NetBSD: xsess.c,v 1.8 2015/08/08 10:38:35 shm Exp $");
|
||||
|
||||
#include <assert.h>
|
||||
#include <saslc.h>
|
||||
|
@ -556,8 +556,10 @@ saslc__sess_xxcode(saslc_sess_t *sess, saslc__mech_xxcode_t xxcode,
|
|||
ate = 0;
|
||||
do {
|
||||
len = xxcode(sess, in, inlen, &pkt, &pktlen);
|
||||
if (len == -1) /* error */
|
||||
if (len == -1) {
|
||||
free(buf);
|
||||
return -1;
|
||||
}
|
||||
|
||||
ate += len;
|
||||
in = (const char *)in + len;
|
||||
|
@ -570,7 +572,10 @@ saslc__sess_xxcode(saslc_sess_t *sess, saslc__mech_xxcode_t xxcode,
|
|||
continue;
|
||||
|
||||
buflen += pktlen;
|
||||
p = buf;
|
||||
if ((buf = realloc(buf, buflen)) == NULL) {
|
||||
/* we should free memory if realloc(2) failed */
|
||||
free(p);
|
||||
saslc__error_set_errno(ERR(sess), ERROR_NOMEM);
|
||||
return -1;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue