From 6cb5a0d097e52577e956d3527b753c1bcf124d9d Mon Sep 17 00:00:00 2001
From: Volker Lendecke <vl@samba.org>
Date: Thu, 5 Aug 2010 13:06:11 +0200
Subject: s3: Remove some pointless wrapper functions

---
 source3/libsmb/cliconnect.c |  4 ++--
 source3/libsmb/clilist.c    | 20 +++++++++++++-------
 source3/libsmb/cliprint.c   |  3 ++-
 source3/libsmb/clirap.c     | 12 ++++++------
 4 files changed, 23 insertions(+), 16 deletions(-)

(limited to 'source3/libsmb')

diff --git a/source3/libsmb/cliconnect.c b/source3/libsmb/cliconnect.c
index 9d84585398..54e6a5d166 100644
--- a/source3/libsmb/cliconnect.c
+++ b/source3/libsmb/cliconnect.c
@@ -2146,8 +2146,8 @@ static void cli_negprot_done(struct tevent_req *subreq)
 		cli->serverzone = SVALS(vwv + 10, 0);
 		cli->serverzone *= 60;
 		/* this time is converted to GMT by make_unix_date */
-		cli->servertime = cli_make_unix_date(
-			cli, (char *)(vwv + 8));
+		cli->servertime = make_unix_date(
+			(char *)(vwv + 8), cli->serverzone);
 		cli->readbraw_supported = ((SVAL(vwv + 5, 0) & 0x1) != 0);
 		cli->writebraw_supported = ((SVAL(vwv + 5, 0) & 0x2) != 0);
 		cli->secblob = data_blob(bytes, num_bytes);
diff --git a/source3/libsmb/clilist.c b/source3/libsmb/clilist.c
index a8e3bd5017..56de119281 100644
--- a/source3/libsmb/clilist.c
+++ b/source3/libsmb/clilist.c
@@ -70,9 +70,12 @@ static size_t interpret_long_filename(TALLOC_CTX *ctx,
 			if (pdata_end - base < 27) {
 				return pdata_end - base;
 			}
-			finfo->ctime_ts = convert_time_t_to_timespec(cli_make_unix_date2(cli, p+4));
-			finfo->atime_ts = convert_time_t_to_timespec(cli_make_unix_date2(cli, p+8));
-			finfo->mtime_ts = convert_time_t_to_timespec(cli_make_unix_date2(cli, p+12));
+			finfo->ctime_ts = convert_time_t_to_timespec(
+				make_unix_date2(p+4, cli->serverzone));
+			finfo->atime_ts = convert_time_t_to_timespec(
+				make_unix_date2(p+8, cli->serverzone));
+			finfo->mtime_ts = convert_time_t_to_timespec(
+				make_unix_date2(p+12, cli->serverzone));
 			finfo->size = IVAL(p,16);
 			finfo->mode = CVAL(p,24);
 			len = CVAL(p, 26);
@@ -115,9 +118,12 @@ static size_t interpret_long_filename(TALLOC_CTX *ctx,
 			if (pdata_end - base < 31) {
 				return pdata_end - base;
 			}
-			finfo->ctime_ts = convert_time_t_to_timespec(cli_make_unix_date2(cli, p+4));
-			finfo->atime_ts = convert_time_t_to_timespec(cli_make_unix_date2(cli, p+8));
-			finfo->mtime_ts = convert_time_t_to_timespec(cli_make_unix_date2(cli, p+12));
+			finfo->ctime_ts = convert_time_t_to_timespec(
+				make_unix_date2(p+4, cli->serverzone));
+			finfo->atime_ts = convert_time_t_to_timespec(
+				make_unix_date2(p+8, cli->serverzone));
+			finfo->mtime_ts = convert_time_t_to_timespec(
+				make_unix_date2(p+12, cli->serverzone));
 			finfo->size = IVAL(p,16);
 			finfo->mode = CVAL(p,24);
 			len = CVAL(p, 30);
@@ -510,7 +516,7 @@ static bool interpret_short_filename(TALLOC_CTX *ctx,
 	finfo->mode = CVAL(p,21);
 
 	/* this date is converted to GMT by make_unix_date */
-	finfo->ctime_ts.tv_sec = cli_make_unix_date(cli, p+22);
+	finfo->ctime_ts.tv_sec = make_unix_date(p+22, cli->serverzone);
 	finfo->ctime_ts.tv_nsec = 0;
 	finfo->mtime_ts.tv_sec = finfo->atime_ts.tv_sec = finfo->ctime_ts.tv_sec;
 	finfo->mtime_ts.tv_nsec = finfo->atime_ts.tv_nsec = 0;
diff --git a/source3/libsmb/cliprint.c b/source3/libsmb/cliprint.c
index 723ae02623..89e80d2904 100644
--- a/source3/libsmb/cliprint.c
+++ b/source3/libsmb/cliprint.c
@@ -98,7 +98,8 @@ int cli_print_queue(struct cli_state *cli,
 				fstrcpy(job.user,
 					fix_char_ptr(SVAL(p,4), converter,
 						     rdata, rdrcnt));
-				job.t = cli_make_unix_date3(cli, p + 12);
+				job.t = make_unix_date3(
+					p + 12, cli->serverzone);
 				job.size = IVAL(p,16);
 				fstrcpy(job.name,fix_char_ptr(SVAL(p,24),
 							      converter,
diff --git a/source3/libsmb/clirap.c b/source3/libsmb/clirap.c
index 6c4826dd8e..a1b911cfbf 100644
--- a/source3/libsmb/clirap.c
+++ b/source3/libsmb/clirap.c
@@ -590,26 +590,26 @@ NTSTATUS cli_qpathinfo1_recv(struct tevent_req *req,
 		req, struct cli_qpathinfo1_state);
 	NTSTATUS status;
 
-	time_t (*date_fn)(struct cli_state *, const void *);
+	time_t (*date_fn)(const void *buf, int serverzone);
 
 	if (tevent_req_is_nterror(req, &status)) {
 		return status;
 	}
 
 	if (state->cli->win95) {
-		date_fn = cli_make_unix_date;
+		date_fn = make_unix_date;
 	} else {
-		date_fn = cli_make_unix_date2;
+		date_fn = make_unix_date2;
 	}
 
 	if (change_time) {
-		*change_time = date_fn(state->cli, state->data+0);
+		*change_time = date_fn(state->data+0, state->cli->serverzone);
 	}
 	if (access_time) {
-		*access_time = date_fn(state->cli, state->data+4);
+		*access_time = date_fn(state->data+4, state->cli->serverzone);
 	}
 	if (write_time) {
-		*write_time = date_fn(state->cli, state->data+8);
+		*write_time = date_fn(state->data+8, state->cli->serverzone);
 	}
 	if (size) {
 		*size = IVAL(state->data, 12);
-- 
cgit