From 4eb557bf258f2fcc771be130763634910f2a60d2 Mon Sep 17 00:00:00 2001 From: Stefan Metzmacher Date: Tue, 21 Apr 2009 15:33:53 +0200 Subject: Revert "Use an absolute path to ensure that we can always regenerate tables.c" This reverts commit b57c8ff4400e5f2bd0776247496b34dab68bde97. This actually breaks the merged build... Doing a merged build and a samba4 build within the same checkout, without a git clean -x -d -f (this removes everything that's not checked into git! save files you don't want to delete!) running in the toplevel directory, is not supported. metze --- source4/librpc/idl-deps.pl | 2 -- 1 file changed, 2 deletions(-) (limited to 'source4') diff --git a/source4/librpc/idl-deps.pl b/source4/librpc/idl-deps.pl index b3f0451eeb..4d97139fc5 100644 --- a/source4/librpc/idl-deps.pl +++ b/source4/librpc/idl-deps.pl @@ -1,7 +1,6 @@ #!/usr/bin/perl use strict; use File::Basename; -use Cwd 'abs_path'; sub add($$) { @@ -20,7 +19,6 @@ foreach(@ARGV) { my $gen_ndr = dirname($f); $gen_ndr =~ s/\/idl$/\/gen_ndr/; - $gen_ndr = abs_path($gen_ndr); print "# $f\n"; add("IDL_FILES", $f); -- cgit From 0bd8b7c5ff4d4d301297376dcabd0b8ca7d997c5 Mon Sep 17 00:00:00 2001 From: Jelmer Vernooij Date: Tue, 21 Apr 2009 16:50:20 +0200 Subject: Fix make install; don't install drsuapi.h and fix the location of credentials.h --- source4/headermap.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'source4') diff --git a/source4/headermap.txt b/source4/headermap.txt index 1c86f9e934..fb6be56936 100644 --- a/source4/headermap.txt +++ b/source4/headermap.txt @@ -30,7 +30,7 @@ auth/credentials/credentials.h: credentials.h auth/credentials/credentials_krb5.h: credentials/krb5.h rpc_server/dcerpc_server.h: dcerpc_server.h rpc_server/common/common.h: dcerpc_server/common.h -libcli/auth/credentials.h: domain_credentials.h +../libcli/auth/credentials.h: domain_credentials.h ../lib/util/charset/charset.h: charset.h libcli/ldap/ldap.h: ldap-util.h ../lib/torture/torture.h: torture.h -- cgit From c78a4ac7ced4abdc0bd090f13cde927356097b0e Mon Sep 17 00:00:00 2001 From: Björn Jacke Date: Mon, 30 Mar 2009 13:39:42 +0200 Subject: ѕ4: fix a "not handled in switch" compile warning MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- source4/libcli/smb_composite/connect.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'source4') diff --git a/source4/libcli/smb_composite/connect.c b/source4/libcli/smb_composite/connect.c index 3db777ddc8..a5d05b7af1 100644 --- a/source4/libcli/smb_composite/connect.c +++ b/source4/libcli/smb_composite/connect.c @@ -418,6 +418,8 @@ static void state_handler(struct composite_context *c) case CONNECT_TCON: c->status = connect_tcon(c, state->io); break; + case CONNECT_DONE: + break; } if (state->stage == CONNECT_DONE) { -- cgit From 42e3ae655091d48e5a921969849c6173a498c963 Mon Sep 17 00:00:00 2001 From: Björn Jacke Date: Mon, 30 Mar 2009 13:47:58 +0200 Subject: libcli: fix a "not handled in switch" compile warning return NULL also for RAW_CLOSE_GENERIC --- source4/libcli/raw/rawfile.c | 1 + 1 file changed, 1 insertion(+) (limited to 'source4') diff --git a/source4/libcli/raw/rawfile.c b/source4/libcli/raw/rawfile.c index 63de051b33..6fac7b8605 100644 --- a/source4/libcli/raw/rawfile.c +++ b/source4/libcli/raw/rawfile.c @@ -796,6 +796,7 @@ _PUBLIC_ struct smbcli_request *smb_raw_close_send(struct smbcli_tree *tree, uni break; case RAW_CLOSE_SMB2: + case RAW_CLOSE_GENERIC: return NULL; } -- cgit From 488bac4038e12a952fa912b64624facf17d6eff0 Mon Sep 17 00:00:00 2001 From: Björn Jacke Date: Mon, 30 Mar 2009 13:18:34 +0200 Subject: s4:ldb: do talloc_free and return NULL when we have no matches to return --- source4/dsdb/samdb/ldb_modules/anr.c | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'source4') diff --git a/source4/dsdb/samdb/ldb_modules/anr.c b/source4/dsdb/samdb/ldb_modules/anr.c index a04f5ebfa6..deeccac072 100644 --- a/source4/dsdb/samdb/ldb_modules/anr.c +++ b/source4/dsdb/samdb/ldb_modules/anr.c @@ -89,6 +89,7 @@ static struct ldb_parse_tree *make_match_tree(struct ldb_module *module, match_tree->u.substring.chunks = talloc_array(match_tree, struct ldb_val *, 2); if (match_tree->u.substring.chunks == NULL){ + talloc_free(match_tree); ldb_oom(ldb); return NULL; } @@ -99,6 +100,9 @@ static struct ldb_parse_tree *make_match_tree(struct ldb_module *module, match_tree->u.equality.attr = attr; match_tree->u.equality.value = *match; break; + default: + talloc_free(match_tree); + return NULL; } return match_tree; } -- cgit From 1563796b44f43be33fecf9907c967d0fdd14f0ff Mon Sep 17 00:00:00 2001 From: Björn Jacke Date: Fri, 3 Apr 2009 19:27:16 +0200 Subject: s4:ldb: fix extrasemi compile warning --- source4/dsdb/samdb/ldb_modules/partition.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'source4') diff --git a/source4/dsdb/samdb/ldb_modules/partition.c b/source4/dsdb/samdb/ldb_modules/partition.c index 3231f7ab0f..663ff074af 100644 --- a/source4/dsdb/samdb/ldb_modules/partition.c +++ b/source4/dsdb/samdb/ldb_modules/partition.c @@ -151,7 +151,7 @@ static struct dsdb_control_current_partition *find_partition(struct partition_pr } return NULL; -}; +} /** * fire the caller's callback for every entry, but only send 'done' once. -- cgit From 57267a300f35e8555ece9015c46353aa73e8eb2e Mon Sep 17 00:00:00 2001 From: Kai Blin Date: Sun, 5 Apr 2009 15:38:09 +0200 Subject: s4-build: Also search ../nsswitch for make (c|e)tags --- source4/build/make/rules.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'source4') diff --git a/source4/build/make/rules.mk b/source4/build/make/rules.mk index e38496ff34..96b2a1ff8b 100644 --- a/source4/build/make/rules.mk +++ b/source4/build/make/rules.mk @@ -182,7 +182,7 @@ showflags:: @echo ' MDLD_FLAGS = $(MDLD_FLAGS)' @echo ' SHLIBEXT = $(SHLIBEXT)' -base_srcdirs = $(srcdir) ../librpc/ ../lib/ ../libcli ../libgpo +base_srcdirs = $(srcdir) ../librpc/ ../lib/ ../libcli ../libgpo ../nsswitch etags: etags $(ETAGS_OPTIONS) `find $(base_srcdirs) -name "*.[ch]"` -- cgit From 7a9be21916589f2c2956e8b264648b66d074bfcb Mon Sep 17 00:00:00 2001 From: Kai Blin Date: Thu, 16 Apr 2009 11:49:25 +0200 Subject: errormap: Add wbcErr to NTSTATUS mappings --- source4/libcli/util/errormap.c | 37 +++++++++++++++++++++++++++++++++++++ 1 file changed, 37 insertions(+) (limited to 'source4') diff --git a/source4/libcli/util/errormap.c b/source4/libcli/util/errormap.c index 930e45b214..3ffadce5c6 100644 --- a/source4/libcli/util/errormap.c +++ b/source4/libcli/util/errormap.c @@ -21,6 +21,7 @@ #include "includes.h" #include "librpc/ndr/libndr.h" +#include "nsswitch/libwbclient/wbclient.h" /* This map was extracted by the ERRMAPEXTRACT smbtorture command. The setup was a Samba HEAD (2002-01-03) PDC and an Win2k member @@ -1406,3 +1407,39 @@ NTSTATUS ndr_map_error2ntstatus(enum ndr_err_code ndr_err) /* we should map all error codes to different status codes */ return NT_STATUS_INVALID_PARAMETER; } + +static const struct { + wbcErr wbc_err; + NTSTATUS nt_status; +} wbcErr_ntstatus_map[] = { + { WBC_ERR_SUCCESS, NT_STATUS_OK }, + { WBC_ERR_NOT_IMPLEMENTED, NT_STATUS_NOT_IMPLEMENTED }, + { WBC_ERR_UNKNOWN_FAILURE, NT_STATUS_INTERNAL_ERROR }, + { WBC_ERR_NO_MEMORY, NT_STATUS_NO_MEMORY }, + { WBC_ERR_INVALID_SID, NT_STATUS_INVALID_SID }, + { WBC_ERR_INVALID_PARAM, NT_STATUS_INVALID_PARAMETER }, + { WBC_ERR_WINBIND_NOT_AVAILABLE, NT_STATUS_SERVER_DISABLED }, + { WBC_ERR_DOMAIN_NOT_FOUND, NT_STATUS_NO_SUCH_DOMAIN }, + { WBC_ERR_INVALID_RESPONSE, NT_STATUS_INVALID_NETWORK_RESPONSE }, + { WBC_ERR_NSS_ERROR, NT_STATUS_INTERNAL_ERROR }, + { WBC_ERR_AUTH_ERROR, NT_STATUS_LOGON_FAILURE }, + { WBC_ERR_UNKNOWN_USER, NT_STATUS_NO_SUCH_USER }, + { WBC_ERR_UNKNOWN_GROUP, NT_STATUS_NO_SUCH_GROUP }, + { WBC_ERR_PWD_CHANGE_FAILED, NT_STATUS_PASSWORD_RESTRICTION } +}; + +NTSTATUS map_nt_error_from_wbcErr(wbcErr wbc_err) +{ + int i; + + /* Look through list */ + for (i=0;i Date: Thu, 23 Apr 2009 00:23:56 +0200 Subject: errormap: Change the WBC_ERR_UNKNOWN_ERROR to map to NT_STATUS_UNSUCCESSFUL This allows us to distinguish on NTSTATUS level if we got a WBC_ERR_UNKNOWN_ERROR or WBC_ERR_NSS_ERROR. --- source4/libcli/util/errormap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'source4') diff --git a/source4/libcli/util/errormap.c b/source4/libcli/util/errormap.c index 3ffadce5c6..22bf6121c9 100644 --- a/source4/libcli/util/errormap.c +++ b/source4/libcli/util/errormap.c @@ -1414,7 +1414,7 @@ static const struct { } wbcErr_ntstatus_map[] = { { WBC_ERR_SUCCESS, NT_STATUS_OK }, { WBC_ERR_NOT_IMPLEMENTED, NT_STATUS_NOT_IMPLEMENTED }, - { WBC_ERR_UNKNOWN_FAILURE, NT_STATUS_INTERNAL_ERROR }, + { WBC_ERR_UNKNOWN_FAILURE, NT_STATUS_UNSUCCESSFUL }, { WBC_ERR_NO_MEMORY, NT_STATUS_NO_MEMORY }, { WBC_ERR_INVALID_SID, NT_STATUS_INVALID_SID }, { WBC_ERR_INVALID_PARAM, NT_STATUS_INVALID_PARAMETER }, -- cgit From 951de687216b13e35876325e7b3c8081ea725dc8 Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Thu, 23 Apr 2009 10:49:41 +0200 Subject: Fix Coverity ID 628, Andrew B., please check! --- source4/rpc_server/lsa/dcesrv_lsa.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'source4') diff --git a/source4/rpc_server/lsa/dcesrv_lsa.c b/source4/rpc_server/lsa/dcesrv_lsa.c index 1f1f6afa6d..cbadf654fb 100644 --- a/source4/rpc_server/lsa/dcesrv_lsa.c +++ b/source4/rpc_server/lsa/dcesrv_lsa.c @@ -933,7 +933,7 @@ static NTSTATUS dcesrv_lsa_CreateTrustedDomain_base(struct dcesrv_call_state *dc if (r->in.info->trust_direction & LSA_TRUST_DIRECTION_INBOUND) { msg_user = ldb_msg_new(mem_ctx); - if (msg == NULL) { + if (msg_user == NULL) { ldb_transaction_cancel(trusted_domain_state->policy->sam_ldb); return NT_STATUS_NO_MEMORY; } -- cgit