The branch, master has been updated
       via  de71a67 s3:libsmb/clifile: make use of cli_set_timeout()
       via  71c695d s3:cli_np_tstream: make use of cli_set_timeout()
       via  2abe723 s3:torture: make use of cli_set_timeout()
       via  71cec7b s3:winbindd_cm: make use of cli_set_timeout()
       via  b7d5cd9 s3:libsmb/clidfs: make use of cli_state_encryption_on()
      from  f854209 s4-libcli: Fix the fd leak. Close open file descriptor 
before return.

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


- Log -----------------------------------------------------------------
commit de71a67a1c442a72a7ab88674430b44d371c09d5
Author: Stefan Metzmacher <me...@samba.org>
Date:   Tue Aug 2 22:58:57 2011 +0200

    s3:libsmb/clifile: make use of cli_set_timeout()
    
    metze
    
    Autobuild-User: Stefan Metzmacher <me...@samba.org>
    Autobuild-Date: Wed Aug  3 10:16:18 CEST 2011 on sn-devel-104

commit 71c695d8d1ddcb4927daab4ad967f9fcfdff76c7
Author: Stefan Metzmacher <me...@samba.org>
Date:   Tue Aug 2 22:56:52 2011 +0200

    s3:cli_np_tstream: make use of cli_set_timeout()
    
    metze

commit 2abe723e60bd6ab4e5f0fd41cb233cd1e9c1aa5e
Author: Stefan Metzmacher <me...@samba.org>
Date:   Tue Aug 2 22:55:32 2011 +0200

    s3:torture: make use of cli_set_timeout()
    
    metze

commit 71cec7b37a221c42aeabe3b9c72878dd0714a60b
Author: Stefan Metzmacher <me...@samba.org>
Date:   Tue Aug 2 22:55:00 2011 +0200

    s3:winbindd_cm: make use of cli_set_timeout()
    
    metze

commit b7d5cd90d5eed90023b0266db386515ece911652
Author: Stefan Metzmacher <me...@samba.org>
Date:   Tue Aug 2 22:54:28 2011 +0200

    s3:libsmb/clidfs: make use of cli_state_encryption_on()
    
    metze

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

Summary of changes:
 source3/libsmb/cli_np_tstream.c |    3 ++-
 source3/libsmb/clidfs.c         |    4 ++--
 source3/libsmb/clifile.c        |   34 ++++++++++++++++++++++------------
 source3/torture/torture.c       |    4 ++--
 source3/winbindd/winbindd_cm.c  |    2 +-
 5 files changed, 29 insertions(+), 18 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/libsmb/cli_np_tstream.c b/source3/libsmb/cli_np_tstream.c
