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)));

Reply via email to