[SCM] Samba Shared Repository - branch master updated - release-4-0-0alpha7-1520-g5c76472

[Stefan Metzmacher]

The branch, master has been updated
       via  5c76472491b813350d01a6398687af0e52fda819 (commit)
      from  6c935f95b3d4aaf0922c5a7baf59c1e8224c019a (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 5c76472491b813350d01a6398687af0e52fda819
Author: Stefan Metzmacher <metze at samba.org>
Date:   Fri May 8 18:40:35 2009 +0200

    s3:libsmb: fix layering of cli_ntrename_internal and its callers
    
    It's easier to have cli_ntrename_internal as a semetric async
    tevent_req function. cli_ntrename() and cli_nt_hardlink() should
    be callers on top of cli_ntrename_internal().
    
    metze

-----------------------------------------------------------------------

Summary of changes:
 source3/libsmb/clifile.c |   64 +++++++++++++++++++++++++---------------------
 1 files changed, 35 insertions(+), 29 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/libsmb/clifile.c b/source3/libsmb/clifile.c
index cd9f486..fdfa257 100644
--- a/source3/libsmb/clifile.c
+++ b/source3/libsmb/clifile.c
@@ -552,13 +552,13 @@ NTSTATUS cli_rename(struct cli_state *cli, const char *fname_src, const char *fn
  NT Rename a file.
 ****************************************************************************/
 
-static void cli_ntrename_done(struct tevent_req *subreq);
+static void cli_ntrename_internal_done(struct tevent_req *subreq);
 
-struct cli_ntrename_state {
+struct cli_ntrename_internal_state {
 	uint16_t vwv[4];
 };
 
-static struct tevent_req *cli_ntrename_send_internal(TALLOC_CTX *mem_ctx,
+static struct tevent_req *cli_ntrename_internal_send(TALLOC_CTX *mem_ctx,
 				struct event_context *ev,
 				struct cli_state *cli,
 				const char *fname_src,
@@ -566,11 +566,12 @@ static struct tevent_req *cli_ntrename_send_internal(TALLOC_CTX *mem_ctx,
 				uint16_t rename_flag)
 {
 	struct tevent_req *req = NULL, *subreq = NULL;
-	struct cli_ntrename_state *state = NULL;
+	struct cli_ntrename_internal_state *state = NULL;
 	uint8_t additional_flags = 0;
 	uint8_t *bytes = NULL;
 
-	req = tevent_req_create(mem_ctx, &state, struct cli_ntrename_state);
+	req = tevent_req_create(mem_ctx, &state,
+				struct cli_ntrename_internal_state);
 	if (req == NULL) {
 		return NULL;
 	}
@@ -607,25 +608,11 @@ static struct tevent_req *cli_ntrename_send_internal(TALLOC_CTX *mem_ctx,
 	if (tevent_req_nomem(subreq, req)) {
 		return tevent_req_post(req, ev);
 	}
-	tevent_req_set_callback(subreq, cli_ntrename_done, req);
+	tevent_req_set_callback(subreq, cli_ntrename_internal_done, req);
 	return req;
 }
 
-struct tevent_req *cli_ntrename_send(TALLOC_CTX *mem_ctx,
-				struct event_context *ev,
-				struct cli_state *cli,
-				const char *fname_src,
-				const char *fname_dst)
-{
-	return cli_ntrename_send_internal(mem_ctx,
-					ev,
-					cli,
-					fname_src,
-					fname_dst,
-					RENAME_FLAG_RENAME);
-}
-
-static void cli_ntrename_done(struct tevent_req *subreq)
+static void cli_ntrename_internal_done(struct tevent_req *subreq)
 {
 	struct tevent_req *req = tevent_req_callback_data(
 				subreq, struct tevent_req);
@@ -640,11 +627,30 @@ static void cli_ntrename_done(struct tevent_req *subreq)
 	tevent_req_done(req);
 }
 
-NTSTATUS cli_ntrename_recv(struct tevent_req *req)
+static NTSTATUS cli_ntrename_internal_recv(struct tevent_req *req)
 {
 	return tevent_req_simple_recv_ntstatus(req);
 }
 
+struct tevent_req *cli_ntrename_send(TALLOC_CTX *mem_ctx,
+				struct event_context *ev,
+				struct cli_state *cli,
+				const char *fname_src,
+				const char *fname_dst)
+{
+	return cli_ntrename_internal_send(mem_ctx,
+					  ev,
+					  cli,
+					  fname_src,
+					  fname_dst,
+					  RENAME_FLAG_RENAME);
+}
+
+NTSTATUS cli_ntrename_recv(struct tevent_req *req)
+{
+	return cli_ntrename_internal_recv(req);
+}
+
 NTSTATUS cli_ntrename(struct cli_state *cli, const char *fname_src, const char *fname_dst)
 {
 	TALLOC_CTX *frame = talloc_stackframe();
@@ -697,17 +703,17 @@ struct tevent_req *cli_nt_hardlink_send(TALLOC_CTX *mem_ctx,
 				const char *fname_src,
 				const char *fname_dst)
 {
-	return cli_ntrename_send_internal(mem_ctx,
-					ev,
-					cli,
-					fname_src,
-					fname_dst,
-					RENAME_FLAG_HARD_LINK);
+	return cli_ntrename_internal_send(mem_ctx,
+					  ev,
+					  cli,
+					  fname_src,
+					  fname_dst,
+					  RENAME_FLAG_HARD_LINK);
 }
 
 NTSTATUS cli_nt_hardlink_recv(struct tevent_req *req)
 {
-	return tevent_req_simple_recv_ntstatus(req);
+	return cli_ntrename_internal_recv(req);
 }
 
 NTSTATUS cli_nt_hardlink(struct cli_state *cli, const char *fname_src, const char *fname_dst)


-- 
Samba Shared Repository