svn commit: samba r9715 - in branches/tmp/RPCREWRITE/source/rpc_client: .

jra at samba.org jra at samba.org
Sun Aug 28 21:46:46 GMT 2005


Author: jra
Date: 2005-08-28 21:46:46 +0000 (Sun, 28 Aug 2005)
New Revision: 9715

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

Log:
Fix old bug in cli_spoolss code. Convert shutdown to new style.
Jeremy.

Modified:
   branches/tmp/RPCREWRITE/source/rpc_client/cli_shutdown.c
   branches/tmp/RPCREWRITE/source/rpc_client/cli_spoolss.c


Changeset:
Modified: branches/tmp/RPCREWRITE/source/rpc_client/cli_shutdown.c
===================================================================
--- branches/tmp/RPCREWRITE/source/rpc_client/cli_shutdown.c	2005-08-28 21:24:54 UTC (rev 9714)
+++ branches/tmp/RPCREWRITE/source/rpc_client/cli_shutdown.c	2005-08-28 21:46:46 UTC (rev 9715)
@@ -3,10 +3,7 @@
    RPC Pipe client
  
    Copyright (C) Andrew Tridgell              1992-1998,
-   Copyright (C) Luke Kenneth Casson Leighton 1996-1998,
-   Copyright (C) Paul Ashton                  1997-1998.
-   Copyright (C) Jeremy Allison                    1999,
-   Copyright (C) Simo Sorce                        2001,
+   Largely rewritten by Jeremy Allison (C)	   2005.
    Copyright (C) Jim McDonough (jmcd at us.ibm.com)   2003.
    
    This program is free software; you can redistribute it and/or modify
@@ -28,119 +25,95 @@
 
 /* Shutdown a server */
 
