r5941: Commit this patch much earlier than I would normally prefer, but metze needs a working tree...

The main volume of this patch was what I started working on today:
 - Cleans up memory handling around DCE/RPC pipes, to have a parent talloc context.
 - Uses sepereate inner loops for some of the DCE/RPC tests

The other and more important part of this patch fixes issues
surrounding the new credentials framwork:

This makes the struct cli_credentials always a talloc() structure,
rather than on the stack.  Parts of the cli_credentials code already
assumed this.

There were other issues, particularly in the DCERPC over SMB handling,
as well as little things that had to be tidied up before test_w2k3.sh
would start to pass.

Andrew Bartlett
(This used to be commit 0453f9d05d2e336fba1f85dbf2718d01fa2bf778)

1 files changed, 3 insertions, 2 deletions

diff --git a/source4/gtk/tools/gwcrontab.c b/source4/gtk/tools/gwcrontab.c
index 47a7d68dc9..a1b57225db 100644
--- a/source4/gtk/tools/gwcrontab.c
+++ b/source4/gtk/tools/gwcrontab.c
@@ -103,7 +103,7 @@ on_connect_activate                    (GtkMenuItem     *menuitem,
 	mem_ctx = talloc_init("gwcrontab_connect");
 	/* If connected, get list of jobs */
 	
-	status = dcerpc_pipe_connect_b(&at_pipe,
+	status = dcerpc_pipe_connect_b(mem_ctx, &at_pipe,
 				       gtk_rpc_binding_dialog_get_binding(d, mem_ctx),
 				       DCERPC_ATSVC_UUID,
 				       DCERPC_ATSVC_VERSION,
@@ -118,6 +118,7 @@ on_connect_activate                    (GtkMenuItem     *menuitem,
 	}
 	gtk_widget_destroy(GTK_WIDGET(d));
 
+	at_pipe = talloc_reference(talloc_autofree_context(), at_pipe);
 	talloc_free(mem_ctx);
 	update_joblist();
 }
@@ -126,7 +127,7 @@ void
 on_quit_activate                      (GtkMenuItem     *menuitem,
 									   gpointer         user_data)
 {
-	if(at_pipe)dcerpc_pipe_close(at_pipe);
+	talloc_free(at_pipe);
 	gtk_main_quit();
 }