index 7521181..37fd68a 100644
--- a/source3/libsmb/cli_np_tstream.c
+++ b/source3/libsmb/cli_np_tstream.c
@@ -194,7 +194,8 @@ NTSTATUS _tstream_cli_np_open_recv(struct tevent_req *req,
        cli_nps->cli = state->cli;
        cli_nps->npipe = talloc_move(cli_nps, &state->npipe);
        cli_nps->fnum = state->fnum;
-       cli_nps->default_timeout = state->cli->timeout;
+       cli_nps->default_timeout = cli_set_timeout(state->cli, 0);
+       cli_set_timeout(state->cli, cli_nps->default_timeout);
 
        talloc_set_destructor(cli_nps, tstream_cli_np_destructor);
 
diff --git a/source3/libsmb/clidfs.c b/source3/libsmb/clidfs.c
index e54da6f..a445649 100644
--- a/source3/libsmb/clidfs.c
+++ b/source3/libsmb/clidfs.c
@@ -857,7 +857,7 @@ NTSTATUS cli_resolve_path(TALLOC_CTX *ctx,
                             "IPC$",
                             dfs_auth_info,
                             false,
-                            (rootcli->trans_enc_state != NULL),
+                            cli_state_encryption_on(rootcli),
                             cli_state_protocol(rootcli),
                             0,
                             0x20,
@@ -908,7 +908,7 @@ NTSTATUS cli_resolve_path(TALLOC_CTX *ctx,
                             share,
                             dfs_auth_info,
                             false,
-                            (rootcli->trans_enc_state != NULL),
+                            cli_state_encryption_on(rootcli),
                             cli_state_protocol(rootcli),
                             0,
                             0x20,
diff --git a/source3/libsmb/clifile.c b/source3/libsmb/clifile.c
index 812fcde..f9e3cbd 100644
--- a/source3/libsmb/clifile.c
+++ b/source3/libsmb/clifile.c
@@ -2594,7 +2594,8 @@ NTSTATUS cli_locktype(struct cli_state *cli, uint16_t 
fnum,
        uint16_t vwv[8];
        uint8_t bytes[10];
        NTSTATUS status;
-       int saved_timeout;
+       unsigned int set_timeout = 0;
+       unsigned int saved_timeout = 0;
 
        SCVAL(vwv + 0, 0, 0xff);
        SCVAL(vwv + 0, 1, 0);
@@ -2610,17 +2611,21 @@ NTSTATUS cli_locktype(struct cli_state *cli, uint16_t 
fnum,
        SIVAL(bytes, 2, offset);
        SIVAL(bytes, 6, len);
 
-       saved_timeout = cli->timeout;
-
        if (timeout != 0) {
-               cli->timeout = (timeout == -1)
-                       ? 0x7FFFFFFF : (timeout + 2*1000);
+               if (timeout == -1) {
+                       set_timeout = 0x7FFFFFFF;
+               } else {
+                       set_timeout = timeout + 2*1000;
+               }
+               saved_timeout = cli_set_timeout(cli, set_timeout);
        }
 
        status = cli_smb(talloc_tos(), cli, SMBlockingX, 0, 8, vwv,
                         10, bytes, NULL, 0, NULL, NULL, NULL, NULL);
 
-       cli->timeout = saved_timeout;
+       if (saved_timeout != 0) {
+               cli_set_timeout(cli, saved_timeout);
+       }
 
        return status;
 }
@@ -2761,7 +2766,8 @@ NTSTATUS cli_lock64(struct cli_state *cli, uint16_t fnum,
 {
        uint16_t vwv[8];
        uint8_t bytes[20];
-        int saved_timeout = cli->timeout;
+       unsigned int set_timeout = 0;
+       unsigned int saved_timeout = 0;
        int ltype;
        NTSTATUS status;
 
@@ -2786,17 +2792,21 @@ NTSTATUS cli_lock64(struct cli_state *cli, uint16_t 
fnum,
        SOFF_T_R(bytes, 4, offset);
        SOFF_T_R(bytes, 12, len);
 
-       saved_timeout = cli->timeout;
-
        if (timeout != 0) {
-               cli->timeout = (timeout == -1)
-                       ? 0x7FFFFFFF : (timeout + 2*1000);
+               if (timeout == -1) {
+                       set_timeout = 0x7FFFFFFF;
+               } else {
+                       set_timeout = timeout + 2*1000;
+               }
+               saved_timeout = cli_set_timeout(cli, set_timeout);
        }
 
        status = cli_smb(talloc_tos(), cli, SMBlockingX, 0, 8, vwv,
                         20, bytes, NULL, 0, NULL, NULL, NULL, NULL);
 
-       cli->timeout = saved_timeout;
+       if (saved_timeout != 0) {
+               cli_set_timeout(cli, saved_timeout);
+       }
 
        return status;
 }
diff --git a/source3/torture/torture.c b/source3/torture/torture.c
index 03a0cdf..d5c2a56 100644
--- a/source3/torture/torture.c
+++ b/source3/torture/torture.c
@@ -191,7 +191,7 @@ static struct cli_state *open_nbt_connection(void)
 
        c->use_kerberos = use_kerberos;
 
-       c->timeout = 120000; /* set a really long timeout (2 minutes) */
+       cli_set_timeout(c, 120000); /* set a really long timeout (2 minutes) */
        if (use_oplocks) c->use_oplocks = True;
        if (use_level_II_oplocks) c->use_level_II_oplocks = True;
 
@@ -374,7 +374,7 @@ static bool torture_open_connection_share(struct cli_state 
**c,
                return False;
        }
 
-       (*c)->timeout = 120000; /* set a really long timeout (2 minutes) */
+       cli_set_timeout(*c, 120000); /* set a really long timeout (2 minutes) */
 
        if (do_encrypt) {
                return force_cli_encryption(*c,
diff --git a/source3/winbindd/winbindd_cm.c b/source3/winbindd/winbindd_cm.c
index 3dd129f..26cd8bc 100644
--- a/source3/winbindd/winbindd_cm.c
+++ b/source3/winbindd/winbindd_cm.c
@@ -813,7 +813,7 @@ static NTSTATUS cm_prepare_connection(const struct 
winbindd_domain *domain,
                goto done;
        }
 
-       (*cli)->timeout = 10000;        /* 10 seconds */
+       cli_set_timeout(*cli, 10000); /* 10 seconds */
 
        (*cli)->use_kerberos = True;
 


-- 
Samba Shared Repository

Reply via email to