The branch, master has been updated
       via  d51bb3137fa whitespace: Conform to coding convention
       via  a0fe965c362 param: Allow rpc server dynamic port range to roundtrip
      from  b730f350c22 s4:torture: fix file cleanup in smb2.create.delete

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


- Log -----------------------------------------------------------------
commit d51bb3137fad727baa487c4210ac4d5020bf7a1e
Author: Garming Sam <garm...@catalyst.net.nz>
Date:   Wed Sep 11 13:58:48 2019 +1200

    whitespace: Conform to coding convention
    
    Signed-off-by: Garming Sam <garm...@catalyst.net.nz>
    Reviewed-by: Noel Power <npo...@samba.org>
    
    Autobuild-User(master): Noel Power <npo...@samba.org>
    Autobuild-Date(master): Fri Sep 13 12:33:32 UTC 2019 on sn-devel-184

commit a0fe965c36218c5992ed2a6fe7c7fdb00b420702
Author: Garming Sam <garm...@catalyst.net.nz>
Date:   Wed Sep 11 13:55:37 2019 +1200

    param: Allow rpc server dynamic port range to roundtrip
    
    Originally this parameter only set two other parameters, but never set
    the original string. By setting the string as well, we can make it be
    emitted by testparm -v correctly (instead of ''), and set it back as the
    value for the parameter.
    
    Signed-off-by: Garming Sam <garm...@catalyst.net.nz>
    Reviewed-by: Noel Power <npo...@samba.org>

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

Summary of changes:
 lib/param/loadparm.c       | 16 +++++++++++++++-
 python/samba/tests/docs.py |  2 --
 2 files changed, 15 insertions(+), 3 deletions(-)


Changeset truncated at 500 lines:

diff --git a/lib/param/loadparm.c b/lib/param/loadparm.c
index 97addf45470..9a577aa188c 100644
--- a/lib/param/loadparm.c
+++ b/lib/param/loadparm.c
@@ -1445,7 +1445,7 @@ bool handle_smb_ports(struct loadparm_context *lp_ctx, 
struct loadparm_service *
                }
        }
 
-       if(!set_variable_helper(lp_ctx->globals->ctx, parm_num, ptr, "smb 
ports",
+       if (!set_variable_helper(lp_ctx->globals->ctx, parm_num, ptr, "smb 
ports",
                                pszParmValue)) {
                return false;
        }
@@ -1474,9 +1474,17 @@ bool handle_rpc_server_dynamic_port_range(struct 
loadparm_context *lp_ctx,
                                          const char *pszParmValue,
                                          char **ptr)
 {
+       static int parm_num = -1;
        int low_port = -1, high_port = -1;
        int rc;
 
+       if (parm_num == -1) {
+               parm_num = lpcfg_map_parameter("rpc server dynamic port range");
+               if (parm_num == -1) {
+                       return false;
+               }
+       }
+
        if (pszParmValue == NULL || pszParmValue[0] == '\0') {
                return false;
        }
@@ -1494,6 +1502,12 @@ bool handle_rpc_server_dynamic_port_range(struct 
loadparm_context *lp_ctx,
                return false;
        }
 
+       if (!set_variable_helper(lp_ctx->globals->ctx, parm_num, ptr,
+                                "rpc server dynamic port range",
+                                pszParmValue)) {
+               return false;
+       }
+
        lp_ctx->globals->rpc_low_port = low_port;
        lp_ctx->globals->rpc_high_port = high_port;
 
diff --git a/python/samba/tests/docs.py b/python/samba/tests/docs.py
index b6c15b7d8de..32a16a98fbc 100644
--- a/python/samba/tests/docs.py
+++ b/python/samba/tests/docs.py
@@ -123,7 +123,6 @@ class SmbDotConfTests(TestCase):
         'template homedir',
         'max open files',
         'include system krb5 conf',
-        'rpc server dynamic port range',
         'mit kdc command',
         'smbd max async dosmode',
     ])
@@ -247,7 +246,6 @@ class SmbDotConfTests(TestCase):
 
             exceptions = set([
                 'printing',
-                'rpc server dynamic port range',
                 'smbd max async dosmode',
             ])
 


-- 
Samba Shared Repository

Reply via email to