Attachment 5211 Details for Bug 7021 – Backported patch from #7020

[patch] Backported patch from #7020

patch1 (text/plain), 1.09 KB, created by Debian samba package maintainers (PUBLIC MAILING LIST) on 2010-01-21 01:19:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Debian samba package maintainers (PUBLIC MAILING LIST)

Created: 2010-01-21 01:19:03 UTC

Size: 1.09 KB

patch

obsolete

>From 1e7411ee9afb148c517a0ec82d5684b347e8f564 Mon Sep 17 00:00:00 2001
>From: Jeremy Allison <jra@samba.org>
>Date: Wed, 6 Jan 2010 13:29:35 -0800
>Subject: [PATCH] Second part of the fix for bug #7020 - smbd using 2G memory.
>
>There was a second leak in the processing of the out_data.frag
>prs_struct. It needs freeing once the current pdu has been returned
>asynchronously.
>
>Jeremy.
>---
> source/rpc_server/srv_pipe_hnd.c |    7 +++++++
> 1 files changed, 7 insertions(+), 0 deletions(-)
>
>Index: samba-3.2.5/source/rpc_server/srv_pipe_hnd.c
>===================================================================
>--- samba-3.2.5.orig/source/rpc_server/srv_pipe_hnd.c
>+++ samba-3.2.5/source/rpc_server/srv_pipe_hnd.c
>@@ -1086,6 +1086,14 @@
>   out:
> 
> 	(*is_data_outstanding) = p->out_data.current_pdu_len > n;
>+
>+	if (p->out_data.current_pdu_sent == prs_offset(&p->out_data.frag)) {
>+		/* We've returned everything in the out_data.frag
>+		 * so we're done with this pdu. Free it and reset
>+		 * current_pdu_sent. */
>+		p->out_data.current_pdu_sent = 0;
>+		prs_mem_free(&p->out_data.frag);
>+	}
> 	return data_returned;
> }
>

Actions: View

Attachments on bug 7021: 5165 | 5166 | 5201 | 5203 | 5204 | 5205 | 5207 | 5209 | 5211 | 5212