svn commit: samba r19846 - in branches/SAMBA_4_0/source: librpc/idl torture/rpc

jelmer at samba.org jelmer at samba.org
Wed Nov 22 18:06:09 GMT 2006


Author: jelmer
Date: 2006-11-22 18:06:08 +0000 (Wed, 22 Nov 2006)
New Revision: 19846

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

Log:
Fix bunch of IDL warnings.

Modified:
   branches/SAMBA_4_0/source/librpc/idl/oxidresolver.idl
   branches/SAMBA_4_0/source/librpc/idl/remact.idl
   branches/SAMBA_4_0/source/librpc/idl/rot.idl
   branches/SAMBA_4_0/source/librpc/idl/unixinfo.idl
   branches/SAMBA_4_0/source/librpc/idl/winbind.idl
   branches/SAMBA_4_0/source/librpc/idl/winreg.idl
   branches/SAMBA_4_0/source/librpc/idl/winsrepl.idl
   branches/SAMBA_4_0/source/librpc/idl/winstation.idl
   branches/SAMBA_4_0/source/torture/rpc/unixinfo.c


Changeset:
Modified: branches/SAMBA_4_0/source/librpc/idl/oxidresolver.idl
===================================================================
--- branches/SAMBA_4_0/source/librpc/idl/oxidresolver.idl	2006-11-22 17:13:19 UTC (rev 19845)
+++ branches/SAMBA_4_0/source/librpc/idl/oxidresolver.idl	2006-11-22 18:06:08 UTC (rev 19846)
@@ -16,8 +16,7 @@
 	uuid("99fcfec4-5260-101b-bbcb-00aa0021347a"),
 	helpstring("Object Exporter ID Resolver"),
 	endpoint("ncacn_np:[\\pipe\\epmapper]", "ncacn_ip_tcp:[135]", "ncalrpc:"),
-	pointer_default(unique),
-  	pointer_default_top(unique)
+	pointer_default(unique)
 ]
 interface IOXIDResolver
 {
@@ -33,7 +32,7 @@
 		 [in] OXID pOxid,
 		 [in] uint16 cRequestedProtseqs,
 		 [in, size_is(cRequestedProtseqs)] uint16 arRequestedProtseqs[],
-		 [out] DUALSTRINGARRAY *ppdsaOxidBindings,
+		 [out] DUALSTRINGARRAY **ppdsaOxidBindings,
 		 [out,ref] IPID *pipidRemUnknown,
 		 [out,ref] uint32 *pAuthnHint
 			 );
@@ -78,7 +77,7 @@
 											  [in] OXID pOxid,
 											  [in] uint16 cRequestedProtseqs,
 											  [in, size_is(cRequestedProtseqs)] uint16 arRequestedProtseqs[],
-											  [out] DUALSTRINGARRAY *pdsaOxidBindings,
+											  [out] DUALSTRINGARRAY **pdsaOxidBindings,
 											  [out,ref] IPID *ipidRemUnknown,
 											  [out,ref] uint32 *AuthnHint,
 											  [out,ref] COMVERSION *ComVersion

Modified: branches/SAMBA_4_0/source/librpc/idl/remact.idl
===================================================================
--- branches/SAMBA_4_0/source/librpc/idl/remact.idl	2006-11-22 17:13:19 UTC (rev 19845)
+++ branches/SAMBA_4_0/source/librpc/idl/remact.idl	2006-11-22 18:06:08 UTC (rev 19846)
@@ -8,7 +8,6 @@
 [
 	uuid("4d9f4ab8-7d1c-11cf-861e-0020af6e7c57"),
 	pointer_default(unique),
-  	pointer_default_top(unique),
 	endpoint("ncalrpc:", "ncacn_ip_tcp:[135]", "ncacn_np:[\\pipe\\epmapper]")
 ]
 interface IRemoteActivation

