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 <c...@samba.org> Date: Tue Apr 26 12:15:20 2016 -0700 ctdb/ltdbtool: Fix static declarations Signed-off-by: Christof Schmitt <c...@samba.org> Reviewed-by: Jeremy Allison <j...@samba.org> Autobuild-User(master): Jeremy Allison <j...@samba.org> Autobuild-Date(master): Sat Apr 30 04:28:13 CEST 2016 on sn-devel-144 commit e2642da130f96c542aa4962b76c1b9d1f677162d Author: Christof Schmitt <c...@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 <c...@samba.org> Reviewed-by: Jeremy Allison <j...@samba.org> commit 8bb4fccd2758cba5907134621ef57186e70a7073 Author: Christof Schmitt <c...@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 <c...@samba.org> Reviewed-by: Jeremy Allison <j...@samba.org> commit 3d33ebe36bc2aece6b16d86eb18d51c241c30fc5 Author: Christof Schmitt <c...@samba.org> Date: Sun Apr 17 21:27:31 2016 -0700 winbindd: Remove unused prototypes for winbindd_group.c Signed-off-by: Christof Schmitt <c...@samba.org> Reviewed-by: Jeremy Allison <j...@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