From: Kamen Mazdrashki <kamenim@samba.org>
Date: Fri, 23 Apr 2010 13:35:21 +0000 (+0300)
Subject: s4/dsdb: schemaInfo revision may be 0
X-Git-Url: http://git.samba.org/?a=commitdiff_plain;h=3a8a767174d708c23e2b04d04f5e85a5220ce918;p=kamenim%2Fsamba.git

s4/dsdb: schemaInfo revision may be 0

In case schemaInfo value is still not set, WinAD supplies
schemaInfo blob with revision = 0 and GUID_ZERO
---

diff --git a/source4/dsdb/schema/schema_info_attr.c b/source4/dsdb/schema/schema_info_attr.c
index ac5bdc2fa9..30f19fe383 100644
--- a/source4/dsdb/schema/schema_info_attr.c
+++ b/source4/dsdb/schema/schema_info_attr.c
@@ -135,10 +135,6 @@ WERROR dsdb_blob_from_schema_info(const struct dsdb_schema_info *schema_info,
 	enum ndr_err_code ndr_err;
 	struct schemaInfoBlob schema_info_blob;
 
-	if (schema_info->revision < 1) {
-		return WERR_INVALID_PARAMETER;
-	}
-
 	schema_info_blob.marker		= 0xFF;
 	schema_info_blob.revision	= schema_info->revision;
 	schema_info_blob.invocation_id  = schema_info->invocation_id;
diff --git a/source4/torture/drs/unit/schemainfo_tests.c b/source4/torture/drs/unit/schemainfo_tests.c
index 55d782f86c..7015c56ad4 100644
--- a/source4/torture/drs/unit/schemainfo_tests.c
+++ b/source4/torture/drs/unit/schemainfo_tests.c
@@ -36,7 +36,7 @@
  *   Rev:  01
  *   GUID: 071c82fd-45c7-4351-a3db-51f75a630a7f
  */
-#define SCHEMA_INFO_INIT_STR		"FF0000000100000000000000000000000000000000"
+#define SCHEMA_INFO_INIT_STR		"FF0000000000000000000000000000000000000000"
 
 /**
  * Default schema_info string to be used for testing
@@ -65,6 +65,13 @@ static const struct {
 	WERROR		werr_expected;
 	bool 		test_both_ways;
 } _schemainfo_test_data[] = {
+	{
+		.schema_info_str = "FF0000000000000000000000000000000000000000",
+		.revision = 0,
+		.guid_str = "00000000-0000-0000-0000-000000000000",
+		.werr_expected = WERR_OK,
+		.test_both_ways = true
+	},
 	{
 		.schema_info_str = "FF00000001FD821C07C7455143A3DB51F75A630A7F",
 		.revision = 1,
@@ -79,18 +86,6 @@ static const struct {
 		.werr_expected = WERR_OK,
 		.test_both_ways = true
 	},
-#if 0
-	/* removed until kamen can take a look - revision 0 is sent by
-	 * w2k8r2, and we need to accept it, possibly only when the
-	 * other fields are zero */
-	{ /* revision > 0 */
-		.schema_info_str = "FF00000000FD821C07C7455143A3DB51F75A630A7F",
-		.revision = 0,
-		.guid_str = "071c82fd-45c7-4351-a3db-51f75a630a7f",
-		.werr_expected = WERR_INVALID_PARAMETER,
-		.test_both_ways = true
-	},
-#endif
 	{ /* len == 21 */
 		.schema_info_str = "FF00000001FD821C07C7455143A3DB51F75A630A7F00",
 		.revision = 1,
@@ -333,10 +328,6 @@ static bool test_dsdb_module_schema_info_update(struct torture_context *tctx,
 	werr = dsdb_schema_info_from_blob(&blob, priv, &schema_info);
 	torture_assert_werr_ok(tctx, werr, "dsdb_schema_info_from_blob() failed");
 
-	/* decrement revision to be able to compare
-	 * against default schemaInfo later */
-	schema_info->revision--;
-
 	/* check against default schema_info */
 	torture_assert_schema_info_equal(tctx, schema_info, priv->schema_info,
 					  "schemaInfo attribute no updated correctly");