[SCM] Samba Shared Repository - branch master updated

Jeremy Allison jra at samba.org
Sat Apr 30 02:29:04 UTC 2016


The branch, master has been updated
       via  84a13e7 ctdb/ltdbtool: Fix static declarations
       via  e2642da smbcacls: Do not read old ACL for 'set' operation
       via  8bb4fcc gensec: Change log level of message when no PAC is found
       via  3d33ebe winbindd: Remove unused prototypes for winbindd_group.c
      from  e0d8c6b s3:client:smbspool_krb5_wrapper: fix the non clearenv build.

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


- Log -----------------------------------------------------------------
commit 84a13e70a61ae0b1207805bb08b34b0fb32e5248
Author: Christof Schmitt <cs at samba.org>
Date:   Tue Apr 26 12:15:20 2016 -0700

    ctdb/ltdbtool: Fix static declarations
    
    Signed-off-by: Christof Schmitt <cs at samba.org>
    Reviewed-by: Jeremy Allison <jra at samba.org>
    
    Autobuild-User(master): Jeremy Allison <jra at samba.org>
    Autobuild-Date(master): Sat Apr 30 04:28:13 CEST 2016 on sn-devel-144

commit e2642da130f96c542aa4962b76c1b9d1f677162d
Author: Christof Schmitt <cs at samba.org>
Date:   Fri Apr 29 15:12:38 2016 -0700

    smbcacls: Do not read old ACL for 'set' operation
    
    Signed-off-by: Christof Schmitt <cs at samba.org>
    Reviewed-by: Jeremy Allison <jra at samba.org>

commit 8bb4fccd2758cba5907134621ef57186e70a7073
Author: Christof Schmitt <cs at samba.org>
Date:   Thu Apr 28 11:23:41 2016 -0700

    gensec: Change log level of message when no PAC is found
    
    For a Samba server that uses a non-AD KDC this message is triggered on
    every new connection. Change the log level from warning/1 to a more
    appropriate notice/5.
    
    Signed-off-by: Christof Schmitt <cs at samba.org>
    Reviewed-by: Jeremy Allison <jra at samba.org>

commit 3d33ebe36bc2aece6b16d86eb18d51c241c30fc5
Author: Christof Schmitt <cs at samba.org>
Date:   Sun Apr 17 21:27:31 2016 -0700

    winbindd: Remove unused prototypes for winbindd_group.c
    
    Signed-off-by: Christof Schmitt <cs at samba.org>
    Reviewed-by: Jeremy Allison <jra at samba.org>

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

Summary of changes:
 auth/gensec/gensec_util.c         |  4 ++--
 ctdb/tools/ltdbtool.c             | 25 ++++++++++++++-----------
 source3/utils/smbcacls.c          | 14 ++++++++++----
 source3/winbindd/winbindd_proto.h | 11 -----------
 4 files changed, 26 insertions(+), 28 deletions(-)


Changeset truncated at 500 lines:

diff --git a/auth/gensec/gensec_util.c b/auth/gensec/gensec_util.c
index 64fffb1..da2e2e5 100644
--- a/auth/gensec/gensec_util.c
+++ b/auth/gensec/gensec_util.c
@@ -48,8 +48,8 @@ NTSTATUS gensec_generate_session_info_pac(TALLOC_CTX *mem_ctx,
 				  principal_string));
 			return NT_STATUS_ACCESS_DENIED;
 		}
