svn commit: samba r9753 - in branches/tmp/RPCREWRITE/source/rpc_server: .

jra at samba.org jra at samba.org
Mon Aug 29 17:54:15 GMT 2005


Author: jra
Date: 2005-08-29 17:54:14 +0000 (Mon, 29 Aug 2005)
New Revision: 9753

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

Log:
Merge with head.
Jeremy.

Modified:
   branches/tmp/RPCREWRITE/source/rpc_server/srv_spoolss_nt.c


Changeset:
Modified: branches/tmp/RPCREWRITE/source/rpc_server/srv_spoolss_nt.c
===================================================================
--- branches/tmp/RPCREWRITE/source/rpc_server/srv_spoolss_nt.c	2005-08-29 17:48:01 UTC (rev 9752)
+++ branches/tmp/RPCREWRITE/source/rpc_server/srv_spoolss_nt.c	2005-08-29 17:54:14 UTC (rev 9753)
@@ -4171,7 +4171,11 @@
 	if ( ntprinter->info_2->secdesc_buf 
 		&& ntprinter->info_2->secdesc_buf->len != 0 ) 
 	{
-		printer->secdesc = dup_sec_desc( get_talloc_ctx(), ntprinter->info_2->secdesc_buf->sec );
+		/* don't use talloc_steal() here unless you do a deep steal of all 
+		   the SEC_DESC members */
+
+		printer->secdesc = dup_sec_desc( get_talloc_ctx(), 
+			ntprinter->info_2->secdesc_buf->sec );
 	}
 
 	free_a_printer(&ntprinter, 2);
@@ -4205,7 +4209,11 @@
 	printer->flags = 0x4; 
 
 	if (ntprinter->info_2->secdesc_buf && ntprinter->info_2->secdesc_buf->len != 0) {
-		printer->secdesc = dup_sec_desc( get_talloc_ctx(), ntprinter->info_2->secdesc_buf->sec );
+		/* don't use talloc_steal() here unless you do a deep steal of all 
+		   the SEC_DESC members */
+
+		printer->secdesc = dup_sec_desc( get_talloc_ctx(), 
+			ntprinter->info_2->secdesc_buf->sec );
 	}
 
 	free_a_printer(&ntprinter, 2);



More information about the samba-cvs mailing list