The branch, master has been updated
       via  d3abc90 s3: Avoid a few calls to cli_errstr
       via  c7a5933 s3: Remove some unused code
       via  6f3e110 s3: Convert cli_lock64 to cli_smb
       via  dd2f234 s3: Convert cli_lock to use cli_locktype
       via  0e2854f s3: Convert cli_locktype to cli_smb
       via  7b054ec s3: Make passing up "result_parent" from cli_smb optional
       via  b33f114 s3: Remove unused cli_send_keepalive
       via  6e684c8 s3: Remove unused cli_send/receive_trans
       via  cba5a26 s3: Convert try_trans2 to cli_trans
       via  8ac68fc s3: Convert cli_api to cli_trans
       via  cb412d2 s3: Make cli_setpathinfo_basic use cli_setpathinfo
       via  21d5485 s3: Add cli_setpathinfo
       via  2068b96 s3: Rename cli_setpathinfo->cli_setpathinfo_basic
       via  2d44a05 s3: Remove unused cli_send/receive_nt_trans
       via  d5decfb s3: Convert try_nttrans to cli_trans
       via  5dee39b s3: Move dump_ntquota* to smbcquotas.c
       via  a1f5417 s3: Convert cli_set_fs_quota_info to cli_trans
       via  797c027 s3: Convert cli_get_fs_quota_info to cli_trans
       via  483e044 s3: Convert cli_list_user_quota to cli_trans
       via  eb7db50 s3: Convert cli_set_user_quota to cli_trans
       via  e2a9671 s3: Convert cli_get_user_quota to cli_trans
       via  bc5554b s3: Fix some nonempty blank lines
       via  d920462 s3: Fix a C++ warning
       via  674dcd6 s3: Remove cli_send_trans from cli_qpathinfo_alt_name
       via  fc8bfee s3: Remove cli_send_trans from cli_oem_change_password
      from  7320f19 ldb: new ABI sigs file

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


- Log -----------------------------------------------------------------
commit d3abc90401c65e66de26cdfb2432ef023732dce7
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 21:13:29 2011 +0100

    s3: Avoid a few calls to cli_errstr
    
    Autobuild-User: Volker Lendecke <vlen...@samba.org>
    Autobuild-Date: Mon Jan 17 08:47:25 CET 2011 on sn-devel-104

commit c7a5933e4b907d0dc50c01b299a5822392c86049
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 20:16:25 2011 +0100

    s3: Remove some unused code

commit 6f3e11088c2783ccc5c7e785d3a67afb82e20db0
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 20:09:17 2011 +0100

    s3: Convert cli_lock64 to cli_smb

commit dd2f234412a004ca8c259b60302f1aac42c3a882
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 20:04:17 2011 +0100

    s3: Convert cli_lock to use cli_locktype

commit 0e2854f7433680b9b76002b0ff3d3b039661bd43
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 19:50:46 2011 +0100

    s3: Convert cli_locktype to cli_smb

commit 7b054ecaed589ed4228fe5cd5680540f8fef201d
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 19:20:43 2011 +0100

    s3: Make passing up "result_parent" from cli_smb optional

commit b33f1147a6074b547a908b6c07fe2d4679c36383
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 19:11:45 2011 +0100

    s3: Remove unused cli_send_keepalive

commit 6e684c8460ed5be9347aec36bb51bd8c823a0565
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 13:13:20 2011 +0100

    s3: Remove unused cli_send/receive_trans

commit cba5a26734686deeed4785eabd87d7608b3d90f4
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 13:12:16 2011 +0100

    s3: Convert try_trans2 to cli_trans

commit 8ac68fc3cbff6acb9b780f01a61c2457555fd579
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 13:07:35 2011 +0100

    s3: Convert cli_api to cli_trans

commit cb412d22eba8cdb81878628aa4bce3ca4591770f
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 12:56:09 2011 +0100

    s3: Make cli_setpathinfo_basic use cli_setpathinfo

commit 21d5485e062e37f112ce82ab3ae4e73c53ce081d
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 12:49:59 2011 +0100

    s3: Add cli_setpathinfo

commit 2068b96dce77735a9b93bee49b5750c574e79652
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 12:45:22 2011 +0100

    s3: Rename cli_setpathinfo->cli_setpathinfo_basic

