[SCM] Samba Shared Repository - branch v3-6-test updated

Karolin Seeger kseeger at samba.org
Tue Jun 21 12:07:47 MDT 2011


The branch, v3-6-test has been updated
       via  8a7a3c6 s3: Fix Coverity ID 2582: FORWARD_NULL
      from  229c724 s3-spoolss: Fix some valgrind warnings.

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-6-test


- Log -----------------------------------------------------------------
commit 8a7a3c6704d8f2c48c8fbc2ad1bddb5b67131b4c
Author: Volker Lendecke <vl at samba.org>
Date:   Sun Jun 19 19:23:47 2011 +0200

    s3: Fix Coverity ID 2582: FORWARD_NULL
    
    Autobuild-User: Volker Lendecke <vlendec at samba.org>
    Autobuild-Date: Sun Jun 19 20:46:43 CEST 2011 on sn-devel-104
    (cherry picked from commit 5290faca7a5ae5f3f0309a42586768a5c93bfb9d)
    
    Fix bug #8247 (SMB2 shadow copy can be crashed remotely).

-----------------------------------------------------------------------

Summary of changes:
 source3/smbd/smb2_ioctl.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/smbd/smb2_ioctl.c b/source3/smbd/smb2_ioctl.c
index 3ee0453..21cc1be 100644
--- a/source3/smbd/smb2_ioctl.c
+++ b/source3/smbd/smb2_ioctl.c
@@ -398,6 +398,11 @@ static struct tevent_req *smbd_smb2_ioctl_send(TALLOC_CTX *mem_ctx,
 		char *pdata;
 		NTSTATUS status;
 
+		if (fsp == NULL) {
+			tevent_req_nterror(req, NT_STATUS_FILE_CLOSED);
+			return tevent_req_post(req, ev);
+		}
+
 		if (in_max_output < 16) {
 			DEBUG(0,("FSCTL_GET_SHADOW_COPY_DATA: "
 				 "in_max_output(%u) < 16 is invalid!\n",


-- 
Samba Shared Repository


More information about the samba-cvs mailing list