[SCM] Samba Shared Repository - branch master updated

Jeremy Allison jra at samba.org
Tue Jan 5 15:36:21 MST 2010


The branch, master has been updated
       via  f6bd654... Fix bug #7020 - smbd using 2G memory.
      from  a08a0ff... s4:SAMLDB LDB module - Fix trailing whitespaces

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit f6bd654a6c878111f0f0f4d07adb644fc3bbf224
Author: Jeremy Allison <jra at samba.org>
Date:   Tue Jan 5 14:34:00 2010 -0800

    Fix bug #7020 - smbd using 2G memory.
    
    When deferring an async pipe writeX and readX transfer
    the outstanding request struct onto the conn struct. This
    needs freeing after the packet is finally processed.
    
    Jeremy.

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

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


Changeset truncated at 500 lines:

diff --git a/source3/smbd/pipes.c b/source3/smbd/pipes.c
index bf64c59..9bc3fdf 100644
--- a/source3/smbd/pipes.c
+++ b/source3/smbd/pipes.c
@@ -353,6 +353,11 @@ static void pipe_write_andx_done(struct tevent_req *subreq)
 
  done:
 	chain_reply(req);
+	/*
+	 * We must free here as the ownership of req was
+	 * moved to the connection struct in reply_pipe_write_and_X().
+	 */
+	TALLOC_FREE(req);
 }
 
 /****************************************************************************
@@ -458,4 +463,9 @@ static void pipe_read_andx_done(struct tevent_req *subreq)
 
  done:
 	chain_reply(req);
+	/*
+	 * We must free here as the ownership of req was
+	 * moved to the connection struct in reply_pipe_read_and_X().
+	 */
+	TALLOC_FREE(req);
 }


-- 
Samba Shared Repository


More information about the samba-cvs mailing list