commit 2d44a0583b6e32849741d97ccac8bd0468d0e7e8
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 12:34:43 2011 +0100

    s3: Remove unused cli_send/receive_nt_trans

commit d5decfbb260a6770e5023abada56dbe5d9e02e7b
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 12:33:24 2011 +0100

    s3: Convert try_nttrans to cli_trans

commit 5dee39b5bf65faaf8695b72dfc6247ef4789ef49
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 12:26:45 2011 +0100

    s3: Move dump_ntquota* to smbcquotas.c

commit a1f541723d40f95e7fff13e8c06a037306deccce
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 12:14:26 2011 +0100

    s3: Convert cli_set_fs_quota_info to cli_trans

commit 797c027cc898a88ec7873a3d34908b09eb2e68ce
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 12:03:07 2011 +0100

    s3: Convert cli_get_fs_quota_info to cli_trans

commit 483e0447f5afaf6ef6d636447f7759117039ac06
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 11:53:03 2011 +0100

    s3: Convert cli_list_user_quota to cli_trans

commit eb7db50f60b96433c62ac7755b8ff771d4aa48ce
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 11:33:35 2011 +0100

    s3: Convert cli_set_user_quota to cli_trans

commit e2a96711d708dd6df32bc68883a5bb0ef908a639
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 16 08:34:58 2011 +0100

    s3: Convert cli_get_user_quota to cli_trans

commit bc5554bdcc9ece0e1f443653d87a541f74210fa0
Author: Volker Lendecke <v...@samba.org>
Date:   Sat Jan 15 16:07:31 2011 +0100

    s3: Fix some nonempty blank lines

commit d9204622e371f1f88be0f0d20c7365d440c616bc
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 9 19:54:33 2011 +0100

    s3: Fix a C++ warning

commit 674dcd68a635c183a7cafb9547d2b2f1b0a24081
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 9 19:44:13 2011 +0100

    s3: Remove cli_send_trans from cli_qpathinfo_alt_name

commit fc8bfeea31528b2842fdde22203d576fab8adb92
Author: Volker Lendecke <v...@samba.org>
Date:   Sun Jan 9 19:13:15 2011 +0100

    s3: Remove cli_send_trans from cli_oem_change_password

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

Summary of changes:
 source3/auth/auth_server.c   |    6 +-
 source3/client/smbspool.c    |   16 +-
 source3/include/proto.h      |   55 ++---
 source3/libsmb/clientgen.c   |   21 +--
 source3/libsmb/clifile.c     |  209 ++++++--------
 source3/libsmb/cliprint.c    |  105 -------
 source3/libsmb/cliquota.c    |  448 ++++++++++------------------
 source3/libsmb/clirap.c      |  228 ++++++---------
 source3/libsmb/clitrans.c    |  672 ------------------------------------------
 source3/libsmb/libsmb_file.c |   12 +-
 source3/smbd/smb2_server.c   |    2 +-
 source3/torture/scanner.c    |  113 ++++----
 source3/utils/smbcacls.c     |    2 +-
 source3/utils/smbcquotas.c   |  181 ++++++++++--
 14 files changed, 590 insertions(+), 1480 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/auth/auth_server.c b/source3/auth/auth_server.c
index ac757d5..fc46ce0 100644
--- a/source3/auth/auth_server.c
+++ b/source3/auth/auth_server.c
@@ -135,11 +135,11 @@ static struct cli_state *server_cryptkey(TALLOC_CTX 
*mem_ctx)
           this one...
        */
 
-       if (!NT_STATUS_IS_OK(cli_session_setup(cli, "", "", 0, "", 0,
-                                              ""))) {
+       status = cli_session_setup(cli, "", "", 0, "", 0, "");
+       if (!NT_STATUS_IS_OK(status)) {
                TALLOC_FREE(mutex);
                DEBUG(0,("%s rejected the initial session setup (%s)\n",
-                        desthost, cli_errstr(cli)));
+                        desthost, nt_errstr(status)));
                cli_shutdown(cli);
                return NULL;
        }
