svn commit: samba r17954 - in branches/SAMBA_4_0/source: libnet torture/rpc

[abartlet at samba.org]

Author: abartlet
Date: 2006-08-31 08:15:23 +0000 (Thu, 31 Aug 2006)
New Revision: 17954

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

Log:
Avoid including \n in error strings (left over from DEBUG() conversion).

Make it easier to debug CrackNames failures.

Andrew Bartlett

Modified:
   branches/SAMBA_4_0/source/libnet/libnet_join.c
   branches/SAMBA_4_0/source/torture/rpc/testjoin.c


Changeset:
Modified: branches/SAMBA_4_0/source/libnet/libnet_join.c
===================================================================
--- branches/SAMBA_4_0/source/libnet/libnet_join.c	2006-08-31 07:43:57 UTC (rev 17953)
+++ branches/SAMBA_4_0/source/libnet/libnet_join.c	2006-08-31 08:15:23 UTC (rev 17954)
@@ -135,14 +135,14 @@
 		if (NT_STATUS_EQUAL(status, NT_STATUS_NET_WRITE_FAULT)) {
 			r->out.error_string
 				= talloc_asprintf(r,
-						  "dcerpc_drsuapi_DsBind failed - %s\n", 
+						  "dcerpc_drsuapi_DsBind failed - %s", 
 						  dcerpc_errstr(tmp_ctx, drsuapi_pipe->last_fault_code));
 			talloc_free(tmp_ctx);
 			return status;
 		} else {
 			r->out.error_string
 				= talloc_asprintf(r,
-						  "dcerpc_drsuapi_DsBind failed - %s\n", 
+						  "dcerpc_drsuapi_DsBind failed - %s", 
 						  nt_errstr(status));
 			talloc_free(tmp_ctx);
 			return status;
@@ -150,7 +150,7 @@
 	} else if (!W_ERROR_IS_OK(r_drsuapi_bind.out.result)) {
 		r->out.error_string
 				= talloc_asprintf(r,
-						  "DsBind failed - %s\n", 
+						  "DsBind failed - %s", 
 						  win_errstr(r_drsuapi_bind.out.result));
 			talloc_free(tmp_ctx);
 		return NT_STATUS_UNSUCCESSFUL;
@@ -179,7 +179,7 @@
 		if (NT_STATUS_EQUAL(status, NT_STATUS_NET_WRITE_FAULT)) {
 			r->out.error_string
 				= talloc_asprintf(r,
-						  "dcerpc_drsuapi_DsCrackNames for [%s] failed - %s\n", 
+						  "dcerpc_drsuapi_DsCrackNames for [%s] failed - %s", 
 						  names[0].str,
 						  dcerpc_errstr(tmp_ctx, drsuapi_pipe->last_fault_code));
 			talloc_free(tmp_ctx);
@@ -187,7 +187,7 @@
 		} else {
 			r->out.error_string
 				= talloc_asprintf(r,
-						  "dcerpc_drsuapi_DsCrackNames for [%s] failed - %s\n", 
+						  "dcerpc_drsuapi_DsCrackNames for [%s] failed - %s", 
 						  names[0].str,
 						  nt_errstr(status));
 			talloc_free(tmp_ctx);
@@ -196,17 +196,23 @@
 	} else if (!W_ERROR_IS_OK(r_crack_names.out.result)) {
 		r->out.error_string
 				= talloc_asprintf(r,
-						  "DsCrackNames failed - %s\n", win_errstr(r_crack_names.out.result));
+						  "DsCrackNames failed - %s", win_errstr(r_crack_names.out.result));
 		talloc_free(tmp_ctx);
 		return NT_STATUS_UNSUCCESSFUL;
 	} else if (r_crack_names.out.level != 1 
 		   || !r_crack_names.out.ctr.ctr1 
-		   || r_crack_names.out.ctr.ctr1->count != 1 
-		   || !r_crack_names.out.ctr.ctr1->array[0].result_name
-		   || r_crack_names.out.ctr.ctr1->array[0].status != DRSUAPI_DS_NAME_STATUS_OK) {
-		r->out.error_string = talloc_asprintf(r, "DsCrackNames failed\n");
+		   || r_crack_names.out.ctr.ctr1->count != 1) {
+		r->out.error_string = talloc_asprintf(r, "DsCrackNames failed");
 		talloc_free(tmp_ctx);
+		return NT_STATUS_INVALID_PARAMETER;
+	} else if (r_crack_names.out.ctr.ctr1->array[0].status != DRSUAPI_DS_NAME_STATUS_OK) {
+		r->out.error_string = talloc_asprintf(r, "DsCrackNames failed: %d", r_crack_names.out.ctr.ctr1->array[0].status);
+		talloc_free(tmp_ctx);
 		return NT_STATUS_UNSUCCESSFUL;
+	} else if (r_crack_names.out.ctr.ctr1->array[0].result_name == NULL) {
+		r->out.error_string = talloc_asprintf(r, "DsCrackNames failed: no result name");
+		talloc_free(tmp_ctx);
+		return NT_STATUS_INVALID_PARAMETER;
 	}
 
 	/* Store the DN of our machine account. */
@@ -346,7 +352,7 @@
 	} else if (!W_ERROR_IS_OK(r_crack_names.out.result)) {
 		r->out.error_string
 			= talloc_asprintf(r,
-					  "DsCrackNames failed - %s\n", win_errstr(r_crack_names.out.result));
+					  "DsCrackNames failed - %s", win_errstr(r_crack_names.out.result));
 		talloc_free(tmp_ctx);
 		return NT_STATUS_UNSUCCESSFUL;
 	} else if (r_crack_names.out.level != 1 
@@ -354,7 +360,7 @@
 		   || r_crack_names.out.ctr.ctr1->count != 1
 		   || !r_crack_names.out.ctr.ctr1->array[0].result_name		  
 		   || r_crack_names.out.ctr.ctr1->array[0].status != DRSUAPI_DS_NAME_STATUS_OK) {
-		r->out.error_string = talloc_asprintf(r, "DsCrackNames failed\n");
+		r->out.error_string = talloc_asprintf(r, "DsCrackNames failed");
 		talloc_free(tmp_ctx);
 		return NT_STATUS_UNSUCCESSFUL;
 	}

Modified: branches/SAMBA_4_0/source/torture/rpc/testjoin.c
===================================================================
--- branches/SAMBA_4_0/source/torture/rpc/testjoin.c	2006-08-31 07:43:57 UTC (rev 17953)
+++ branches/SAMBA_4_0/source/torture/rpc/testjoin.c	2006-08-31 08:15:23 UTC (rev 17954)
@@ -327,9 +327,9 @@
 	status = libnet_JoinDomain(libnet_ctx, libnet_r, libnet_r);
 	if (!NT_STATUS_IS_OK(status)) {
 		if (libnet_r->out.error_string) {
-			DEBUG(0, ("Domain join failed - %s.\n", libnet_r->out.error_string));
+			DEBUG(0, ("Domain join failed - %s\n", libnet_r->out.error_string));
 		} else {
-			DEBUG(0, ("Domain join failed - %s.\n", nt_errstr(status)));
+			DEBUG(0, ("Domain join failed - %s\n", nt_errstr(status)));
 		}
 		talloc_free(tj);
                 return NULL;