-		DEBUG(1, ("Unable to find PAC for %s, resorting to local user lookup\n",
-			  principal_string));
+		DBG_NOTICE("Unable to find PAC for %s, resorting to local "
+			   "user lookup\n", principal_string);
 	}
 
 	if (gensec_security->auth_context && gensec_security->auth_context->generate_session_info_pac) {
diff --git a/ctdb/tools/ltdbtool.c b/ctdb/tools/ltdbtool.c
index 7ac70e2..2352079 100644
--- a/ctdb/tools/ltdbtool.c
+++ b/ctdb/tools/ltdbtool.c
@@ -302,9 +302,10 @@ ltdb_traverse_fn(TDB_CONTEXT* tdb, TDB_DATA key, TDB_DATA val,
 	return ctx->fn(tdb, key, val, &hdr.hdr, ctx->state);
 }
 
-int ltdb_traverse(TDB_CONTEXT *tdb,
-		  int (*fn)(TDB_CONTEXT *,TDB_DATA,TDB_DATA,struct ctdb_ltdb_header*,void *),
-		  void *state, int hsize, bool skip_empty)
+static int ltdb_traverse(TDB_CONTEXT *tdb,
+			 int (*fn)(TDB_CONTEXT*, TDB_DATA, TDB_DATA,
+				   struct ctdb_ltdb_header*, void *),
+			 void *state, int hsize, bool skip_empty)
 {
 	struct ltdb_traverse_ctx ctx = {
 		.fn = fn,
@@ -318,9 +319,9 @@ int ltdb_traverse(TDB_CONTEXT *tdb,
 	return (ret < 0) ? ret : (ret - ctx.nempty);
 }
 
-int write_record(TDB_CONTEXT* tdb, TDB_DATA key, TDB_DATA val,
-		 struct ctdb_ltdb_header* hdr,
-		 void* write_record_ctx)
+static int write_record(TDB_CONTEXT* tdb, TDB_DATA key, TDB_DATA val,
+			struct ctdb_ltdb_header* hdr,
+			void* write_record_ctx)
 {
 	struct write_record_ctx* ctx
 		= (struct write_record_ctx*)write_record_ctx;
@@ -347,9 +348,9 @@ int write_record(TDB_CONTEXT* tdb, TDB_DATA key, TDB_DATA val,
 	return 0;
 }
 
-int dump_record(TDB_CONTEXT* tdb, TDB_DATA key, TDB_DATA val,
-		struct ctdb_ltdb_header* hdr,
-		void* dump_record_ctx)
+static int dump_record(TDB_CONTEXT* tdb, TDB_DATA key, TDB_DATA val,
+		       struct ctdb_ltdb_header* hdr,
+		       void* dump_record_ctx)
 {
 	struct dump_record_ctx* ctx = (struct dump_record_ctx*)dump_record_ctx;
 
@@ -363,14 +364,16 @@ int dump_record(TDB_CONTEXT* tdb, TDB_DATA key, TDB_DATA val,
 	return 0;
 }
 
-void dump_header_full(struct dump_record_ctx* c, struct ctdb_ltdb_header* h)
+static void dump_header_full(struct dump_record_ctx* c,
+			     struct ctdb_ltdb_header* h)
 {
 	fprintf(c->file, "dmaster: %d\nrsn: %llu\nflags: 0x%X\n",
 		(int)h->dmaster,
 		(unsigned long long)h->rsn, h->flags);
 }
 
-void print_data_tdbdump(FILE* file, TDB_DATA data) {
+static void print_data_tdbdump(FILE* file, TDB_DATA data)
+{
 	unsigned char *ptr = data.dptr;
 	fputc('"', file);
 	while (data.dsize--) {
diff --git a/source3/utils/smbcacls.c b/source3/utils/smbcacls.c
index d50219c..5cb707d 100644
--- a/source3/utils/smbcacls.c
+++ b/source3/utils/smbcacls.c
@@ -505,10 +505,16 @@ static int cacl_set(struct cli_state *cli, const char *filename,
 	if (!sd) return EXIT_PARSE_ERROR;
 	if (test_args) return EXIT_OK;
 
-	old = get_secdesc(cli, filename);
-
-	if (!old) {
-		return EXIT_FAILED;
+	if (mode != SMB_ACL_SET) {
+		/*
+		 * Do not fetch old ACL when it will be overwritten
+		 * completely with a new one.
+		 */
+		old = get_secdesc(cli, filename);
+
+		if (!old) {
+			return EXIT_FAILED;
+		}
 	}
 
 	/* the logic here is rather more complex than I would like */
diff --git a/source3/winbindd/winbindd_proto.h b/source3/winbindd/winbindd_proto.h
index 49ae475..4d99927 100644
--- a/source3/winbindd/winbindd_proto.h
+++ b/source3/winbindd/winbindd_proto.h
@@ -306,17 +306,6 @@ NTSTATUS winbindd_reinit_after_fork(const struct winbindd_child *myself,
 struct winbindd_domain *wb_child_domain(void);
 
 /* The following definitions come from winbindd/winbindd_group.c  */
-
-void winbindd_getgrnam(struct winbindd_cli_state *state);
-void winbindd_getgrgid(struct winbindd_cli_state *state);
-void winbindd_setgrent(struct winbindd_cli_state *state);
-void winbindd_endgrent(struct winbindd_cli_state *state);
-void winbindd_getgrent(struct winbindd_cli_state *state);
-void winbindd_list_groups(struct winbindd_cli_state *state);
-void winbindd_getgroups(struct winbindd_cli_state *state);
-void winbindd_getusersids(struct winbindd_cli_state *state);
-void winbindd_getuserdomgroups(struct winbindd_cli_state *state);
-void winbindd_getsidaliases(struct winbindd_cli_state *state);
 bool fill_grent(TALLOC_CTX *mem_ctx, struct winbindd_gr *gr,
 		const char *dom_name, const char *gr_name, gid_t unix_gid);
 NTSTATUS winbindd_print_groupmembers(struct talloc_dict *members,


-- 
Samba Shared Repository



More information about the samba-cvs mailing list