diff options
author | Andreas Schneider <asn@cryptomilk.org> | 2019-04-29 10:39:33 +0200 |
---|---|---|
committer | Andreas Schneider <asn@cryptomilk.org> | 2019-04-29 13:17:21 +0200 |
commit | 643ca67f88817b0176827554cc209033b1d9e35b (patch) | |
tree | 9ea09bbea4ec694faa99c5c26d224721da92f171 /src/gssapi.c | |
parent | 8a885f0bd38987ef135ff1638eeeb98a0a1edd70 (diff) | |
download | libssh-643ca67f88817b0176827554cc209033b1d9e35b.tar.gz libssh-643ca67f88817b0176827554cc209033b1d9e35b.tar.xz libssh-643ca67f88817b0176827554cc209033b1d9e35b.zip |
gssapi: Add missing malloc checks
Fixes T141
Reported-By: Ramin Farajpour Cami
Signed-off-by: Andreas Schneider <asn@cryptomilk.org>
Reviewed-by: Jakub Jelen <jjelen@redhat.com>
Diffstat (limited to 'src/gssapi.c')
-rw-r--r-- | src/gssapi.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/src/gssapi.c b/src/gssapi.c index f2ca38df..5c3fb61b 100644 --- a/src/gssapi.c +++ b/src/gssapi.c @@ -328,7 +328,10 @@ static char *ssh_gssapi_name_to_char(gss_name_t name){ "converting name", maj_stat, min_stat); - ptr=malloc(buffer.length + 1); + ptr = malloc(buffer.length + 1); + if (ptr == NULL) { + return NULL; + } memcpy(ptr, buffer.value, buffer.length); ptr[buffer.length] = '\0'; gss_release_buffer(&min_stat, &buffer); @@ -794,6 +797,10 @@ static gss_OID ssh_gssapi_oid_from_string(ssh_string oid_s){ return NULL; } ret->elements = malloc(len - 2); + if (ret->elements == NULL) { + SAFE_FREE(ret); + return NULL; + } memcpy(ret->elements, &data[2], len-2); ret->length = len-2; |