diff --git a/source3/client/smbspool.c b/source3/client/smbspool.c
index 0804eb5..c80ad42 100644
--- a/source3/client/smbspool.c
+++ b/source3/client/smbspool.c
@@ -558,6 +558,7 @@ smb_print(struct cli_state * cli,   /* I - SMB connection */
                        tbytes; /* Total bytes read */
        char            buffer[8192],   /* Buffer for copy */
                       *ptr;    /* Pointer into title */
+       NTSTATUS nt_status;
 
 
        /*
@@ -574,10 +575,12 @@ smb_print(struct cli_state * cli, /* I - SMB connection */
          * Open the printer device...
          */
 
-       if (!NT_STATUS_IS_OK(cli_open(cli, title, O_RDWR | O_CREAT | O_TRUNC, 
DENY_NONE, &fnum))) {
+       nt_status = cli_open(cli, title, O_RDWR | O_CREAT | O_TRUNC, DENY_NONE,
+                         &fnum);
+       if (!NT_STATUS_IS_OK(nt_status)) {
                fprintf(stderr, "ERROR: %s opening remote spool %s\n",
-                       cli_errstr(cli), title);
-               return (get_exit_code(cli, cli_nt_error(cli)));
+                       nt_errstr(nt_status), title);
+               return get_exit_code(cli, nt_status);
        }
 
        /*
@@ -602,10 +605,11 @@ smb_print(struct cli_state * cli, /* I - SMB connection */
                tbytes += nbytes;
        }
 
-       if (!NT_STATUS_IS_OK(cli_close(cli, fnum))) {
+       nt_status = cli_close(cli, fnum);
+       if (!NT_STATUS_IS_OK(nt_status)) {
                fprintf(stderr, "ERROR: %s closing remote spool %s\n",
-                       cli_errstr(cli), title);
-               return (get_exit_code(cli, cli_nt_error(cli)));
+                       nt_errstr(nt_status), title);
+               return get_exit_code(cli, nt_status);
        } else {
                return (0);
        }
diff --git a/source3/include/proto.h b/source3/include/proto.h
index 0e66a52..f3ec4d1 100644
--- a/source3/include/proto.h
+++ b/source3/include/proto.h
@@ -1746,7 +1746,6 @@ void cli_shutdown(struct cli_state *cli);
 void cli_sockopt(struct cli_state *cli, const char *options);
 uint16 cli_setpid(struct cli_state *cli, uint16 pid);
 bool cli_set_case_sensitive(struct cli_state *cli, bool case_sensitive);
-bool cli_send_keepalive(struct cli_state *cli);
 struct tevent_req *cli_echo_send(TALLOC_CTX *mem_ctx, struct event_context *ev,
                                 struct cli_state *cli, uint16_t num_echos,
                                 DATA_BLOB data);
@@ -1786,6 +1785,11 @@ struct tevent_req *cli_setpathinfo_send(TALLOC_CTX 
*mem_ctx,
                                        uint8_t *data,
                                        size_t data_len);
 NTSTATUS cli_setpathinfo_recv(struct tevent_req *req);
