summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Revert "s3: Fix bug 8371"Stefan Metzmacher2012-06-031-2/+0
| | | | | | | | | | This reverts commit 2642f385887fbd3aecd4286a4d6223a21d981714. This is not needed anymore, as 035342c11719d1daa647c0b2ae7cec27a969f83a "Fix bug #8373 - Can't join XP Pro workstations to 3.6.1 DC." is the more generic fix for the problem. metze
* s3:smbd/close: call del_share_mode() directly before TALLOC_FREE(lck)Stefan Metzmacher2012-06-031-5/+15
| | | | | | | metze Autobuild-User: Stefan Metzmacher <metze@samba.org> Autobuild-Date: Sun Jun 3 21:08:19 CEST 2012 on sn-devel-104
* s3:smbd/close: remove unused goto out from close_directory()Stefan Metzmacher2012-06-031-3/+1
| | | | metze
* s3:smbd/close: do an early return in close_directory()Stefan Metzmacher2012-06-031-2/+1
| | | | metze
* s3: Fix an assert to trigger in close_remove_share_modeStefan Metzmacher2012-06-031-6/+17
| | | | | | | | | | | | | | | | | | | | | In case we have a left-over entry in the share mode entry array, the SMB_ASSERT(got_tokens) is likely to kick in. It happens when we are about to delete a file with initial delete on close. We don't have a delete on close token set in the locking.tdb record. We see the fsp->initial_delete_on_close set, add the delete_on_close token to lck. Then "delete_file" is being set to true. Then later on we do the notify_deferred_opens. This walks the list, also checking for share_mode_stale_pid. We have already deleted our own share mode entry, share_mode_stale_pid() sees the left-over entry. It not also deletes that one but also the delete on close token. This leads to a different view of "delete_file" a.k.a. "got_tokens" further down in close_remove_share_mode, leading the SMB_ASSERT to fire. This patch attempts to fix the issue by keeping around our own share mode entry for almost the whole routine, preventing share_mode_stale_pid() from removing the delete tokens. Pair-Programmed-With: Volker Lendecke <vl@samba.org>
* s3: Do an early return in close_remove_share_modeVolker Lendecke2012-06-031-2/+1
| | | | | | Without "lck" we don't have much to clean up Signed-off-by: Stefan Metzmacher <metze@samba.org>
* s3: Make close_remove_share_mode a bit more readableVolker Lendecke2012-06-031-2/+4
| | | | | | by introducing a variable simplifying a boolean expression Signed-off-by: Stefan Metzmacher <metze@samba.org>
* s3: Simplify get_delete_on_close_token moreVolker Lendecke2012-06-031-6/+2
| | | | | | All remaining callers actually want the token Signed-off-by: Stefan Metzmacher <metze@samba.org>
* s3: Simplify get_delete_on_close_token slightlyVolker Lendecke2012-06-031-19/+32
| | | | | | | Introduce find_delete_on_close_token. Thus is_delete_on_close_set does not have to call get_delete_on_close_token anymore. Signed-off-by: Stefan Metzmacher <metze@samba.org>
* s3: Slightly simplify close_remove_share_modeVolker Lendecke2012-06-031-10/+15
| | | | | | Remove a level of indentation by more use of "continue;" Signed-off-by: Stefan Metzmacher <metze@samba.org>
* s3:smbd/close: avoid procid_is_me()Stefan Metzmacher2012-06-031-1/+2
| | | | metze
* s3:smbd/open: avoid procid_is_me()Stefan Metzmacher2012-06-031-4/+5
| | | | metze
* s3:smbd/oplock: avoid procid_is_me()Stefan Metzmacher2012-06-031-2/+4
| | | | metze
* build: Match autoconf build and disable ACLs on MacOS (darwin)Andrew Bartlett2012-06-031-3/+7
| | | | | Autobuild-User: Andrew Bartlett <abartlet@samba.org> Autobuild-Date: Sun Jun 3 16:03:59 CEST 2012 on sn-devel-104
* lib/krb5_wrap: Remove unused smb_krb5_get_credsAndrew Bartlett2012-06-032-125/+0
| | | | | | | | | | | | | | | | | | | | | | Found by callcatcher This has been unsused since commit 61f0b247633501d6bf4103ca8345048e537c043d Author: Günther Deschner <gd@samba.org> Date: Thu Nov 12 15:42:03 2009 +0100 s3-kerberos: remove smb_krb5_get_tkt_from_creds(). Now that cli_krb5_get_ticket() already handles S4U2SELF impersonation, remove smb_krb5_get_tkt_from_creds() which is not required anymore. Guenther Andrew Bartlett Autobuild-User: Andrew Bartlett <abartlet@samba.org> Autobuild-Date: Sun Jun 3 13:04:06 CEST 2012 on sn-devel-104
* librepace: put #defines after #include "sys/xattr.h"Andrew Bartlett2012-06-032-40/+8
| | | | | | | | | This avoids redefining the system xattr functions, which should fix MacOS. Andrew Bartlett Autobuild-User: Andrew Bartlett <abartlet@samba.org> Autobuild-Date: Sun Jun 3 09:46:44 CEST 2012 on sn-devel-104
* lib/replace: Undo change of 0 -> NULLAndrew Bartlett2012-06-031-1/+1
| | | | | | | | | This reverts part of e9d797e153ae95561dbb10b56a41281b2472f137 as in the autoconf tests, NULL isn't available! (it is available in waf, which caused confusion) Andrew Bartlett
* s3-libnetapi: Cope with popt versions without POPT_TABLEENDAndrew Bartlett2012-06-032-0/+8
|
* libreplace: Fix up MacOS xattr functionsAndrew Bartlett2012-06-031-0/+18
| | | | | | | | | We need undo the rep_ macro to call the real OS function. Andrew Bartlett Autobuild-User: Andrew Bartlett <abartlet@samba.org> Autobuild-Date: Sun Jun 3 06:21:21 CEST 2012 on sn-devel-104
* s3-build: Remove finddeadAndrew Bartlett2012-06-031-9/+0
| | | | | | | | | | This won't work with the new tree structure, and we have the same output at https://build.samba.org/lcov/data/coverage/samba_4_0_test/unused-fns.txt Using the callcatcher framework. Andrew Bartlett
* libreplace: Link libreplace against attr when requiredAndrew Bartlett2012-06-032-2/+9
| | | | | | | | The autoconf Samba build will return to over-linking with -lattr on systems with both the XFS compat API and native xattrs. Autobuild-User: Andrew Bartlett <abartlet@samba.org> Autobuild-Date: Sun Jun 3 03:56:05 CEST 2012 on sn-devel-104
* libreplace: Use true rather than True in xattr.cAndrew Bartlett2012-06-031-2/+2
| | | | | | | | | This should fix the build on IRIX. Andrew Bartlett Autobuild-User: Andrew Bartlett <abartlet@samba.org> Autobuild-Date: Sun Jun 3 02:05:35 CEST 2012 on sn-devel-104
* libreplace: Fix build on MacOS where we have the same fn name but more argumentsAndrew Bartlett2012-06-021-8/+8
| | | | | Autobuild-User: Andrew Bartlett <abartlet@samba.org> Autobuild-Date: Sat Jun 2 15:52:51 CEST 2012 on sn-devel-104
* libreplace: Solaris needs system/dir.h for that xattr wrapper implementationAndrew Bartlett2012-06-021-0/+1
|
* build: Remove check for res_ninit (Revert "s3: Check for res_ninit")Andrew Bartlett2012-06-021-8/+0
| | | | | | | | This reverts commit 4f6b1e8c81776cf1fa386574e60e54fe6b4fe82e because I cannot find any use of this configure check, and it is one of the differences between the WAF and autoconf builds. Andrew Bartlett
* build: Show extra configure results found by WAF as wellAndrew Bartlett2012-06-021-1/+3
| | | | | | | | | | This may assist in debugging a difference in a configure test on the build farm. Andrew Bartlett Autobuild-User: Andrew Bartlett <abartlet@samba.org> Autobuild-Date: Sat Jun 2 09:46:09 CEST 2012 on sn-devel-104
* ccan: fix autoconf test for isblank()Andrew Bartlett2012-06-021-6/+6
| | | | | | The define in the C code is HAVE_ISBLANK Andrew Bartlett
* libreplace: Add missing tests for HAVE_MREMAP and HAVE_SHARED_MMAPAndrew Bartlett2012-06-021-0/+10
|
* libreplace: Fix autoconf build on platforms needing xattrsAndrew Bartlett2012-06-021-1/+1
| | | | | Autobuild-User: Andrew Bartlett <abartlet@samba.org> Autobuild-Date: Sat Jun 2 07:23:32 CEST 2012 on sn-devel-104
* build: Check correctly for sendfile (missing .h in header)Andrew Bartlett2012-06-021-1/+1
| | | | | | This will hopefully fix the build on SLES8, which does not support 64-bit sendfile. Andrew Bartlett
* lib/replace: Relicence xattr.c to LGPLv3Andrew Bartlett2012-06-021-11/+14
| | | | | | | | | | By the kind consent of the copyright holders. (There wasn't any code from tridge in the code brought in from source3/lib/system.c). Andrew Bartlett Autobuild-User: Andrew Bartlett <abartlet@samba.org> Autobuild-Date: Sat Jun 2 04:00:42 CEST 2012 on sn-devel-104
* build: Always attempt to build posix ACLsAndrew Bartlett2012-06-021-8/+3
| | | | | | | These are on more systems than just linux. If the configure test passes then assume it is available. Andrew Bartlett
* lib/replace: Merge remaining xattr test details from lib/utilAndrew Bartlett2012-06-026-40/+34
| | | | | | | I prefer the longer XATTR_ADDITIONAL_OPTIONS define and the NULL rather than 0 values in the getxattr test. Andrew Bartlett
* s4-xattr: Use libreplace xattr functions directlyAndrew Bartlett2012-06-0210-182/+18
|
* lib/replace: xattr wrappers in lib/replace rather than source3/lib/system.cAndrew Bartlett2012-06-029-791/+191
| | | | | | | This also moves all the still-used configure tests etc. The unused OSF API is also removed at this time. Andrew Bartlett
* lib/replace: We cannot use strchr_m in lib/replaceAndrew Bartlett2012-06-021-6/+6
| | | | | | In any case, it is always safe to search for . even in a multibyte string. Andrew Bartlett
* lib/replace: DEBUG is not acceptable here, as this may not be linked into SambaAndrew Bartlett2012-06-021-7/+0
|
* lib/replace: Copy lib/system.c xattr wrappers to lib/replaceAndrew Bartlett2012-06-021-0/+703
|
* s3: Same fix as 8576256, this time for fgetxattrVolker Lendecke2012-06-021-1/+1
| | | | | | | Signed-off-by: Jeremy Allison <jra@samba.org> Autobuild-User: Jeremy Allison <jra@samba.org> Autobuild-Date: Sat Jun 2 02:12:31 CEST 2012 on sn-devel-104
* s3: Avoid a lot of calls to serverid_exists()Volker Lendecke2012-06-011-1/+1
| | | | | | | Signed-off-by: Stefan Metzmacher <metze@samba.org> Autobuild-User: Stefan Metzmacher <metze@samba.org> Autobuild-Date: Fri Jun 1 18:58:30 CEST 2012 on sn-devel-104
* Revert "waf-mitkrb5: enable dcerpc_server library to support OpenChange ↵Alexander Bokovoy2012-06-013-41/+23
| | | | | | | | | | | | | | | client code" This reverts commit f8c447b1a48eaf12dcf70b92fd7525c4ad26c246. After discussing with Julien (Openchange) and Metze, I decided to revert this code. Instead I made a patch to Openchange which allows to build client side only. Openchange server code requires working s4 member DC and --without-ad-dc build does not provide working provisioning even if we enable dcerpc_server and end point mapper. Autobuild-User: Alexander Bokovoy <ab@samba.org> Autobuild-Date: Fri Jun 1 16:46:08 CEST 2012 on sn-devel-104
* waf: check for krb5_create_checksum and krb5_creds.flags for some Heimdal ↵Alexander Bokovoy2012-06-014-3/+29
| | | | | | | | | versions Signed-off-by: Andreas Schneider <asn@samba.org> Autobuild-User: Andreas Schneider <asn@cryptomilk.org> Autobuild-Date: Fri Jun 1 11:23:21 CEST 2012 on sn-devel-104
* waf-mitkrb5: enable dcerpc_server library to support OpenChange client codeAlexander Bokovoy2012-06-013-23/+41
| | | | Signed-off-by: Andreas Schneider <asn@samba.org>
* .gitignore: remove already-gone gen-8bit-gap.shAndrew Bartlett2012-06-011-1/+0
| | | | | Autobuild-User: Andrew Bartlett <abartlet@samba.org> Autobuild-Date: Fri Jun 1 04:53:53 CEST 2012 on sn-devel-104
* build: Add automatic compare of config.h files to recursive waf buildAndrew Bartlett2012-06-013-4/+27
| | | | | | | | | | | | This uses the fact that we have both build systems running at the same time. The krb5 checks are skipped because we typically are comparing internal Heimdal with the system krb5, so they do not make sense. The required checks for krb5 are pretty well understood in any case, as we have a limited set of supported libraries. Andrew Bartlett
* Remove an unused variable.Jeremy Allison2012-06-011-3/+1
| | | | | Autobuild-User: Jeremy Allison <jra@samba.org> Autobuild-Date: Fri Jun 1 03:03:12 CEST 2012 on sn-devel-104
* Forward port of Richard Sharpe's <realrichardsharpe@gmail.com> fix for bug ↵Jeremy Allison2012-05-315-4/+16
| | | | #8970 - Possible memory leaks in the samba master process.
* We are triggering the cleanup_timeout_fn() too often, on exiting when an ↵Jeremy Allison2012-05-311-11/+4
| | | | | | | smbd is idle. Calls to exit_server_cleanly() should be treated as a "clean" shutdown, and not trigger the master smbd to call cleanup_timeout_fn.
* s3:smbd: use server_messaging_context() instead of sconn->msg_ctx in ↵Stefan Metzmacher2012-05-311-2/+3
| | | | | | | | | | | | | exit_server_common() sconn is not available in the parent anymore. Thanks to Volker Lendecke <vl@samba.org> for finding this! metze Autobuild-User: Stefan Metzmacher <metze@samba.org> Autobuild-Date: Thu May 31 18:08:06 CEST 2012 on sn-devel-104
* s3:vfs_hpuxacl: remove unused file_find_fd() callsStefan Metzmacher2012-05-311-18/+2
| | | | metze