-NTSTATUS cli_shutdown_init(struct cli_state * cli, TALLOC_CTX *mem_ctx,
+NTSTATUS rpccli_shutdown_init(struct rpc_pipe_client *cli, TALLOC_CTX *mem_ctx,
 			   const char *msg, uint32 timeout, BOOL do_reboot,
 			   BOOL force)
 {
 	prs_struct qbuf;
 	prs_struct rbuf; 
-	SHUTDOWN_Q_INIT q_s;
-	SHUTDOWN_R_INIT r_s;
+	SHUTDOWN_Q_INIT q;
+	SHUTDOWN_R_INIT r;
 	WERROR result = WERR_GENERAL_FAILURE;
 
 	if (msg == NULL) 
 		return NT_STATUS_INVALID_PARAMETER;
 
-	ZERO_STRUCT (q_s);
-	ZERO_STRUCT (r_s);
+	ZERO_STRUCT (q);
+	ZERO_STRUCT (r);
 
-	prs_init(&qbuf , RPC_MAX_PDU_FRAG_LEN, mem_ctx, MARSHALL);
-	prs_init(&rbuf, 0, mem_ctx, UNMARSHALL);
-
 	/* Marshall data and send request */
 
-	init_shutdown_q_init(&q_s, msg, timeout, do_reboot, force);
+	init_shutdown_q_init(&q, msg, timeout, do_reboot, force);
 
-	if (!shutdown_io_q_init("", &q_s, &qbuf, 0) ||
-	    !rpc_api_pipe_req(cli, PI_SHUTDOWN, SHUTDOWN_INIT, &qbuf, &rbuf))
-		goto done;
-	
-	/* Unmarshall response */
-	
-	if(shutdown_io_r_init("", &r_s, &rbuf, 0))
-		result = r_s.status;
+	CLI_DO_RPC(cli, mem_ctx, PI_SHUTDOWN, SHUTDOWN_INIT,
+		q, r,
+		qbuf, rbuf,
+		shutdown_io_q_init,
+		shutdown_io_r_init,
+		NT_STATUS_UNSUCCESSFUL);
 
-done:
-	prs_mem_free(&rbuf);
-	prs_mem_free(&qbuf);
-
+	result = r.status;
 	return werror_to_ntstatus(result);
 }
 
 /* Shutdown a server */
 
-NTSTATUS cli_shutdown_init_ex(struct cli_state * cli, TALLOC_CTX *mem_ctx,
+NTSTATUS rpccli_shutdown_init_ex(struct rpc_pipe_client *cli, TALLOC_CTX *mem_ctx,
 			   const char *msg, uint32 timeout, BOOL do_reboot,
 			   BOOL force, uint32 reason)
 {
 	prs_struct qbuf;
 	prs_struct rbuf; 
-	SHUTDOWN_Q_INIT_EX q_s;
-	SHUTDOWN_R_INIT_EX r_s;
+	SHUTDOWN_Q_INIT_EX q;
+	SHUTDOWN_R_INIT_EX r;
 	WERROR result = WERR_GENERAL_FAILURE;
 
 	if (msg == NULL) 
 		return NT_STATUS_INVALID_PARAMETER;
 
-	ZERO_STRUCT (q_s);
-	ZERO_STRUCT (r_s);
+	ZERO_STRUCT (q);
+	ZERO_STRUCT (r);
 
-	prs_init(&qbuf , RPC_MAX_PDU_FRAG_LEN, mem_ctx, MARSHALL);
-	prs_init(&rbuf, 0, mem_ctx, UNMARSHALL);
-
 	/* Marshall data and send request */
 
-	init_shutdown_q_init_ex(&q_s, msg, timeout, do_reboot, force, reason);
+	init_shutdown_q_init_ex(&q, msg, timeout, do_reboot, force, reason);
 
-	if (!shutdown_io_q_init_ex("", &q_s, &qbuf, 0) ||
-	    !rpc_api_pipe_req(cli, PI_SHUTDOWN, SHUTDOWN_INIT_EX, &qbuf, &rbuf))
-		goto done;
-	
-	/* Unmarshall response */
-	
-	if(shutdown_io_r_init_ex("", &r_s, &rbuf, 0))
-		result = r_s.status;
+	CLI_DO_RPC(cli, mem_ctx, PI_SHUTDOWN, SHUTDOWN_INIT_EX,
+		q, r,
+		qbuf, rbuf,
+		shutdown_io_q_init_ex,
+		shutdown_io_r_init_ex,
+		NT_STATUS_UNSUCCESSFUL);
 
-done:
-	prs_mem_free(&rbuf);
-	prs_mem_free(&qbuf);
-
+	result = r.status;
 	return werror_to_ntstatus(result);
 }
 
 
 /* Abort a server shutdown */
 
-NTSTATUS cli_shutdown_abort(struct cli_state * cli, TALLOC_CTX *mem_ctx)
+NTSTATUS rpccli_shutdown_abort(struct rpc_pipe_client *cli, TALLOC_CTX *mem_ctx)
 {
 	prs_struct rbuf;
 	prs_struct qbuf; 
-	SHUTDOWN_Q_ABORT q_s;
-	SHUTDOWN_R_ABORT r_s;
+	SHUTDOWN_Q_ABORT q;
+	SHUTDOWN_R_ABORT r;
 	WERROR result = WERR_GENERAL_FAILURE;
 
-	ZERO_STRUCT (q_s);
-	ZERO_STRUCT (r_s);
+	ZERO_STRUCT (q);
+	ZERO_STRUCT (r);
 
-	prs_init(&qbuf , RPC_MAX_PDU_FRAG_LEN, mem_ctx, MARSHALL);
-	prs_init(&rbuf, 0, mem_ctx, UNMARSHALL);
-	
 	/* Marshall data and send request */
 
-	init_shutdown_q_abort(&q_s);
+	init_shutdown_q_abort(&q);
 
-	if (!shutdown_io_q_abort("", &q_s, &qbuf, 0) ||
-	    !rpc_api_pipe_req(cli, PI_SHUTDOWN, SHUTDOWN_ABORT, &qbuf, &rbuf))
-	    	goto done;
-	
-		/* Unmarshall response */
-	
-	if (shutdown_io_r_abort("", &r_s, &rbuf, 0))
-		result = r_s.status;
+	CLI_DO_RPC(cli, mem_ctx, PI_SHUTDOWN, SHUTDOWN_ABORT,
+		q, r,
+		qbuf, rbuf,
+		shutdown_io_q_abort,
+		shutdown_io_r_abort,
+		NT_STATUS_UNSUCCESSFUL);
 
-done:
-	prs_mem_free(&rbuf);
-	prs_mem_free(&qbuf );
-
+	result = r.status;
 	return werror_to_ntstatus(result);
 }

Modified: branches/tmp/RPCREWRITE/source/rpc_client/cli_spoolss.c
===================================================================
--- branches/tmp/RPCREWRITE/source/rpc_client/cli_spoolss.c	2005-08-28 21:24:54 UTC (rev 9714)
+++ branches/tmp/RPCREWRITE/source/rpc_client/cli_spoolss.c	2005-08-28 21:46:46 UTC (rev 9715)
@@ -827,8 +827,7 @@
 	ZERO_STRUCT(in);
 	ZERO_STRUCT(out);
 	
-	/* This looks wrong to me - shouldn't client be *our* name ? */
-        slprintf(client, sizeof(fstring)-1, "\\\\%s", cli->cli->desthost);
+        slprintf(client, sizeof(fstring)-1, "\\\\%s", global_myname());
         slprintf(server, sizeof(fstring)-1, "\\\\%s", cli->cli->desthost);
 	
         strupper_m(client);



More information about the samba-cvs mailing list