+NTSTATUS cli_setpathinfo(struct cli_state *cli,
+                        uint16_t level,
+                        const char *path,
+                        uint8_t *data,
+                        size_t data_len);
 
 struct tevent_req *cli_posix_symlink_send(TALLOC_CTX *mem_ctx,
                                        struct event_context *ev,
@@ -2247,20 +2251,21 @@ void cli_oplock_handler(struct cli_state *cli,
 int cli_print_queue(struct cli_state *cli,
                    void (*fn)(struct print_job_info *));
 int cli_printjob_del(struct cli_state *cli, int job);
-int cli_spl_open(struct cli_state *cli, const char *fname, int flags, int 
share_mode);
-bool cli_spl_close(struct cli_state *cli, uint16_t fnum);
 
 /* The following definitions come from libsmb/cliquota.c  */
 
 NTSTATUS cli_get_quota_handle(struct cli_state *cli, uint16_t *quota_fnum);
 void free_ntquota_list(SMB_NTQUOTA_LIST **qt_list);
-bool cli_get_user_quota(struct cli_state *cli, int quota_fnum, 
SMB_NTQUOTA_STRUCT *pqt);
-bool cli_set_user_quota(struct cli_state *cli, int quota_fnum, 
SMB_NTQUOTA_STRUCT *pqt);
-bool cli_list_user_quota(struct cli_state *cli, int quota_fnum, 
SMB_NTQUOTA_LIST **pqt_list);
-bool cli_get_fs_quota_info(struct cli_state *cli, int quota_fnum, 
SMB_NTQUOTA_STRUCT *pqt);
-bool cli_set_fs_quota_info(struct cli_state *cli, int quota_fnum, 
SMB_NTQUOTA_STRUCT *pqt);
-void dump_ntquota(SMB_NTQUOTA_STRUCT *qt, bool _verbose, bool _numeric, void 
(*_sidtostring)(fstring str, struct dom_sid *sid, bool _numeric));
-void dump_ntquota_list(SMB_NTQUOTA_LIST **qtl, bool _verbose, bool _numeric, 
void (*_sidtostring)(fstring str, struct dom_sid *sid, bool _numeric));
+NTSTATUS cli_get_user_quota(struct cli_state *cli, int quota_fnum,
+                           SMB_NTQUOTA_STRUCT *pqt);
+NTSTATUS cli_set_user_quota(struct cli_state *cli, int quota_fnum,
+                           SMB_NTQUOTA_STRUCT *pqt);
+NTSTATUS cli_list_user_quota(struct cli_state *cli, int quota_fnum,
+                            SMB_NTQUOTA_LIST **pqt_list);
+NTSTATUS cli_get_fs_quota_info(struct cli_state *cli, int quota_fnum,
+                              SMB_NTQUOTA_STRUCT *pqt);
+NTSTATUS cli_set_fs_quota_info(struct cli_state *cli, int quota_fnum,
+                              SMB_NTQUOTA_STRUCT *pqt);
 
 /* The following definitions come from libsmb/clirap.c  */
 
@@ -2293,12 +2298,12 @@ NTSTATUS cli_qpathinfo1(struct cli_state *cli,
                        time_t *write_time,
                        SMB_OFF_T *size,
                        uint16 *mode);
-bool cli_setpathinfo(struct cli_state *cli, const char *fname,
-                     time_t create_time,
-                     time_t access_time,
-                     time_t write_time,
-                     time_t change_time,
-                     uint16 mode);
+NTSTATUS cli_setpathinfo_basic(struct cli_state *cli, const char *fname,
+                              time_t create_time,
+                              time_t access_time,
+                              time_t write_time,
+                              time_t change_time,
+                              uint16 mode);
 struct tevent_req *cli_qpathinfo2_send(TALLOC_CTX *mem_ctx,
                                       struct event_context *ev,
                                       struct cli_state *cli,
@@ -2580,24 +2585,6 @@ size_t clistr_align_out(struct cli_state *cli, const 
void *p, int flags);
 
 /* The following definitions come from libsmb/clitrans.c  */
 
-bool cli_send_trans(struct cli_state *cli, int trans,
-                   const char *pipe_name,
-                   int fid, int flags,
-                   uint16 *setup, unsigned int lsetup, unsigned int msetup,
-                   const char *param, unsigned int lparam, unsigned int mparam,
-                   const char *data, unsigned int ldata, unsigned int mdata);
-bool cli_receive_trans(struct cli_state *cli,int trans,
-                              char **param, unsigned int *param_len,
-                              char **data, unsigned int *data_len);
-bool cli_send_nt_trans(struct cli_state *cli,
-                      int function,
-                      int flags,
-                      uint16 *setup, unsigned int lsetup, unsigned int msetup,
-                      char *param, unsigned int lparam, unsigned int mparam,
-                      char *data, unsigned int ldata, unsigned int mdata);
-bool cli_receive_nt_trans(struct cli_state *cli,
-                         char **param, unsigned int *param_len,
-                         char **data, unsigned int *data_len);
 struct tevent_req *cli_trans_send(
        TALLOC_CTX *mem_ctx, struct event_context *ev,
        struct cli_state *cli, uint8_t cmd,
diff --git a/source3/libsmb/clientgen.c b/source3/libsmb/clientgen.c
index 2c49a8b..3816419 100644
--- a/source3/libsmb/clientgen.c
+++ b/source3/libsmb/clientgen.c
@@ -766,25 +766,6 @@ bool cli_set_case_sensitive(struct cli_state *cli, bool 
case_sensitive)
        return ret;
 }
 
-/****************************************************************************
-Send a keepalive packet to the server
-****************************************************************************/
-
-bool cli_send_keepalive(struct cli_state *cli)
-{
-        if (cli->fd == -1) {
-                DEBUG(3, ("cli_send_keepalive: fd == -1\n"));
-                return false;
-        }
-        if (!send_keepalive(cli->fd)) {
-                close(cli->fd);
-                cli->fd = -1;
-                DEBUG(0,("Error sending keepalive packet to client.\n"));
-                return false;
-        }
-        return true;
-}
-
 struct cli_echo_state {
        uint16_t vwv[1];
        DATA_BLOB data;
@@ -973,7 +954,7 @@ NTSTATUS cli_smb(TALLOC_CTX *mem_ctx, struct cli_state *cli,
                              pnum_bytes, pbytes);
 fail:
         TALLOC_FREE(ev);
-       if (NT_STATUS_IS_OK(status)) {
+       if (NT_STATUS_IS_OK(status) && (result_parent != NULL)) {
                *result_parent = req;
        }
         return status;
diff --git a/source3/libsmb/clifile.c b/source3/libsmb/clifile.c
index 331777b..68dd6c8 100644
--- a/source3/libsmb/clifile.c
+++ b/source3/libsmb/clifile.c
@@ -3,17 +3,17 @@
    client file operations
    Copyright (C) Andrew Tridgell 1994-1998
    Copyright (C) Jeremy Allison 2001-2009
-   
+
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
    the Free Software Foundation; either version 3 of the License, or
    (at your option) any later version.
-   
+
    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    GNU General Public License for more details.
-   
+
    You should have received a copy of the GNU General Public License
    along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
@@ -205,6 +205,41 @@ NTSTATUS cli_setpathinfo_recv(struct tevent_req *req)
        return tevent_req_simple_recv_ntstatus(req);
 }
 
+NTSTATUS cli_setpathinfo(struct cli_state *cli,
+                        uint16_t level,
+                        const char *path,
+                        uint8_t *data,
+                        size_t data_len)
+{
+       TALLOC_CTX *frame = talloc_stackframe();
+       struct tevent_context *ev;
+       struct tevent_req *req;
+       NTSTATUS status = NT_STATUS_NO_MEMORY;
+
+       if (cli_has_async_calls(cli)) {
+               /*
+                * Can't use sync call while an async call is in flight
+                */
+               status = NT_STATUS_INVALID_PARAMETER;
+               goto fail;
+       }
+       ev = tevent_context_init(frame);
+       if (ev == NULL) {
+               goto fail;
+       }
+       req = cli_setpathinfo_send(ev, ev, cli, level, path, data, data_len);
+       if (req == NULL) {
+               goto fail;
+       }
+       if (!tevent_req_poll_ntstatus(req, ev, &status)) {
+               goto fail;
+       }
+       status = cli_setpathinfo_recv(req);
+ fail:
+       TALLOC_FREE(frame);
+       return status;
+}
+
 /****************************************************************************
  Hard/Symlink a file (UNIX extensions).
  Creates new name (sym)linked to oldname.
@@ -2462,48 +2497,38 @@ NTSTATUS cli_locktype(struct cli_state *cli, uint16_t 
fnum,
                      uint32_t offset, uint32_t len,
                      int timeout, unsigned char locktype)
 {
-       char *p;
-       int saved_timeout = cli->timeout;
-
-       memset(cli->outbuf,'\0',smb_size);
-       memset(cli->inbuf,'\0', smb_size);
-
-       cli_set_message(cli->outbuf,8,0,True);
-
-       SCVAL(cli->outbuf,smb_com,SMBlockingX);
-       SSVAL(cli->outbuf,smb_tid,cli->cnum);
-       cli_setup_packet(cli);
-
-       SCVAL(cli->outbuf,smb_vwv0,0xFF);
-       SSVAL(cli->outbuf,smb_vwv2,fnum);
-       SCVAL(cli->outbuf,smb_vwv3,locktype);
-       SIVALS(cli->outbuf, smb_vwv4, timeout);
-       SSVAL(cli->outbuf,smb_vwv6,0);
-       SSVAL(cli->outbuf,smb_vwv7,1);
-
-       p = smb_buf(cli->outbuf);
-       SSVAL(p, 0, cli->pid);
-       SIVAL(p, 2, offset);
-       SIVAL(p, 6, len);
+       uint16_t vwv[8];
+       uint8_t bytes[10];
+       NTSTATUS status;
+       int saved_timeout;
 
-       p += 10;
+       SCVAL(vwv + 0, 0, 0xff);
+       SCVAL(vwv + 0, 1, 0);
+       SSVAL(vwv + 1, 0, 0);
+       SSVAL(vwv + 2, 0, fnum);
+       SCVAL(vwv + 3, 0, locktype);
+       SCVAL(vwv + 3, 1, 0);
+       SIVALS(vwv + 4, 0, timeout);
+       SSVAL(vwv + 6, 0, 0);
+       SSVAL(vwv + 7, 0, 1);
 
-       cli_setup_bcc(cli, p);
+       SSVAL(bytes, 0, cli->pid);
+       SIVAL(bytes, 2, offset);
+       SIVAL(bytes, 6, len);
 
-       cli_send_smb(cli);
+       saved_timeout = cli->timeout;
 
        if (timeout != 0) {
-               cli->timeout = (timeout == -1) ? 0x7FFFFFFF : (timeout + 
2*1000);
+               cli->timeout = (timeout == -1)
+                       ? 0x7FFFFFFF : (timeout + 2*1000);
        }
 
-       if (!cli_receive_smb(cli)) {
-               cli->timeout = saved_timeout;
-               return NT_STATUS_UNSUCCESSFUL;
-       }
+       status = cli_smb(talloc_tos(), cli, SMBlockingX, 0, 8, vwv,
+                        10, bytes, NULL, 0, NULL, NULL, NULL, NULL);
 
        cli->timeout = saved_timeout;
 
-       return cli_nt_error(cli);
+       return status;
 }
 
 /****************************************************************************
@@ -2512,54 +2537,15 @@ NTSTATUS cli_locktype(struct cli_state *cli, uint16_t 
fnum,
 ****************************************************************************/
 
 bool cli_lock(struct cli_state *cli, uint16_t fnum,
-             uint32_t offset, uint32_t len, int timeout, enum brl_type 
lock_type)
+                 uint32_t offset, uint32_t len, int timeout,
+                 enum brl_type lock_type)
 {
-       char *p;
-       int saved_timeout = cli->timeout;
-
-       memset(cli->outbuf,'\0',smb_size);
-       memset(cli->inbuf,'\0', smb_size);
-
-       cli_set_message(cli->outbuf,8,0,True);
-
-       SCVAL(cli->outbuf,smb_com,SMBlockingX);
-       SSVAL(cli->outbuf,smb_tid,cli->cnum);
-       cli_setup_packet(cli);
-
-       SCVAL(cli->outbuf,smb_vwv0,0xFF);
-       SSVAL(cli->outbuf,smb_vwv2,fnum);
-       SCVAL(cli->outbuf,smb_vwv3,(lock_type == READ_LOCK? 1 : 0));
-       SIVALS(cli->outbuf, smb_vwv4, timeout);
-       SSVAL(cli->outbuf,smb_vwv6,0);
-       SSVAL(cli->outbuf,smb_vwv7,1);
-
-       p = smb_buf(cli->outbuf);
-       SSVAL(p, 0, cli->pid);
-       SIVAL(p, 2, offset);
-       SIVAL(p, 6, len);
-
-       p += 10;
-
-       cli_setup_bcc(cli, p);
-
-       cli_send_smb(cli);
-
-       if (timeout != 0) {
-               cli->timeout = (timeout == -1) ? 0x7FFFFFFF : (timeout*2 + 
5*1000);
-       }
-
-       if (!cli_receive_smb(cli)) {
-               cli->timeout = saved_timeout;
-               return False;
-       }
-
-       cli->timeout = saved_timeout;
-
-       if (cli_is_error(cli)) {
-               return False;
-       }
+       NTSTATUS status;
 
-       return True;
+       status = cli_locktype(cli, fnum, offset, len, timeout,
+                             (lock_type == READ_LOCK? 1 : 0));
+       cli_set_error(cli, status);
+       return NT_STATUS_IS_OK(status);
 }
 
 /****************************************************************************
@@ -2681,11 +2667,14 @@ NTSTATUS cli_unlock(struct cli_state *cli,
 ****************************************************************************/
 
 bool cli_lock64(struct cli_state *cli, uint16_t fnum,
-               uint64_t offset, uint64_t len, int timeout, enum brl_type 
lock_type)
+               uint64_t offset, uint64_t len, int timeout,
+               enum brl_type lock_type)
 {
-       char *p;
+       uint16_t vwv[8];
+       uint8_t bytes[20];
         int saved_timeout = cli->timeout;
        int ltype;
+       NTSTATUS status;
 
        if (! (cli->capabilities & CAP_LARGE_FILES)) {
                return cli_lock(cli, fnum, offset, len, timeout, lock_type);
@@ -2694,47 +2683,34 @@ bool cli_lock64(struct cli_state *cli, uint16_t fnum,
        ltype = (lock_type == READ_LOCK? 1 : 0);
        ltype |= LOCKING_ANDX_LARGE_FILES;
 
-       memset(cli->outbuf,'\0',smb_size);
-       memset(cli->inbuf,'\0', smb_size);
-
-       cli_set_message(cli->outbuf,8,0,True);
-
-       SCVAL(cli->outbuf,smb_com,SMBlockingX);
-       SSVAL(cli->outbuf,smb_tid,cli->cnum);
-       cli_setup_packet(cli);
+       SCVAL(vwv + 0, 0, 0xff);
+       SCVAL(vwv + 0, 1, 0);
+       SSVAL(vwv + 1, 0, 0);
+       SSVAL(vwv + 2, 0, fnum);
+       SCVAL(vwv + 3, 0, ltype);
+       SCVAL(vwv + 3, 1, 0);
+       SIVALS(vwv + 4, 0, timeout);
+       SSVAL(vwv + 6, 0, 0);
+       SSVAL(vwv + 7, 0, 1);
 
-       SCVAL(cli->outbuf,smb_vwv0,0xFF);
-       SSVAL(cli->outbuf,smb_vwv2,fnum);
-       SCVAL(cli->outbuf,smb_vwv3,ltype);
-       SIVALS(cli->outbuf, smb_vwv4, timeout);
-       SSVAL(cli->outbuf,smb_vwv6,0);
-       SSVAL(cli->outbuf,smb_vwv7,1);
+       SIVAL(bytes, 0, cli->pid);
+       SOFF_T_R(bytes, 4, offset);
+       SOFF_T_R(bytes, 12, len);
 
-       p = smb_buf(cli->outbuf);
-       SIVAL(p, 0, cli->pid);
-       SOFF_T_R(p, 4, offset);
-       SOFF_T_R(p, 12, len);
-       p += 20;
-
-       cli_setup_bcc(cli, p);
-       cli_send_smb(cli);
+       saved_timeout = cli->timeout;
 
        if (timeout != 0) {
-               cli->timeout = (timeout == -1) ? 0x7FFFFFFF : (timeout + 
5*1000);
+               cli->timeout = (timeout == -1)
+                       ? 0x7FFFFFFF : (timeout + 2*1000);
        }
 
-       if (!cli_receive_smb(cli)) {
-                cli->timeout = saved_timeout;
-               return False;
-       }
+       status = cli_smb(talloc_tos(), cli, SMBlockingX, 0, 8, vwv,
+                        20, bytes, NULL, 0, NULL, NULL, NULL, NULL);
 
        cli->timeout = saved_timeout;
 
-       if (cli_is_error(cli)) {
-               return False;
-       }
-
-       return True;
+       cli_set_error(cli, status);
+       return NT_STATUS_IS_OK(status);
 }
 
 /****************************************************************************
@@ -4047,14 +4023,13 @@ NTSTATUS cli_raw_ioctl(struct cli_state *cli, uint16_t 
fnum, uint32_t code, DATA
 {
        uint16_t vwv[3];
        NTSTATUS status;
-       struct tevent_req *result_parent;
 
        SSVAL(vwv+0, 0, fnum);
        SSVAL(vwv+1, 0, code>>16);
        SSVAL(vwv+2, 0, (code&0xFFFF));
 
        status = cli_smb(talloc_tos(), cli, SMBioctl, 0, 3, vwv, 0, NULL,
-                        &result_parent, 0, NULL, NULL, NULL, NULL);
+                        NULL, 0, NULL, NULL, NULL, NULL);
        if (!NT_STATUS_IS_OK(status)) {
                return status;
        }


-- 
Samba Shared Repository

Reply via email to