svn commit: samba r20418 - in branches/SAMBA_4_0/source: libnet librpc/idl

[metze at samba.org]

Author: metze
Date: 2006-12-30 10:51:04 +0000 (Sat, 30 Dec 2006)
New Revision: 20418

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

Log:
start to get step by step rid of the union for the attribute
specific attribute values.

start with ntSecurityDescriptor

metze
Modified:
   branches/SAMBA_4_0/source/libnet/libnet_become_dc.c
   branches/SAMBA_4_0/source/librpc/idl/drsuapi.idl


Changeset:
Modified: branches/SAMBA_4_0/source/libnet/libnet_become_dc.c
===================================================================
--- branches/SAMBA_4_0/source/libnet/libnet_become_dc.c	2006-12-30 10:42:01 UTC (rev 20417)
+++ branches/SAMBA_4_0/source/libnet/libnet_become_dc.c	2006-12-30 10:51:04 UTC (rev 20418)
@@ -29,6 +29,7 @@
 #include "dsdb/common/flags.h"
 #include "librpc/gen_ndr/ndr_drsuapi_c.h"
 #include "libcli/security/security.h"
+#include "librpc/gen_ndr/ndr_security.h"
 
 struct libnet_BecomeDC_state {
 	struct composite_context *creq;
@@ -1050,21 +1051,25 @@
 
 	/* ntSecurityDescriptor */
 	{
-		struct drsuapi_DsAttributeValueSecurityDescriptor *vs;
+		struct drsuapi_DsAttributeValueDataBlob *vs;
+		DATA_BLOB *vd;
 		struct security_descriptor *v;
 		struct dom_sid *domain_admins_sid;
 		const char *domain_admins_sid_str;
 
-		vs = talloc_array(attrs, struct drsuapi_DsAttributeValueSecurityDescriptor, 1);
+		vs = talloc_array(attrs, struct drsuapi_DsAttributeValueDataBlob, 1);
 		if (composite_nomem(vs, c)) return;
 
+		vd = talloc_array(vs, DATA_BLOB, 1);
+		if (composite_nomem(vd, c)) return;
+
 		domain_admins_sid = dom_sid_add_rid(vs, s->domain.sid, DOMAIN_RID_ADMINS);
 		if (composite_nomem(domain_admins_sid, c)) return;
 
 		domain_admins_sid_str = dom_sid_string(domain_admins_sid, domain_admins_sid);
 		if (composite_nomem(domain_admins_sid_str, c)) return;
 
-		v = security_descriptor_create(vs,
+		v = security_descriptor_create(vd,
 					       /* owner: domain admins */
 					       domain_admins_sid_str,
 					       /* owner group: domain admins */
@@ -1108,12 +1113,15 @@
 					       NULL);
 		if (composite_nomem(v, c)) return;
 
-		vs[0].sd		= v;
+		c->status = ndr_push_struct_blob(&vd[0], r, v,(ndr_push_flags_fn_t)ndr_push_security_descriptor);
+		if (!composite_is_ok(c)) return;
 
-		attrs[i].attid						= DRSUAPI_ATTRIBUTE_ntSecurityDescriptor;
-		attrs[i].value_ctr.security_descriptor.num_values	= 1;
-		attrs[i].value_ctr.security_descriptor.values		= vs;
+		vs[0].data		= &vd[0];
 
+		attrs[i].attid					= DRSUAPI_ATTRIBUTE_ntSecurityDescriptor;
+		attrs[i].value_ctr.data_blob.num_values		= 1;
+		attrs[i].value_ctr.data_blob.values		= vs;
+
 		i++;
 	}
 

Modified: branches/SAMBA_4_0/source/librpc/idl/drsuapi.idl
===================================================================
--- branches/SAMBA_4_0/source/librpc/idl/drsuapi.idl	2006-12-30 10:42:01 UTC (rev 20417)
+++ branches/SAMBA_4_0/source/librpc/idl/drsuapi.idl	2006-12-30 10:51:04 UTC (rev 20418)
@@ -519,17 +519,6 @@
 		[size_is(num_values)] drsuapi_DsAttributeValueSID *values;
 	} drsuapi_DsAttributeValueCtrSID;
 
-	/* SecurityDescriptor values */
-	typedef struct {
-		[range(0,10485760),value(ndr_size_security_descriptor(sd,ndr->flags))] uint32 __ndr_size;
-		[subcontext(4)] security_descriptor *sd;
-	} drsuapi_DsAttributeValueSecurityDescriptor;
-
-	typedef struct {
-		[range(0,10485760)] uint32 num_values;
-		[size_is(num_values)] drsuapi_DsAttributeValueSecurityDescriptor *values;
-	} drsuapi_DsAttributeValueCtrSecurityDescriptor;
-
 	/* NTTIME_1sec values */
 	typedef struct {
 		[range(0,10485760),value(8)] uint32 __ndr_size;
@@ -569,9 +558,6 @@
 		/* SID */
 		[case(DRSUAPI_ATTRIBUTE_objectSid)] drsuapi_DsAttributeValueCtrSID sid;
 
-		/* SecurityDescriptor */
-		[case(DRSUAPI_ATTRIBUTE_ntSecurityDescriptor)] drsuapi_DsAttributeValueCtrSecurityDescriptor security_descriptor;
-
 		/* UnicodeString */
 		[case(DRSUAPI_ATTRIBUTE_description)]		drsuapi_DsAttributeValueCtrUnicodeString unicode_string;
 		[case(DRSUAPI_ATTRIBUTE_adminDisplayName)]	drsuapi_DsAttributeValueCtrUnicodeString unicode_string;