diff options
author | Volker Lendecke <vlendec@samba.org> | 2007-02-20 12:16:20 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 12:18:06 -0500 |
commit | e7e6b8b5e0b00cc0746db4e9baa2e860074f903a (patch) | |
tree | 1d1971d1755f52f59c08253b143ad28701b4327f /source/smbd/sesssetup.c | |
parent | aa69f2481aafee5dccc3783b8a6e23ca4eb0dbfa (diff) | |
download | samba-e7e6b8b5e0b00cc0746db4e9baa2e860074f903a.tar.gz samba-e7e6b8b5e0b00cc0746db4e9baa2e860074f903a.tar.xz samba-e7e6b8b5e0b00cc0746db4e9baa2e860074f903a.zip |
r21461: Not strictly necessary, as data_blob() panics if it can't allocate. But I'd
see this as a design flaw in data_blob() and it made me look in that routine.
Jeremy, revert or merge please :-)
Volker
Diffstat (limited to 'source/smbd/sesssetup.c')
-rw-r--r-- | source/smbd/sesssetup.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/source/smbd/sesssetup.c b/source/smbd/sesssetup.c index 9b9ae4c3533..6ca306ac25b 100644 --- a/source/smbd/sesssetup.c +++ b/source/smbd/sesssetup.c @@ -828,6 +828,10 @@ static NTSTATUS check_spnego_blob_complete(uint16 smbpid, uint16 vuid, DATA_BLOB } pad->needed_len = needed_len - pblob->length; pad->partial_data = data_blob(pblob->data, pblob->length); + if (pad->partial_data.data == NULL) { + SAFE_FREE(pad); + return NT_STATUS_NO_MEMORY; + } pad->smbpid = smbpid; pad->vuid = vuid; DLIST_ADD(pd_list, pad); |