tags 520794 patch
thanks

Quoting Volker Lendecke (volker.lende...@sernet.de):
> On Mon, Mar 23, 2009 at 07:10:51AM +0100, Christian Perrier wrote:
> > Volker (Lendecke, CC'ed), you were the one fixing this. Do you have
> > the patch you used handy somewhere so that we see if we can backport
> > it to Debian's 3.2.5?
> 
> http://git.samba.org/?p=samba.git;a=commitdiff;h=cc70e0f88328e3641008


OK, thanks Volker.

For reference in the Debian BTS, here's the patch that applies to
3.2.5 and could then be applied in the next point release.

Steve, any objection to include that one in the next point release?

Goal: fix memory leak in vfs_full_audit

Fixes: #520794

Status wrt upstream: Fixed in 3.2.8

Author: Volker Lendecke <v...@samba.org>

Index: lenny/source/modules/vfs_full_audit.c
===================================================================
--- lenny.orig/source/modules/vfs_full_audit.c
+++ lenny/source/modules/vfs_full_audit.c
@@ -701,6 +701,7 @@
 static char *audit_prefix(TALLOC_CTX *ctx, connection_struct *conn)
 {
 	char *prefix = NULL;
+	char *result;
 
 	prefix = talloc_strdup(ctx,
 			lp_parm_const_string(SNUM(conn), "full_audit",
@@ -708,12 +709,14 @@
 	if (!prefix) {
 		return NULL;
 	}
-	return talloc_sub_advanced(ctx,
+	result = talloc_sub_advanced(ctx,
 			lp_servicename(SNUM(conn)), conn->user,
 			conn->connectpath, conn->gid,
 			get_current_username(),
 			current_user_info.domain,
 			prefix);
+	TALLOC_FREE(prefix);
+	return result;
 }
 
 static bool log_success(vfs_handle_struct *handle, vfs_op_type op)

Attachment: signature.asc
Description: Digital signature

Reply via email to