The branch, master has been updated
       via  58d3462 source3: Honor the core soft limit of the OS.
       via  85e414a s3/rpc_server/srvsvc/srv_srvsvc_nt.c: change snum to signed 
int
      from  2285519 s3,s4 torture/denytest: fix possible infinite loop

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


- Log -----------------------------------------------------------------
commit 58d3462bc58290d8eb5e554c6c59cf6b73ccf58a
Author: Ira Cooper <i...@samba.org>
Date:   Thu Mar 3 13:47:32 2016 -0500

    source3: Honor the core soft limit of the OS.
    
    We should honor the soft limits set by the operating system.
    
    In any case, 16M doesn't make a useful coredump for modern
    Samba.
    
    Signed-off-by: Ira Cooper <i...@samba.org>
    Reviewed-by: Jeremy Allison <j...@samba.org>
    
    Autobuild-User(master): Jeremy Allison <j...@samba.org>
    Autobuild-Date(master): Sat Mar  5 00:39:48 CET 2016 on sn-devel-144

commit 85e414aefc0c3e5db51d8426bdba62dc5bcc5420
Author: Aurelien Aptel <aap...@suse.com>
Date:   Thu Mar 3 16:30:35 2016 +0100

    s3/rpc_server/srvsvc/srv_srvsvc_nt.c: change snum to signed int
    
    snum is used to store the result of the find_service() call which
    returns a signed int, with -1 being an error.
    
    with an uint32_t the error check (< 0) is always false.
    
    Signed-off-by: Aurelien Aptel <aap...@suse.com>
    Reviewed-by: Volker Lendecke <volker.lende...@sernet.de>
    Reviewed-by: Jeremy Allison <j...@samba.org>

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

Summary of changes:
 source3/lib/dumpcore.c                    | 15 ---------------
 source3/rpc_server/srvsvc/srv_srvsvc_nt.c |  3 ++-
 2 files changed, 2 insertions(+), 16 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/lib/dumpcore.c b/source3/lib/dumpcore.c
index 487ccec..c72aa88 100644
--- a/source3/lib/dumpcore.c
+++ b/source3/lib/dumpcore.c
@@ -251,21 +251,6 @@ void dump_core_setup(const char *progname, const char 
*log_file)
                goto out;
        }
 
-
-#ifdef HAVE_GETRLIMIT
-#ifdef RLIMIT_CORE
-       {
-               struct rlimit rlp;
-               getrlimit(RLIMIT_CORE, &rlp);
-               rlp.rlim_cur = MAX(16*1024*1024,rlp.rlim_cur);
-               setrlimit(RLIMIT_CORE, &rlp);
-               getrlimit(RLIMIT_CORE, &rlp);
-               DEBUG(3,("Maximum core file size limits now %d(soft) 
%d(hard)\n",
-                        (int)rlp.rlim_cur,(int)rlp.rlim_max));
-       }
-#endif
-#endif
-
        /* FIXME: if we have a core-plus-pid facility, configurably set
         * this up here.
         */
diff --git a/source3/rpc_server/srvsvc/srv_srvsvc_nt.c 
b/source3/rpc_server/srvsvc/srv_srvsvc_nt.c
index 279cd9e..980411e 100644
--- a/source3/rpc_server/srvsvc/srv_srvsvc_nt.c
+++ b/source3/rpc_server/srvsvc/srv_srvsvc_nt.c
@@ -1125,7 +1125,8 @@ static WERROR init_srv_conn_info_1(const char *name,
                                   uint32_t *resume_handle_p,
                                   uint32_t *total_entries)
 {
-       uint32_t num_entries = 0, snum = 0;
+       uint32_t num_entries = 0;
+       int snum = 0;
        uint32_t resume_handle = resume_handle_p ? *resume_handle_p : 0;
        char *share_name = NULL;
        struct server_id *svrid_arr = NULL;


-- 
Samba Shared Repository

Reply via email to