Matthieu Patou [Sat, 21 Jun 2014 04:24:40 +0000 (21:24 -0700)]
WIP: enable address-sanitizer
Change-Id: I6b5c3fbb5db8a98fa15be0601789339f6bf81513
Matthieu Patou [Mon, 25 May 2015 16:25:34 +0000 (09:25 -0700)]
Make addlotscontacts executable
Change-Id: I482e21f1663b52e35151f0fa4dd192158a6d0c5c
Matthieu Patou [Mon, 25 May 2015 16:21:58 +0000 (09:21 -0700)]
ldb-modules: avoid reallocating the attributes array if we don't have ranged attributes in the request
Instead of reallocating the array every time, just do it when needed.
It's accounting for 3% of runtime on dbcheck (talloc alone, there is
some other gains due to the lack of talloc_free to do).
Change-Id: I6f44d3582c9b496f78c0b0a42d61bf54dd4cbdbe
Signed-off-by: Matthieu Patou <mat@matws.net>
Matthieu Patou [Mon, 25 May 2015 16:17:55 +0000 (09:17 -0700)]
ldb: create a cache of known wellknown objects instead of continously searching in the db
Profiling on dbcheck have shown that we spend 10% of the time looking
for wellknown objects.
Change-Id: I13ed58e8062d1b7b6179d17b0e7e56f943572c6c
Signed-off-by: Matthieu Patou <mat@matws.net>
Matthieu Patou [Mon, 25 May 2015 16:16:50 +0000 (09:16 -0700)]
dbcheck: do not search for wellknown objects in schema partition: there is none
Change-Id: If9922e8a439c956619e3e02a626e7541dc7c0180
Matthieu Patou [Sat, 23 May 2015 07:02:42 +0000 (00:02 -0700)]
Fix segfault in the very rare case when we are not able to find the rootnamingcontext
Change-Id: I96fd5c7f39280090d5ec1dcdcb445fd7a44bd1c6
Matthieu Patou [Mon, 21 Jan 2013 07:10:38 +0000 (23:10 -0800)]
WIP: sort attributes in ldb when storing them in DB
Change-Id: I0e8d33f1f3ae74af894810cf7ff8dff59cf16d78
Matthieu Patou [Mon, 7 Jan 2013 08:20:47 +0000 (00:20 -0800)]
samba-tool: rewrite the reindex_database to use the latest indexing version
Also modify the @INDEXLIST attribute as it is designed for this purpose
Matthieu Patou [Sat, 22 Dec 2012 02:29:52 +0000 (18:29 -0800)]
dsdb-schema: ignore any difference in IDXMASTERVERSION as it's managed by ldb-tdb
Matthieu Patou [Sat, 22 Dec 2012 02:27:29 +0000 (18:27 -0800)]
provision: use casefolded DN indexing when creating new provision
Matthieu Patou [Sat, 18 May 2013 17:47:41 +0000 (10:47 -0700)]
ldb: Increase version to 1.1.21 due to ldb_dn_compare_base_one
Change-Id: I8d75e15926647a6bcf1f0202c145a3a50ba39f0a
Matthieu Patou [Fri, 28 Dec 2012 06:48:07 +0000 (22:48 -0800)]
ldb-tdb: construct the list of attributes when parsing the search tree
If the list is complete it will be used via ltdb_search_dn1_withlist in
order to limit the number of attributes that are actually returned.
Matthieu Patou [Fri, 28 Dec 2012 06:44:53 +0000 (22:44 -0800)]
ldb-tdb: use ldb_unpack_data_withlist via ltdb_search_dn1_withlist
This new function allow to specify a list of attributes to be unpacked
instead of unpacking all attributes. ltdb_search_dn1 is changed to call
the new function but with an empty list which will cause the full list
of attributes to be returned
Matthieu Patou [Sun, 23 Dec 2012 02:40:24 +0000 (18:40 -0800)]
ldb-tdb: Use talloc pool for the DNs used while doing indexed search
Matthieu Patou [Fri, 14 Dec 2012 09:02:09 +0000 (01:02 -0800)]
ldb-tdb: If the index has case folded DNs mark those DNs as casefolded
This will save a lot of time as casefolding DNs is very expensive
Matthieu Patou [Sun, 23 Dec 2012 00:00:36 +0000 (16:00 -0800)]
ldb-tdb: Store DNs in index entries already casefolded
Performance analysis have shown that casefolding index entries when
doing record lookup takes an important part of a ldb lookup, by storing
DNs already casefolded then can be used as is (almost) and so reduce
time and cpu spent in searches.
Matthieu Patou [Wed, 9 Jan 2013 06:13:10 +0000 (22:13 -0800)]
ldb-tdb: load index version and store the correct version of the index
Don't assume that every index stored is necesserly stored
with the highest version, on newly created index use the latest
version possible.
Matthieu Patou [Sun, 23 Dec 2012 00:06:49 +0000 (16:06 -0800)]
ldb-tdb: Use IDXMASTERVERSION for determining current indexing version in the ldb
Fallback on IDXVERSION that is used by Samba'DSDB to version rules for
casefolding attributes
Matthieu Patou [Wed, 9 Jan 2013 06:03:25 +0000 (22:03 -0800)]
ldb-tdb: keep the current indexing version in the ltdb->cache
Matthieu Patou [Fri, 14 Dec 2012 06:37:30 +0000 (22:37 -0800)]
ldb: Use ldb_dn_compare_base_one in ldb_match_scope
In order to return the number of element in a DN, it has to be exploded
if it's not already the case which is costly, in most case where this
function is called DN are still in non exploded form
Also the ldb_compare_base_one has quick test for DN that are linearized but
are not case folded.
Matthieu Patou [Sun, 30 Dec 2012 05:47:29 +0000 (21:47 -0800)]
ldb: Add ldb_dn_compare_base_one for checking if a dn is just bellow the base
Change-Id: I5626ef10f7c9ef28a1427ab309555c300fb696cd
Matthieu Patou [Thu, 13 Dec 2012 04:47:05 +0000 (20:47 -0800)]
ldb-tdb: split ldb_match_msg_error by 2 seprate calls to ldb_match_scope + ldb_match_message
Calling ldb_match_scope before the ldb_msg is actually loaded from the
database allow to skip the loading of entries that didn't match the
base DN of the request.
Depending on the number of records that will actually be skiped before
loading and unpacking rather than just after.
Matthieu Patou [Fri, 28 Dec 2012 05:38:29 +0000 (21:38 -0800)]
ldb: introduce ldb_unpack_data_withlist to unpack partial list of attributes
When provided with non NULL list ldb_unpack_data_withlist will only
unpack attributes that are specified in the list (+ distinguished name)
ldb_unpack_data is changed to call ldb_unpack_data_withlist behind the
scene.
Matthieu Patou [Fri, 14 Dec 2012 09:02:00 +0000 (01:02 -0800)]
ldb: Add ldb_dn_set_casefolded so that a DN can marked as already casefolded
This is intended for internal use in the indexing code when index
entries are DN stored already case folded.
This function will skip all the costly case folding calls and just mark
the DN as casefolded. The cf_name and cf_value of each componenent will
point to their counter part (name and value).
Matthieu Patou [Thu, 13 Dec 2012 05:15:57 +0000 (21:15 -0800)]
ldb: Make ldb_match_scope and ldb_match_message available to other ldb parts
This allow the indexing code to use them.
Matthieu Patou [Sat, 23 May 2015 09:34:14 +0000 (02:34 -0700)]
Add some documentation in ldb api tests
Change-Id: I466cd9ec5b6e21775760cdce984334c1eaab8d17
Matthieu Patou [Fri, 22 May 2015 12:57:42 +0000 (05:57 -0700)]
Extend ldb_dn_compare tests to target new changes on the to store ldb objects
That is to say we want to insure that spacing has no impact on the
hability to find objects.
Change-Id: I67fe0b868d9d8571be1f1478da83c7b7100fec01
Signed-off-by: Matthieu Patou <mat@matws.net>
Marc Muehlfeld [Tue, 19 May 2015 15:15:41 +0000 (17:15 +0200)]
Update 4.2 Roadmap file
The Roadmap file wasn't refreshed before releasing 4.2. It
still contained the content it had in 4.1. Updated the
file accordingly.
Signed-off-by: Marc Muehlfeld <mmuehlfeld@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Fri May 22 01:44:19 CEST 2015 on sn-devel-104
Karolin Seeger [Thu, 21 May 2015 13:00:32 +0000 (15:00 +0200)]
vfs_hpuxacl: Fix typo in comment...
and remove some trailing whitespaces.
Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Günther Deschner [Mon, 27 Apr 2015 18:46:32 +0000 (20:46 +0200)]
s3-spoolss: use talloc_zero in spoolss notify code
Signed-off-by: Günther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date(master): Thu May 21 22:53:16 CEST 2015 on sn-devel-104
Andreas Schneider [Mon, 27 Apr 2015 14:18:38 +0000 (16:18 +0200)]
testparm: Add warning if the netbios name is too long
Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Guenther Deschner <gd@samba.org>
Jeremy Allison [Thu, 21 May 2015 10:35:43 +0000 (12:35 +0200)]
s3: lib: util: Ensure we read a hex number as %x, not %u.
Based on a patch from Zoe O'Connell <zoe@metail.com>
BUG: https://bugzilla.samba.org/show_bug.cgi?id=11068
Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Jim McDonough <jmcd@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu May 21 15:34:59 CEST 2015 on sn-devel-104
Stefan Metzmacher [Mon, 18 May 2015 11:17:40 +0000 (13:17 +0200)]
s3:winbindd: make sure we remove pending io requests before closing client sockets
This avoids a crash inside the tevent epoll backend.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=11141
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Wed May 20 22:16:54 CEST 2015 on sn-devel-104
Stefan Metzmacher [Mon, 18 May 2015 11:25:33 +0000 (13:25 +0200)]
tevent: add a note to tevent_add_fd()
BUG: https://bugzilla.samba.org/show_bug.cgi?id=11141
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Alexander Bokovoy [Wed, 20 May 2015 08:17:38 +0000 (11:17 +0300)]
s4: libcli/finddcs_cldap: continue processing CLDAP until all addresses are used
This is a subtle bug that causes CLDAP pings to fail if SRV records
discovered cannot be resolved or connection to them cannot be
established. The code that fires up CLDAP ping will silently cancel
the whole tevent request without going to the next server in the queue.
This may happen, for example, when connection to IPv6 addresses couldn't
be established, or when IPv4 address is not online or blocked by
firewall.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=11284
Signed-off-by: Alexander Bokovoy <ab@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Stefan Metzmacher [Wed, 20 May 2015 13:40:58 +0000 (13:40 +0000)]
heimdal:lib/krb5: verify_logonname() to handle multi component principal
FreeIPA can generate tickets with a client principal of
'host/hostname.example.com'.
verify_logonname() should just verify the principal name
in the PAC_LOGON_NAME is the same as the principal of
the client principal (without realm) of the ticket.
Samba commit
b7cc8c1187ff967e44587cd0d09185330378f366
break this. We try to compare ['host']['hostname.example.com']
with ['host/hostname.example.com]' (as we interpret it as enterprise principal)
this fail if we don't compare them as strings.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=11142
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Uri Simchoni [Mon, 4 May 2015 07:21:53 +0000 (10:21 +0300)]
autobuild: Add test for cross-compilation infrastructure
With added option to generate cross-answers by cross-execute script,
a test is added to verify that at least for tested build environments,
the infrastructure is self-consistent.
The test runs samba configuration in three modes:
1. Native
2. Cross compile with cross-execute, using the build machine as its own
target
3. Cross compile with cross-answers, using the answers generated in
step 2
The build environments created by the three processes are then tested to
be equal.
Signed-off-by: Uri Simchoni <urisimchoni@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Alexander Bokovoy <ab@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Wed May 20 14:10:05 CEST 2015 on sn-devel-104
Uri Simchoni [Tue, 19 May 2015 08:50:41 +0000 (11:50 +0300)]
waf: add a script that compares configurations
This script is intended for use with the cross-build infrastructure
self-test during autobuild.
The script takes two or more "default.cache.py" files which are the
result of the waf configure process, and performs an intelligent
comparison - allowing for a limited set of variables to be different.
If the configurations are identical (apart from allowed differences) then
the script exit with 0 code.
If there are differences, the script outputs them to standard output
in unified diff format.
Signed-off-by: Uri Simchoni <urisimchoni@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Alexander Bokovoy <ab@samba.org>
Uri Simchoni [Tue, 19 May 2015 08:44:03 +0000 (11:44 +0300)]
waf: add a cross-execute program that runs natively
This added script is for supporting self-testing of the cross-build
infrastructure using autobuild.
The script can be used as the --cross-execute parameter
to samba's configure process, but it actually runs the program
natively on the build machine. This can be done if the build is
actually a native build disguised as a cross-build, i.e. native
toolchain is being used.
Signed-off-by: Uri Simchoni <urisimchoni@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Alexander Bokovoy <ab@samba.org>
Uri Simchoni [Mon, 4 May 2015 19:03:54 +0000 (22:03 +0300)]
waf: add a sample run-on-target script
This is a sample script that can be used with the --cross-execute
configure switch to run configure tests on a remote target for which
samba is being cross-compiled.
Signed-off-by: Uri Simchoni <urisimchoni@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Alexander Bokovoy <ab@samba.org>
Uri Simchoni [Mon, 4 May 2015 06:12:45 +0000 (09:12 +0300)]
waf: Fix parsing of cross-answers file in case answer includes a colon
The answer provided in the cross-answers file may include a colon,
as in:
Checking uname version type: "#57-Ubuntu SMP Tue Jul 15 03:51:08 UTC 2014"
Signed-off-by: Uri Simchoni <urisimchoni@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Alexander Bokovoy <ab@samba.org>
Uri Simchoni [Tue, 19 May 2015 08:09:55 +0000 (11:09 +0300)]
waf: allow cross-execute program to have arguments with spaces
When cross-compiling samba using the cross-execute method,
allow the cross-excute command to have arguments that contain
spaces by quoting them.
For example:
./configure --cross-compile '--cross-execute=ce-program "par am"'
In this case, for each cross test whose binary is /path/testprog,
waf shall run the equivalent of running from a shell:
ce-program "par am" /path/testprog
This is useful for passing an arbitrary argument list to a subprocess
of the cross-execute program, such as in buildtools/examples/run_on_target.py
Signed-off-by: Uri Simchoni <urisimchoni@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Alexander Bokovoy <ab@samba.org>
Uri Simchoni [Sun, 3 May 2015 19:56:15 +0000 (22:56 +0300)]
waf: improve readability of cross-answers generated by cross-execute
When generating a result for cross-answers from the (retcode, retstring) tuple:
- (0, "output") indicated as "output"
- 1 is interpreted as generic fail code, instead of 255, because most
if not all tests fail with 1 as exit code rather than 255
- For failing test, use NO instead of FAIL, because that's not
necessarily a failure (it could mean that something is NOT
broken)
Signed-off-by: Uri Simchoni <urisimchoni@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Alexander Bokovoy <ab@samba.org>
Uri Simchoni [Mon, 18 May 2015 18:15:19 +0000 (21:15 +0300)]
Adds a new mode to samba cross-compiling.
When both --cross-answers and --cross-execute are set, this means:
- Use cross-answers
- If answer is unknown, then instead of adding UNKNOWN to the cross-answers
file and failing configure, the new mode runs cross-execute to determine the
answer and adds that to the cross-answers file.
Signed-off-by: Uri Simchoni <urisimchoni@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Alexander Bokovoy <ab@samba.org>
Uri Simchoni [Mon, 18 May 2015 18:12:06 +0000 (21:12 +0300)]
waf: sanitize and fix added cross answer
When configuring samba for cross-compilation using the cross-answers
method, the function add_answer receives the standard output and exit code
of a configuration test and updates the cross-answers file accordingly.
This patch sanitizes the standard output to conform to the cross-answers
file format - one line of output. It also adds a missing newline.
(Note - at this point add_answer is only ever called with empty output
but this change is significant for the reminder of this patchset)
Signed-off-by: Uri Simchoni <urisimchoni@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Alexander Bokovoy <ab@samba.org>
Uri Simchoni [Mon, 18 May 2015 17:40:11 +0000 (20:40 +0300)]
waf: always close cross answers file
When configuring samba for cross-compilation using the
cross-answers file, the file is not closed in a couple of
cases - fix that.
Signed-off-by: Uri Simchoni <urisimchoni@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Alexander Bokovoy <ab@samba.org>
Jelmer Vernooij [Mon, 18 May 2015 20:19:23 +0000 (20:19 +0000)]
Factor out submodule presence checking.
Signed-off-by: Jelmer Vernooij <jelmer@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Tue May 19 22:17:48 CEST 2015 on sn-devel-104
Jelmer Vernooij [Mon, 18 May 2015 20:00:30 +0000 (20:00 +0000)]
Make waf fail if submodules are out of date.
Instead, suggest the user run 'git submodule update'.
This should prevent users from accidentally building Samba against
outdated or too new versions of the bundled third party libraries
after switching branches.
I've opted to make this an error rather than actually
running 'git submodule update' directly, as the latter could
cause unpredictable behaviour. If we find that manually updating
submodules is too cumbersome, we can always change this. The normal mode
of operation for developers should not involve any submodules at all,
but system versions of these libraries.
Signed-off-by: Jelmer Vernooij <jelmer@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Jelmer Vernooij [Mon, 18 May 2015 17:37:51 +0000 (17:37 +0000)]
Factor out function for finding Git binary.
Signed-Off-By: Jelmer Vernooij <jelmer@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Jelmer Vernooij [Mon, 18 May 2015 17:15:15 +0000 (17:15 +0000)]
Drop bzr VCS support.
Signed-Off-By: Jelmer Vernooij <jelmer@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Jelmer Vernooij [Tue, 5 May 2015 22:05:14 +0000 (22:05 +0000)]
Simplify instructions for checking out submodules.
Signed-off-by: Jelmer Vernooij <jelmer@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Petr Viktorin [Wed, 6 May 2015 16:05:18 +0000 (18:05 +0200)]
pytalloc: Test for all Python versions
Signed-off-by: Petr Viktorin <pviktori@redhat.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jelmer Vernooij <jelmer@samba.org>
Petr Viktorin [Tue, 10 Mar 2015 17:19:14 +0000 (18:19 +0100)]
buildtools: Add a helper for running Python tests
Add the function samba_utils.RUN_PYTHON_TESTS for running a Python
test. When building for multiple Python versions, all are tested.
Also, add the list of configured Python interpreters to build config.
Signed-off-by: Petr Viktorin <pviktori@redhat.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jelmer Vernooij <jelmer@samba.org>
Petr Viktorin [Wed, 6 May 2015 15:50:57 +0000 (17:50 +0200)]
buildtools: Add a helper to iterate through Python environments
This prevents code duplication to ensure the "extrapython" build
is the same as the normal one.
Signed-off-by: Petr Viktorin <pviktori@redhat.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jelmer Vernooij <jelmer@samba.org>
Petr Viktorin [Wed, 6 May 2015 16:17:06 +0000 (18:17 +0200)]
pytalloc: Build for two Python versions at once
Signed-off-by: Petr Viktorin <pviktori@redhat.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jelmer Vernooij <jelmer@samba.org>
Petr Viktorin [Thu, 15 Jan 2015 13:22:22 +0000 (14:22 +0100)]
buildtools: Add --extra-python configure option
This allows building Python support for two different Python versions
at the same time.
Signed-off-by: Petr Viktorin <pviktori@redhat.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jelmer Vernooij <jelmer@samba.org>
Petr Viktorin [Wed, 6 May 2015 10:45:42 +0000 (12:45 +0200)]
buildtools: Expose the Python 3 ABI tag
Expose the tag in the env to allow using it in pkg-config files
Add a "pyembed_libname" function to correctly form library names.
(This can't be done automatically in SAMBA_LIBRARY because the name
could be used as a dependency for another library)
Signed-off-by: Petr Viktorin <pviktori@redhat.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jelmer Vernooij <jelmer@samba.org>
Petr Viktorin [Thu, 15 Jan 2015 13:07:09 +0000 (14:07 +0100)]
pytalloc: Port to Python 3
- Use native string for repr
- Use rich comparison
Removes the deprecated tp_compare in favor of tp_richcompare.
Disparate types cannot be compared (except for == and !=),
and True or False objects are returned explicitly.
- Use Py_TYPE instead of ob_type
This changed to conform to C aliasing rules,
see http://legacy.python.org/dev/peps/pep-3123/
- Don't provide CObject creation function
A PyCapsule based replacement would be possible,
but might not be necessary considering the function is
not used much.
- Use new-style module initialization
Build changes:
- Use ABI flag in the lib name and pkg-config template
- Use the SAMBA_CHECK_PYTHON macro for finding Python
Signed-off-by: Petr Viktorin <pviktori@redhat.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jelmer Vernooij <jelmer@samba.org>
Matthieu Patou [Tue, 19 May 2015 10:39:59 +0000 (03:39 -0700)]
Fix a typo in the name of the default domain level
Change-Id: I2df44546a74f19ef2c6f1100d29b67fe7362070a
Signed-off-by: Matthieu Patou <mat@matws.net>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Tue May 19 19:26:22 CEST 2015 on sn-devel-104
Stefan Metzmacher [Thu, 18 Sep 2014 20:54:23 +0000 (22:54 +0200)]
s3:tevent_wait: simplify the code by using tevent_req_defer_callback()
This way a user of this could also use tevent_req_error() or wrappers.
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Tue May 19 16:37:52 CEST 2015 on sn-devel-104
Stefan Metzmacher [Fri, 10 Apr 2015 09:12:04 +0000 (11:12 +0200)]
s3:modules: remove unused allow_warnings=True for vfs_nfs4acl_xattr
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Stefan Metzmacher [Fri, 10 Apr 2015 09:29:59 +0000 (11:29 +0200)]
s3:vfs_nfs4acl_xattr: fix compiler warnings
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Stefan Metzmacher [Fri, 10 Apr 2015 09:12:04 +0000 (11:12 +0200)]
s3:modules: remove unused allow_undefined_symbols=False from vfs_glusterfs
This is the default...
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Stefan Metzmacher [Fri, 10 Apr 2015 09:12:04 +0000 (11:12 +0200)]
s3:modules: remove unused allow_undefined_symbols=True from vfs_aio_*
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Stefan Metzmacher [Fri, 10 Apr 2015 09:12:04 +0000 (11:12 +0200)]
s3:modules: remove unused allow_warnings=True for vfs_aio_fork
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Stefan Metzmacher [Fri, 10 Apr 2015 09:10:27 +0000 (11:10 +0200)]
s3:vfs_aio_fork: avoid -Wcast-qual warnings
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Stefan Metzmacher [Fri, 10 Apr 2015 09:12:04 +0000 (11:12 +0200)]
s3:modules: remove unused allow_warnings=True for vfs_preopen
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Stefan Metzmacher [Fri, 10 Apr 2015 09:31:31 +0000 (11:31 +0200)]
s3:vfs_preopen: avoid compiler warnings
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Stefan Metzmacher [Fri, 10 Apr 2015 09:12:04 +0000 (11:12 +0200)]
s3:modules: remove unused allow_warnings=True for vfs_unityed_media
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Stefan Metzmacher [Mon, 18 May 2015 13:00:14 +0000 (15:00 +0200)]
s3:wscript: remove unused uint[16,32] rpc.h checks
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Andreas Schneider [Tue, 19 May 2015 08:35:47 +0000 (10:35 +0200)]
auth: Make sure error_string is not used uninitialized
Signed-off-by: Andreas Schneider <asn@cryptomilk.org>
Reviewed-by: Simo Sorce <idra@samba.org>
Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date(master): Tue May 19 13:42:32 CEST 2015 on sn-devel-104
Richard Sharpe [Mon, 18 May 2015 00:21:06 +0000 (17:21 -0700)]
Remove [u]int[XX] defines from source3/include/includes.h
Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Mon May 18 05:24:22 CEST 2015 on sn-devel-104
Volker Lendecke [Tue, 28 Apr 2015 14:00:06 +0000 (14:00 +0000)]
Use tevent_req_poll_ntstatus
Kill 41 lines ..
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Richard Sharpe [Fri, 15 May 2015 21:56:25 +0000 (14:56 -0700)]
Convert the few instances of int32 there were to int32_t.
Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Sat May 16 03:09:12 CEST 2015 on sn-devel-104
Stefan Metzmacher [Fri, 15 May 2015 06:34:42 +0000 (08:34 +0200)]
pidl:Samba3/ServerNDR: normalize {RETURN_TYPE} by using mapTypeName()
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Fri May 15 22:19:38 CEST 2015 on sn-devel-104
Richard Sharpe [Fri, 15 May 2015 14:38:04 +0000 (07:38 -0700)]
Convert uint64 to uint64_t
We seemed to have very few uses of that.
Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Richard Sharpe [Fri, 15 May 2015 01:08:27 +0000 (18:08 -0700)]
Covert all uint32/16/8 to _t in source3/rpc_server.
This can be committed regardless of the state of the PIDL changes.
Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
David Holder [Wed, 13 May 2015 14:10:47 +0000 (15:10 +0100)]
s3: IPv6 enabled DNS connections for ADS client
This patch makes DNS client connections protocol independent.
For example DNS updates. This makes IPv6-only clients possible.
Signed-off-by: David Holder <david.holder@erion.co.uk>
Reviewed-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Böhme <rb@sernet.de>
Richard Sharpe [Thu, 14 May 2015 00:26:01 +0000 (17:26 -0700)]
Convert all uint32/16/8 to _t in a grab-bag of remaining files.
I still need to fix the rpc stuff, but we are almost there.
Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu May 14 22:16:56 CEST 2015 on sn-devel-104
Richard Sharpe [Thu, 14 May 2015 00:03:05 +0000 (17:03 -0700)]
Convert all uses of uint32/16/8 to _t in source3/groupdb.
Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Volker Lendecke [Thu, 14 May 2015 11:28:16 +0000 (11:28 +0000)]
namequery: Fix a missing TALLOC_FREE
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Ira Cooper <ira@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Thu May 14 16:49:39 CEST 2015 on sn-devel-104
Richard Sharpe [Tue, 12 May 2015 14:25:34 +0000 (07:25 -0700)]
Fix the uint32/16/8 stuff in two proto.h files I missed.
Signed-off-by: Richard Sharpe <rsharpe@gmail.com>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed May 13 22:01:14 CEST 2015 on sn-devel-104
Richard Sharpe [Mon, 11 May 2015 01:17:56 +0000 (18:17 -0700)]
Convert all uses of uint32/16/8 to _t in source3/auth.
Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Richard Sharpe [Sun, 10 May 2015 02:58:40 +0000 (19:58 -0700)]
Convert uint32/16/8 to _t in source3/libnet.
Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Richard Sharpe [Sun, 10 May 2015 02:49:09 +0000 (19:49 -0700)]
Convert all uint32/16/8 to _t in source3/client.
Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Richard Sharpe [Sun, 10 May 2015 00:06:28 +0000 (17:06 -0700)]
Convert a couple of uint32s to _t that I missed in source3/utils.
Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Richard Sharpe [Sat, 9 May 2015 23:59:45 +0000 (16:59 -0700)]
Convert all uint32/16/8 to _t in source3/libsmb.
Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Richard Sharpe [Sat, 9 May 2015 23:33:10 +0000 (16:33 -0700)]
Convert all uint32/16/8 to _t in source3/lib.
Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Richard Sharpe [Sat, 9 May 2015 23:14:39 +0000 (16:14 -0700)]
Convert all uint32/16/8 to _t in source3/locking.
Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Volker Lendecke [Wed, 13 May 2015 06:53:43 +0000 (08:53 +0200)]
namequery: namequery: suppress bogus warning "resolve_name: unknown name switch type"
Based on a patch by Uri Simchoni <urisimchoni@gmail.com>
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Martin Schwenke [Tue, 12 May 2015 04:42:00 +0000 (14:42 +1000)]
ctdb-doc: Document externally managed public IP addresses
For use with DisableIPFailover=1.
Pair-programmed-with: Amitay Isaacs <amitay@gmail.com>
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Autobuild-User(master): Amitay Isaacs <amitay@samba.org>
Autobuild-Date(master): Wed May 13 09:27:36 CEST 2015 on sn-devel-104
Martin Schwenke [Tue, 12 May 2015 04:11:59 +0000 (14:11 +1000)]
ctdb-tests: Add tickle test for external public address handling
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Martin Schwenke [Mon, 11 May 2015 05:29:34 +0000 (15:29 +1000)]
ctdb-scripts: New eventscript 10.external
This is an alternative to 10.interface and is installed as disabled by
default. It should only be used with DisableIPFailover=yes and when
IP failover is being handled externally. In this mode CTDB can be
informed of public IP address movements using "ctdb moveip".
During the "startup" event, this eventscript currently finds any
public IP addresses configured in $CTDB_PUBLIC_ADDRESSES and tells
CTDB which node they are on using "ctdb moveip". This allows CTDB to
send ARPs and tickle-ACKs.
Pair-programmed-with: Amitay Isaacs <amitay@gmail.com>
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Martin Schwenke [Fri, 8 May 2015 10:00:35 +0000 (20:00 +1000)]
ctdb-daemon: Move release of all IPs to startup
This means that DisableIPFailover will be set if it should be.
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Martin Schwenke [Fri, 8 May 2015 05:20:04 +0000 (15:20 +1000)]
ctdb-daemon: Never release all IPs when DisableIPFailover is set
If DisableIPFailover is set then something else may be managing public
IP addresses so CTDB should leave them alone.
Pair-programmed-with: Amitay Isaacs <amitay@gmail.com>
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Martin Schwenke [Fri, 8 May 2015 05:18:48 +0000 (15:18 +1000)]
ctdb-scripts: Drop all public IP addresses from 10.interface
00.ctdb should not know about public IP addresses.
Move related tests to operate on 10.interface.
Pair-programmed-with: Amitay Isaacs <amitay@gmail.com>
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Martin Schwenke [Fri, 8 May 2015 03:14:34 +0000 (13:14 +1000)]
ctdb-daemon: Don't update IP tree if DisableIPFailover is set
There won't be an IP tree. It is only ever initialised during a
takeover run.
The alternate to this would be to avoid sending
CTDB_SRVID_RECD_UPDATE_IP in "ctdb moveip". This logic is probably
best kept out of the CLI tool.
Pair-programmed-with: Amitay Isaacs <amitay@gmail.com>
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Martin Schwenke [Fri, 8 May 2015 07:11:24 +0000 (17:11 +1000)]
ctdb-daemon: Mark interfaces as "up" by default
The potential for public IP addresses to shuffle around during node
initialisation disappeared a while ago because IP addresses can only
be assigned to a node that is in CTDB_RUNSTATE_RUNNING. This means
that interfaces might as well just be initialised as "up". If any
interfaces are actually "down" then this will be rectified by the
"startup" event in 10.interfaces.
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Martin Schwenke [Thu, 7 May 2015 10:41:55 +0000 (20:41 +1000)]
ctdb-daemon: Skip "IP on interface" checks if DisableIPFailover is set
To support external failover of IP addresses if DisableIPFailover is
set. CTDB's idea of IP address assignment can be manipulated using
"ctdb moveip". Checking if the IP address is already held breaks
this in several places.
Pair-programmed-with: Amitay Isaacs <amitay@gmail.com>
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Martin Schwenke [Fri, 8 May 2015 02:34:43 +0000 (12:34 +1000)]
ctdb-daemon: Improve readability of code by nesting if-statements
ctdb_sys_have_ip() should only be run if if do_publicipcheck is set.
This is clearer if written as 2 nested if-statements rather than as a
lazy conjuction.
Pair-programmed-with: Amitay Isaacs <amitay@gmail.com>
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>