From b45442e46d595337c17829aed2d846a8c8963e6e Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Tue, 20 Feb 2007 12:16:20 +0000 Subject: 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 (This used to be commit e7e6b8b5e0b00cc0746db4e9baa2e860074f903a) --- source3/smbd/sesssetup.c | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'source3') diff --git a/source3/smbd/sesssetup.c b/source3/smbd/sesssetup.c index 9b9ae4c353..6ca306ac25 100644 --- a/source3/smbd/sesssetup.c +++ b/source3/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); -- cgit