Modified: branches/SAMBA_4_0/source/librpc/idl/rot.idl
===================================================================
--- branches/SAMBA_4_0/source/librpc/idl/rot.idl	2006-11-22 17:13:19 UTC (rev 19845)
+++ branches/SAMBA_4_0/source/librpc/idl/rot.idl	2006-11-22 18:06:08 UTC (rev 19846)
@@ -4,7 +4,6 @@
 	uuid("b9e79e60-3d52-11ce-aaa1-00006901293f"),
 	version(0.2),
     	pointer_default(unique),
-  	pointer_default_top(unique),
  	endpoint("ncacn_np:[\\pipe\\epmapper]", "ncacn_ip_tcp:[135]", 
 		  "ncalrpc:[EPMAPPER]")
 ] interface rot
@@ -13,7 +12,7 @@
 					[in] uint32 flags, 
 					[in] MInterfacePointer *unk, 
 					[in] MInterfacePointer *moniker, 
-					[out,ref] uint32 *rotid
+					[out] uint32 *rotid
 					);
 	
 	WERROR rot_remove (
@@ -26,7 +25,7 @@
 	
 	WERROR rot_get_interface_pointer (
 					 [in] MInterfacePointer *moniker, 
-					 [out,ref] MInterfacePointer *ip
+					 [out] MInterfacePointer *ip
 					 );
 	
 	WERROR rot_set_modification_time (
@@ -36,7 +35,7 @@
 	
 	WERROR rot_get_modification_time (
 							   [in] MInterfacePointer *moniker,
-							   [out,ref] NTTIME *t
+							   [out] NTTIME *t
 							   );
 	
 	WERROR rot_enum (

Modified: branches/SAMBA_4_0/source/librpc/idl/unixinfo.idl
===================================================================
--- branches/SAMBA_4_0/source/librpc/idl/unixinfo.idl	2006-11-22 17:13:19 UTC (rev 19845)
+++ branches/SAMBA_4_0/source/librpc/idl/unixinfo.idl	2006-11-22 18:06:08 UTC (rev 19846)
@@ -16,28 +16,28 @@
 	/* Function: 0x00 */
 	NTSTATUS unixinfo_SidToUid (
 	    [in]	dom_sid sid,
-	    [out,ref]	hyper *uid
+	    [out]	hyper *uid
 	    );
 
 	/******************/
 	/* Function: 0x01 */
 	NTSTATUS unixinfo_UidToSid (
 	    [in]	hyper uid,
-	    [out,unique]	dom_sid *sid
+	    [out]	dom_sid *sid
 	    );
 
 	/******************/
 	/* Function: 0x02 */
 	NTSTATUS unixinfo_SidToGid (
 	    [in]	dom_sid sid,
-	    [out,ref]	hyper *gid
+	    [out]	hyper *gid
 	    );
 
 	/******************/
 	/* Function: 0x03 */
 	NTSTATUS unixinfo_GidToSid (
 	    [in]	hyper gid,
-	    [out,unique]	dom_sid *sid
+	    [out]	dom_sid *sid
 	    );
 
 	typedef struct {

Modified: branches/SAMBA_4_0/source/librpc/idl/winbind.idl
===================================================================
--- branches/SAMBA_4_0/source/librpc/idl/winbind.idl	2006-11-22 17:13:19 UTC (rev 19845)
+++ branches/SAMBA_4_0/source/librpc/idl/winbind.idl	2006-11-22 18:06:08 UTC (rev 19846)
@@ -9,10 +9,8 @@
 [
   uuid("245f3e6b-3c5d-6e21-3a2d-2a3d645b7221"),
   version(1.0),
-  pointer_default(unique),
-  pointer_default_top(unique)
+  pointer_default(unique)
 ]
-
 interface winbind
 {
 	declare [switch_type(uint16)] union netr_LogonLevel;

Modified: branches/SAMBA_4_0/source/librpc/idl/winreg.idl
===================================================================
--- branches/SAMBA_4_0/source/librpc/idl/winreg.idl	2006-11-22 17:13:19 UTC (rev 19845)
+++ branches/SAMBA_4_0/source/librpc/idl/winreg.idl	2006-11-22 18:06:08 UTC (rev 19846)
@@ -210,7 +210,7 @@
 	WERROR winreg_QueryInfoKey(
 		[in,ref] policy_handle *handle,
 		[in] winreg_String class_in,
-		[out] winreg_String *class_out,
+		[out,unique] winreg_String *class_out,
 		[out,ref] uint32 *num_subkeys,
 		[out,ref] uint32 *max_subkeylen,
 		[out,ref] uint32 *max_subkeysize,

Modified: branches/SAMBA_4_0/source/librpc/idl/winsrepl.idl
===================================================================
--- branches/SAMBA_4_0/source/librpc/idl/winsrepl.idl	2006-11-22 17:13:19 UTC (rev 19845)
+++ branches/SAMBA_4_0/source/librpc/idl/winsrepl.idl	2006-11-22 18:06:08 UTC (rev 19846)
@@ -11,10 +11,6 @@
 
 import "nbt.idl";
 
-[
-  pointer_default(unique),
-  pointer_default_top(unique)
-]
 interface wrepl
 {
 	const int WINS_REPLICATION_PORT = 42;

Modified: branches/SAMBA_4_0/source/librpc/idl/winstation.idl
===================================================================
--- branches/SAMBA_4_0/source/librpc/idl/winstation.idl	2006-11-22 17:13:19 UTC (rev 19845)
+++ branches/SAMBA_4_0/source/librpc/idl/winstation.idl	2006-11-22 18:06:08 UTC (rev 19846)
@@ -6,8 +6,6 @@
 
 [ uuid("5ca4a760-ebb1-11cf-8611-00a0245420ed"),
   version(1.0),
-  pointer_default(unique),
-  pointer_default_top(unique),
   helpstring("Terminal Services remote management")
 ] interface winstation
 {

Modified: branches/SAMBA_4_0/source/torture/rpc/unixinfo.c
===================================================================
--- branches/SAMBA_4_0/source/torture/rpc/unixinfo.c	2006-11-22 17:13:19 UTC (rev 19845)
+++ branches/SAMBA_4_0/source/torture/rpc/unixinfo.c	2006-11-22 18:06:08 UTC (rev 19846)
@@ -25,8 +25,7 @@
 #include "librpc/gen_ndr/ndr_unixinfo_c.h"
 #include "libcli/security/security.h"
 
-
-/*
+/**
   test the SidToUid interface
 */
 static BOOL test_sidtouid(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx)
@@ -41,7 +40,7 @@
 	status = dcerpc_unixinfo_SidToUid(p, mem_ctx, &r);
 	if (NT_STATUS_EQUAL(NT_STATUS_NONE_MAPPED, status)) {
 	} else if (!NT_STATUS_IS_OK(status)) {
-		printf("UidToSid failed == %s\n", nt_errstr(status));
+		printf("SidToUid failed == %s\n", nt_errstr(status));
 		return False;
 	}
 
@@ -55,8 +54,10 @@
 						  struct dcerpc_pipe *p)
 {
 	struct unixinfo_UidToSid r;
+	struct dom_sid sid;
 
 	r.in.uid = 1000;
+	r.out.sid = &sid;
 
 	torture_assert_ntstatus_ok(tctx, dcerpc_unixinfo_UidToSid(p, tctx, &r), 
 							   "UidToSid failed");
@@ -118,8 +119,10 @@
 {
 	NTSTATUS status;
 	struct unixinfo_GidToSid r;
+	struct dom_sid sid;
 
 	r.in.gid = 1000;
+	r.out.sid = &sid;
 
 	status = dcerpc_unixinfo_GidToSid(p, tctx, &r);
 	if (NT_STATUS_EQUAL(NT_STATUS_NO_SUCH_GROUP, status)) {



More information about the samba-cvs mailing list