svn commit: samba r24420 - in branches: SAMBA_3_2/source/nsswitch SAMBA_3_2_0/source/nsswitch

gd at samba.org gd at samba.org
Tue Aug 14 15:34:35 GMT 2007


Author: gd
Date: 2007-08-14 15:34:34 +0000 (Tue, 14 Aug 2007)
New Revision: 24420

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=24420

Log:
Pure pedantism: the talloc_init could have failed...

Guenther

Modified:
   branches/SAMBA_3_2/source/nsswitch/winbindd_dual.c
   branches/SAMBA_3_2_0/source/nsswitch/winbindd_dual.c


Changeset:
Modified: branches/SAMBA_3_2/source/nsswitch/winbindd_dual.c
===================================================================
--- branches/SAMBA_3_2/source/nsswitch/winbindd_dual.c	2007-08-14 15:31:36 UTC (rev 24419)
+++ branches/SAMBA_3_2/source/nsswitch/winbindd_dual.c	2007-08-14 15:34:34 UTC (rev 24420)
@@ -758,9 +758,8 @@
 	} else {
 		result = methods->lockout_policy(child->domain, mem_ctx, &lockout_policy);
 	}
+	TALLOC_FREE(mem_ctx);
 
-	talloc_destroy(mem_ctx);
-
 	if (!NT_STATUS_IS_OK(result)) {
 		DEBUG(10,("account_lockout_policy_handler: lockout_policy failed error %s\n",
 			 nt_errstr(result)));

Modified: branches/SAMBA_3_2_0/source/nsswitch/winbindd_dual.c
===================================================================
--- branches/SAMBA_3_2_0/source/nsswitch/winbindd_dual.c	2007-08-14 15:31:36 UTC (rev 24419)
+++ branches/SAMBA_3_2_0/source/nsswitch/winbindd_dual.c	2007-08-14 15:34:34 UTC (rev 24420)
@@ -758,9 +758,8 @@
 	} else {
 		result = methods->lockout_policy(child->domain, mem_ctx, &lockout_policy);
 	}
+	TALLOC_FREE(mem_ctx);
 
-	talloc_destroy(mem_ctx);
-
 	if (!NT_STATUS_IS_OK(result)) {
 		DEBUG(10,("account_lockout_policy_handler: lockout_policy failed error %s\n",
 			 nt_errstr(result)));



More information about the samba-cvs mailing list