Memory leak fixes from insure.

2 files changed, 5 insertions, 10 deletions

diff --git a/source/nsswitch/winbindd_glue.c b/source/nsswitch/winbindd_glue.c
index 3c789d5a7c8..71b802b68d0 100644
--- a/source/nsswitch/winbindd_glue.c
+++ b/source/nsswitch/winbindd_glue.c
@@ -42,6 +42,8 @@ BOOL wb_lsa_open_policy(char *server, BOOL sec_qos, uint32 des_access,
 	pol->cli = (struct cli_state *)malloc(sizeof(struct cli_state));
 	pol->mem_ctx = talloc_init();
 
+	ZERO_STRUCTP(pol->cli);
+
 	if (!pol->cli || !pol->mem_ctx)
 		return False;
 
@@ -183,6 +185,9 @@ BOOL wb_samr_connect(char *server, uint32 access_mask, CLI_POLICY_HND *pol)
 	ZERO_STRUCTP(pol);
 
 	pol->cli = (struct cli_state *)malloc(sizeof(struct cli_state));
+
+	ZERO_STRUCTP(pol->cli);
+
 	pol->mem_ctx = talloc_init();
 
 	if (!pol->cli || !pol->mem_ctx)
diff --git a/source/nsswitch/winbindd_util.c b/source/nsswitch/winbindd_util.c
index 4343f859ed3..03f9ee13cb0 100644
--- a/source/nsswitch/winbindd_util.c
+++ b/source/nsswitch/winbindd_util.c
@@ -530,11 +530,6 @@ BOOL winbindd_lookup_sid_by_name(char *name, DOM_SID *sid,
         }
     }
     
-    /* Free memory */
-
-    if (types != NULL) free(types);
-    if (sids != NULL) free(sids);
-
     return res;
 }
 
@@ -570,11 +565,6 @@ BOOL winbindd_lookup_name_by_sid(DOM_SID *sid, fstring name,
         }
     }
 
-    /* Free memory */
-
-    safe_free(types);
-    free_char_array(num_names, names);
-
     return res;
 }