samba.git
10 years agoautobuild.py add ntdb to the samba-libs task, to ensure it works as an external library
Andrew Bartlett [Mon, 13 May 2013 02:05:28 +0000 (14:05 +1200)]
autobuild.py add ntdb to the samba-libs task, to ensure it works as an external library

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agorpc_server-drsuapi: Include the failing DN when unable to convert DB objects to DRS
Andrew Bartlett [Tue, 2 Apr 2013 05:10:03 +0000 (16:10 +1100)]
rpc_server-drsuapi: Include the failing DN when unable to convert DB objects to DRS

This is a very serious situation (it should not happen) so getting information on the
objects that this happens for would be very helpful.

Andrew Bartlett

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agodsdb-schema: Print clear debug message when we find a OID in our local DB we cannot...
Andrew Bartlett [Tue, 2 Apr 2013 04:36:47 +0000 (15:36 +1100)]
dsdb-schema: Print clear debug message when we find a OID in our local DB we cannot convert

We need to work out why we are unable to make a mapping for an OID in our database, because
we should not have been able to add it without such a mapping.

Andrew Bartlett

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agodsdb-repl: Allow the name attribute (and name-based schema lookups) to be skipped...
Andrew Bartlett [Tue, 26 Mar 2013 00:51:38 +0000 (11:51 +1100)]
dsdb-repl: Allow the name attribute (and name-based schema lookups) to be skipped in dsdb_repl_make_working_schema()

This allows us to use a schema that may only be valid for attributeID based lookups, during the schema load.

Andrew Bartlett

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agopython-samba-tool domain classicupgrade: Use transactions when adding users/groups...
Andrew Bartlett [Sun, 14 Apr 2013 04:36:08 +0000 (14:36 +1000)]
python-samba-tool domain classicupgrade: Use transactions when adding users/groups/members

This should make things a bit faster when importing very large numbers of users
as we will not constantly rewrite the indicies on disk.

Andrew Bartlett

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agosamba-tool dbcheck: Use dsdb.DS_GUID_DELETED_OBJECTS_CONTAINER rather than the litera...
Andrew Bartlett [Sun, 14 Apr 2013 03:32:05 +0000 (13:32 +1000)]
samba-tool dbcheck: Use dsdb.DS_GUID_DELETED_OBJECTS_CONTAINER rather than the literal value

This is better practice.

Andrew Bartlett

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agopython-samba-tool domain classicupgrade: Correct message about re-promoting BDCs
Andrew Bartlett [Thu, 11 Apr 2013 08:13:16 +0000 (18:13 +1000)]
python-samba-tool domain classicupgrade: Correct message about re-promoting BDCs

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agopdb_ldap: Do not skip accounts without a sambaAcctFlags value
Andrew Bartlett [Thu, 11 Apr 2013 07:21:33 +0000 (17:21 +1000)]
pdb_ldap: Do not skip accounts without a sambaAcctFlags value

We allow this to mean a sambaAcctFlags value of zero in other parts of the code
and by allowing these users to show up in a search, we can read and correct them
during the classicupgrade, rather than not know they exist at all.

Most parts of the code do not look for ACB_NORMAL, which is why
these users appear to work.

Andrew Bartlett

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agopython-samba-tool domain classicupgrade: Actually Skip domain trust accounts
Andrew Bartlett [Wed, 10 Apr 2013 23:33:16 +0000 (09:33 +1000)]
python-samba-tool domain classicupgrade: Actually Skip domain trust accounts

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agopython-samba-tool domain classicupgrade: Skip machine accounts that do not end in $
Andrew Bartlett [Wed, 10 Apr 2013 23:32:13 +0000 (09:32 +1000)]
python-samba-tool domain classicupgrade: Skip machine accounts that do not end in $

These accounts will not work anyway, as all the domain member lookup code in netlogon expects the $.

Andrew Bartlett

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agobuild: Do not set PATH in install_with_python now we set $PYTHON
Andrew Bartlett [Wed, 8 May 2013 00:59:05 +0000 (12:59 +1200)]
build: Do not set PATH in install_with_python now we set $PYTHON

This will ensure that the scripts get rewritten to use this specific python, as it
will not be found in the path by the waf scripts.

Andrew Bartlett

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agos3-rpc_server: Ensure we are root when starting and usiing gensec
Andrew Bartlett [Thu, 2 May 2013 04:29:21 +0000 (16:29 +1200)]
s3-rpc_server: Ensure we are root when starting and usiing gensec

This fixes bug 9465.

Andrew Bartlett

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agogensec: Make the no-hostname status message much less scary
Andrew Bartlett [Mon, 29 Apr 2013 00:00:04 +0000 (10:00 +1000)]
gensec: Make the no-hostname status message much less scary

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agobuild: Blacklist the release-4-0-0 provision as well
Andrew Bartlett [Fri, 26 Apr 2013 06:44:00 +0000 (16:44 +1000)]
build: Blacklist the release-4-0-0 provision as well

This reduces the size of the published tarball while ensuring we test with it
during autobuild.

Andrew Bartlett

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agodns: Fix allocation of txt_record in txt record tests
Kai Blin [Thu, 16 May 2013 10:13:22 +0000 (12:13 +0200)]
dns: Fix allocation of txt_record in txt record tests

Signed-off-by: Kai Blin <kai@samba.org>
Reviewed-By: Amitay Isaacs <amitay@gmail.com>
Autobuild-User(master): Amitay Isaacs <amitay@samba.org>
Autobuild-Date(master): Thu May 16 15:39:15 CEST 2013 on sn-devel-104

10 years agodns: more debug debug options in the tests
Kai Blin [Sun, 13 Jan 2013 23:56:48 +0000 (00:56 +0100)]
dns: more debug debug options in the tests

Signed-off-by: Kai Blin <kai@samba.org>
Reviewed-By: Amitay Isaacs <amitay@gmail.com>
10 years agodns: Add support for MX queries
Kai Blin [Sun, 9 Dec 2012 19:50:05 +0000 (05:50 +1000)]
dns: Add support for MX queries

Due to an oversight, the internal DNS server supports MX record updates,
but not MX record queries. Add support for MX queries and tests.

This should fix bug #9485

Signed-off-by: Kai Blin <kai@samba.org>
Reviewed-By: Amitay Isaacs <amitay@gmail.com>
10 years agodns: Also add a print-out for the out_packet
Kai Blin [Sun, 13 Jan 2013 23:55:37 +0000 (00:55 +0100)]
dns: Also add a print-out for the out_packet

Signed-off-by: Kai Blin <kai@samba.org>
Reviewed-By: Amitay Isaacs <amitay@gmail.com>
10 years agodns: Use new DNS debugclass in DNS server
Kai Blin [Mon, 14 Jan 2013 00:14:29 +0000 (01:14 +0100)]
dns: Use new DNS debugclass in DNS server

Signed-off-by: Kai Blin <kai@samba.org>
Reviewed-By: Amitay Isaacs <amitay@gmail.com>
10 years agodebug: Add debugclass for DNS server
Kai Blin [Mon, 14 Jan 2013 00:13:47 +0000 (01:13 +0100)]
debug: Add debugclass for DNS server

Signed-off-by: Kai Blin <kai@samba.org>
Reviewed-By: Amitay Isaacs <amitay@gmail.com>
10 years agowinbind4: Fix bug 9832 -- talloc use after free
Volker Lendecke [Tue, 7 May 2013 08:17:26 +0000 (10:17 +0200)]
winbind4: Fix bug 9832 -- talloc use after free

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu May 16 13:37:41 CEST 2013 on sn-devel-104

10 years agosource4/winbind/wb_samba3_cmd.c: Fix typo in comment.
Karolin Seeger [Wed, 15 May 2013 16:24:42 +0000 (18:24 +0200)]
source4/winbind/wb_samba3_cmd.c: Fix typo in comment.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu May 16 07:49:24 CEST 2013 on sn-devel-104

10 years agosource4/dsdb/schema/schema_convert_to_ol.c: Fix typo in comment.
Karolin Seeger [Wed, 15 May 2013 16:23:49 +0000 (18:23 +0200)]
source4/dsdb/schema/schema_convert_to_ol.c: Fix typo in comment.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agosource4/dsdb/samdb/ldb_modules/local_password.c: Fix typo in comment.
Karolin Seeger [Wed, 15 May 2013 16:22:39 +0000 (18:22 +0200)]
source4/dsdb/samdb/ldb_modules/local_password.c: Fix typo in comment.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agosource4/auth/kerberos/kerberos-notes.txt: Fix typo.
Karolin Seeger [Wed, 15 May 2013 16:21:52 +0000 (18:21 +0200)]
source4/auth/kerberos/kerberos-notes.txt: Fix typo.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agosource4/TODO: Fix typo.
Karolin Seeger [Wed, 15 May 2013 16:20:42 +0000 (18:20 +0200)]
source4/TODO: Fix typo.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agosource3/utils/ntlm_auth.c: Fix typo in debug message.
Karolin Seeger [Wed, 15 May 2013 16:19:36 +0000 (18:19 +0200)]
source3/utils/ntlm_auth.c: Fix typo in debug message.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agosource3/modules/vfs_catia.c: Fix typo in comment.
Karolin Seeger [Wed, 15 May 2013 16:18:07 +0000 (18:18 +0200)]
source3/modules/vfs_catia.c: Fix typo in comment.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agosource3/libsmb/ntlmssp.c: Fix typo.
Karolin Seeger [Wed, 15 May 2013 16:16:51 +0000 (18:16 +0200)]
source3/libsmb/ntlmssp.c: Fix typo.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agosamba_tool/base.py: Fix typo.
Karolin Seeger [Wed, 15 May 2013 16:16:03 +0000 (18:16 +0200)]
samba_tool/base.py: Fix typo.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agonetcmd/group.py: Fix typo.
Karolin Seeger [Wed, 15 May 2013 16:15:18 +0000 (18:15 +0200)]
netcmd/group.py: Fix typo.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agoWireshark/Conformance.pm: Fix typo.
Karolin Seeger [Wed, 15 May 2013 16:13:44 +0000 (18:13 +0200)]
Wireshark/Conformance.pm: Fix typo.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agopidl/TODO: Fix typo.
Karolin Seeger [Wed, 15 May 2013 16:12:52 +0000 (18:12 +0200)]
pidl/TODO: Fix typo.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agolib/testtools/testtools/tests/test_testresult.py: Fix typos.
Karolin Seeger [Wed, 15 May 2013 16:11:10 +0000 (18:11 +0200)]
lib/testtools/testtools/tests/test_testresult.py: Fix typos.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agolib/subunit/README: Fix typo.
Karolin Seeger [Wed, 15 May 2013 16:09:51 +0000 (18:09 +0200)]
lib/subunit/README: Fix typo.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agodocs-xml: Fix typo in vfs_smb_traffic_analyzer.8.xml.
Karolin Seeger [Wed, 15 May 2013 16:08:51 +0000 (18:08 +0200)]
docs-xml: Fix typo in vfs_smb_traffic_analyzer.8.xml.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agogensec: Make gensec_security_oids_from_ops static
Volker Lendecke [Sat, 11 May 2013 08:36:12 +0000 (10:36 +0200)]
gensec: Make gensec_security_oids_from_ops static

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed May 15 20:05:34 CEST 2013 on sn-devel-104

10 years agogensec: Make gensec_security_by_sasl_list static
Volker Lendecke [Sat, 11 May 2013 08:35:59 +0000 (10:35 +0200)]
gensec: Make gensec_security_by_sasl_list static

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agogensec: Make gensec_interface_version public
Volker Lendecke [Sat, 11 May 2013 08:35:33 +0000 (10:35 +0200)]
gensec: Make gensec_interface_version public

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agosamba-tool/tests: Force the gecos of the user to a fixed value.
Matthieu Patou [Tue, 14 May 2013 13:20:18 +0000 (06:20 -0700)]
samba-tool/tests: Force the gecos of the user to a fixed value.

When --gecos is not specified samba-tool user add will try to read the
gecos field from a getpw call. And if user's GECOS is empty (like the
build user on sn-devel-104) then the test will fail because we can't add
an empty gecos.

Signed-off-by: Matthieu Patou <mat@matws.net>
Reviewed-by: Jelmer Vernooij <jelmer@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Wed May 15 16:19:23 CEST 2013 on sn-devel-104

10 years agodsdb-drepl: create a new schedulable event for running pending operations
Matthieu Patou [Wed, 15 May 2013 11:46:33 +0000 (04:46 -0700)]
dsdb-drepl: create a new schedulable event for running pending operations

So instead of running dreplsrv_periodic_schedule when receiving a
DRS_REPLICA_SYNC request which will force the DC to look for changes
with all the DC it usually replicate to, we reduce it to the DC
specified in the DRS_REPLICA_SYNC request. It will allow also to do have the
correct options as set by the client who send the DRS_REPLICA_SYNC.

Signed-off-by: Matthieu Patou <mat@matws.net>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agolib/param: remove unused 'printer_admin'
Stefan Metzmacher [Mon, 29 Apr 2013 10:27:25 +0000 (12:27 +0200)]
lib/param: remove unused 'printer_admin'

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 14 21:45:19 CEST 2013 on sn-devel-104

10 years agodoc-xml/smbdotconf: fix server [min|max] protocol documentation
Stefan Metzmacher [Tue, 30 Apr 2013 06:39:00 +0000 (08:39 +0200)]
doc-xml/smbdotconf: fix server [min|max] protocol documentation

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
10 years agotalloc: only provide the --enable-talloc-compat1 in standalone build
Stefan Metzmacher [Tue, 7 May 2013 12:15:35 +0000 (14:15 +0200)]
talloc: only provide the --enable-talloc-compat1 in standalone build

The compat library is already only built in standalone build,
so we need the configure option also only in the standalone build.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
10 years agowinbind/idmap_ad: be verbose about the user that we fail to map
Björn Jacke [Tue, 14 May 2013 14:51:28 +0000 (16:51 +0200)]
winbind/idmap_ad: be verbose about the user that we fail to map

Reviewed-by: Stefan Metzmacher <metze@samba.org>
10 years agovfs_gpfs: slightly simplify connect()
Volker Lendecke [Tue, 14 May 2013 13:43:10 +0000 (15:43 +0200)]
vfs_gpfs: slightly simplify connect()

DISCONNECT can be arbitrarily complex, TALLOC_FREE of a simple struct
is easier.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christian Ambach <ambi@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Tue May 14 18:11:29 CEST 2013 on sn-devel-104

10 years agos3:rpc_server/srvsvc check access before doing work
Shekhar Amlekar [Tue, 7 May 2013 12:28:16 +0000 (17:58 +0530)]
s3:rpc_server/srvsvc check access before doing work

Before doing the (potentially) costly enumerations,
check if the user has necessary privileges first

Signed-off-by: Shekhar Amlekar <samlekar@in.ibm.com>
Reviewed-by: Christian Ambach <ambi@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(master): Christian Ambach <ambi@samba.org>
Autobuild-Date(master): Tue May 14 16:22:24 CEST 2013 on sn-devel-104

10 years agotorture: Remove some unused code
Volker Lendecke [Sat, 11 May 2013 08:51:22 +0000 (10:51 +0200)]
torture: Remove some unused code

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christian Ambach <ambi@samba.org>
10 years agotdb: include information about hash function being used in tdbtool info output
Christian Ambach [Fri, 10 May 2013 22:45:15 +0000 (00:45 +0200)]
tdb: include information about hash function being used in tdbtool info output

makes it possible to easily determine if the tdb under examination
uses jenkins hash or not

Signed-off-by: Christian Ambach <ambi@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
10 years agosmbd: Remove a pointless variable
Volker Lendecke [Tue, 14 May 2013 08:52:33 +0000 (10:52 +0200)]
smbd: Remove a pointless variable

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Tue May 14 13:19:44 CEST 2013 on sn-devel-104

10 years agoselftests-drs: make our generated class subclass of classschema
Matthieu Patou [Mon, 6 May 2013 07:58:28 +0000 (00:58 -0700)]
selftests-drs: make our generated class subclass of classschema

Without this change objectclass=["top", "classSchema", "Foobar"] will
not be sorted correctly and will generated an error saying that class
Foobar is unreleated to classSchema (which is not true). It's mimicing what
other classes of the default schema are doing (ie. contact)

Signed-off-by: Matthieu Patou <mat@matws.net>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Tue May 14 07:07:19 CEST 2013 on sn-devel-104

10 years agoExport PROMOTED_DC related variable
Matthieu Patou [Mon, 13 May 2013 16:16:24 +0000 (09:16 -0700)]
Export PROMOTED_DC related variable

Signed-off-by: Matthieu Patou <mat@matws.net>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
10 years agodsdb: make the name of non related class more obvious
Matthieu Patou [Mon, 6 May 2013 08:09:05 +0000 (01:09 -0700)]
dsdb: make the name of non related class more obvious

Signed-off-by: Matthieu Patou <mat@matws.net>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
10 years agobuild: Add missing dep from vfs_nfs4acl_xattr to NDR_NFS4ACL
Andrew Bartlett [Mon, 13 May 2013 21:23:33 +0000 (09:23 +1200)]
build: Add missing dep from vfs_nfs4acl_xattr to NDR_NFS4ACL

Reviewed-by: Andreas Schneider <asn@samba.org>
Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date(master): Tue May 14 01:23:17 CEST 2013 on sn-devel-104

10 years agolib: Fix CID 241650 Sizeof not portable
Volker Lendecke [Sun, 12 May 2013 10:47:39 +0000 (12:47 +0200)]
lib: Fix CID 241650 Sizeof not portable

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): Sun May 12 17:44:55 CEST 2013 on sn-devel-104

10 years agopthreadpool: Fix CID 710828 Sizeof not portable
Volker Lendecke [Sun, 12 May 2013 10:44:41 +0000 (12:44 +0200)]
pthreadpool: Fix CID 710828 Sizeof not portable

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Ira Cooper <ira@samba.org>
10 years agodbwrap: Fix CID 1002092 Uninitialized scalar variable
Volker Lendecke [Sun, 12 May 2013 10:33:49 +0000 (12:33 +0200)]
dbwrap: Fix CID 1002092 Uninitialized scalar variable

We don't set saved_errno in the ret==0 case.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Ira Cooper <ira@samba.org>
10 years agodocs: smb.conf: fix max read/write/trans default values (bug #9871)
Björn Baumbach [Wed, 8 May 2013 08:27:26 +0000 (10:27 +0200)]
docs: smb.conf: fix max read/write/trans default values (bug #9871)

Commit 6d128aac119d948f0ecb0dcf6b400b4eb4027fe6 has increased the limit:
"s3:smb2_server increase defaults for read/write/trans sizes to 1MB"

Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Fri May 10 23:14:50 CEST 2013 on sn-devel-104

10 years agos4:torture fix a build break on AIX
Christian Ambach [Mon, 6 May 2013 17:00:29 +0000 (19:00 +0200)]
s4:torture fix a build break on AIX

Signed-off-by: Christian Ambach <ambi@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Thu May  9 08:05:12 CEST 2013 on sn-devel-104

10 years agos3:modules/vfs_aixacl2 fix compile errors
Christian Ambach [Mon, 6 May 2013 16:56:09 +0000 (16:56 +0000)]
s3:modules/vfs_aixacl2 fix compile errors

fix various compile errors that were introduced with latest ACL changes

Signed-off-by: Christian Ambach <ambi@samba.org>
Pair-Programmed-With: Alexander Werth <alexander.werth@de.ibm.com>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agoFix missing TALLOC_FREE of stackframes.
Jeremy Allison [Tue, 7 May 2013 21:04:24 +0000 (14:04 -0700)]
Fix missing TALLOC_FREE of stackframes.

Signed-off-by: Jeremy Allison <jra@samba.org>
10 years agoTidy up old bool usage. False -> false, True -> true.
Jeremy Allison [Tue, 7 May 2013 20:58:26 +0000 (13:58 -0700)]
Tidy up old bool usage. False -> false, True -> true.

Signed-off-by: Jeremy Allison <jra@samba.org>
10 years agovfs: Allow CREATOR GROUP to be used with vfs_zfsacl
Andrew Bartlett [Sun, 28 Apr 2013 08:20:04 +0000 (18:20 +1000)]
vfs: Allow CREATOR GROUP to be used with vfs_zfsacl

The solaris acl() code requires that both ACE_GROUP|ACE_IDENTIFIER_GROUP be
set to indicate the @group permissions.

Otherwise, it would return Invalid Paramter to clients.

Andrew Bartlett
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agos4-smbtorture: Run tests for nfs4:modes simple and special.
Alexander Werth [Sun, 28 Apr 2013 17:06:59 +0000 (19:06 +0200)]
s4-smbtorture: Run tests for nfs4:modes simple and special.

Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agos3: Update vfs_gpfs man page with new nfs4:mode help text.
Alexander Werth [Fri, 3 May 2013 03:46:25 +0000 (05:46 +0200)]
s3: Update vfs_gpfs man page with new nfs4:mode help text.

Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agos3: Update README.nfs4acls.txt
Alexander Werth [Thu, 2 May 2013 15:45:23 +0000 (17:45 +0200)]
s3: Update README.nfs4acls.txt

Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agos3: Use mode bits in some cases in mode simple.
Alexander Werth [Thu, 2 May 2013 14:53:35 +0000 (16:53 +0200)]
s3: Use mode bits in some cases in mode simple.

Non inheriting ACL entries will show mode bits.
With this an file owner change does affect the effective ACL because
the special owner acl will now refer to the new owner.
This could be fixed by updating the ACL on a file owner change.

Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agos3: Add changes that keep nfs4:mode special behavior.
Alexander Werth [Thu, 2 May 2013 14:50:55 +0000 (16:50 +0200)]
s3: Add changes that keep nfs4:mode special behavior.

Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agos3: Mapping of cifs creator owner to nfs owner@ ace.
Alexander Werth [Thu, 10 May 2012 12:19:41 +0000 (14:19 +0200)]
s3: Mapping of cifs creator owner to nfs owner@ ace.

This is ignored in nfs4mode special for compatibility.
Also ensure that we drop non inheriting creator owner
aces since these don't contribute to who can access
a file.

Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agos3: Mapping of special entries to creator owner in mode simple.
Alexander Werth [Wed, 25 Apr 2012 13:10:20 +0000 (15:10 +0200)]
s3: Mapping of special entries to creator owner in mode simple.

Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agos3: Add params parameter to smbacl4_nfs42win function.
Alexander Werth [Thu, 26 Jul 2012 15:29:12 +0000 (17:29 +0200)]
s3: Add params parameter to smbacl4_nfs42win function.

Reviewed-By: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agos3: Change smbacl4_get_vfs_params to use connection_struct instead of fsp.
Alexander Werth [Thu, 26 Jul 2012 15:11:03 +0000 (17:11 +0200)]
s3: Change smbacl4_get_vfs_params to use connection_struct instead of fsp.

Reviewed-By: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agos3: Move up declaration of params struct and related function.
Alexander Werth [Mon, 15 Apr 2013 14:08:46 +0000 (16:08 +0200)]
s3: Move up declaration of params struct and related function.

    We need the parameters earlier in the code so we move up
    the declaration of the params struct. Since reading the
    parameters is closely related the definition of the function
    smbacl4_get_vfs_params has also been moved up.

Reviewed-By: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agos4-smbtorture: Set result message when failing the inheritance test.
Alexander Werth [Thu, 18 Apr 2013 12:12:34 +0000 (14:12 +0200)]
s4-smbtorture: Set result message when failing the inheritance test.

Reviewed-By: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agovfs: Add inheritance emulation to vfs_nfs4acl_xattr.
Alexander Werth [Tue, 16 Apr 2013 12:11:27 +0000 (14:11 +0200)]
vfs: Add inheritance emulation to vfs_nfs4acl_xattr.

Recursively inherit ACL from parent directory if no acl xattr is
found on the current file.
Use a default ACL if a non-inheriting ACL is encountered.
With this the nfs4acl_xattr.dynamic test passes.
But the nfs4acl_xattr.inheritance test results in an error because
of warnings that cause the test to pass a failed result.

Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agoselftest: Run raw.acls test against the nfs4acl_xattr module
Andrew Bartlett [Sun, 14 Apr 2013 10:04:45 +0000 (20:04 +1000)]
selftest: Run raw.acls test against the nfs4acl_xattr module

This is the first time we have tested the NFSv4 ACL mapping code.
Sadly most tests fail but these can be fixed from here.

This at least shows that the code does not segfault.

Andrew Bartlett
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agolibrpc: Add special owner/group/other constants to nfs4acl.idl
Andrew Bartlett [Sun, 14 Apr 2013 09:51:42 +0000 (19:51 +1000)]
librpc: Add special owner/group/other constants to nfs4acl.idl

As per nfs4acl-0.9/lib/nfs4acl.c (the package where this structure is originally defined)

Andrew Bartlett
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agobuild: Add vfs_nfs4acl to the autoconf build
Andrew Bartlett [Wed, 8 May 2013 06:30:30 +0000 (18:30 +1200)]
build: Add vfs_nfs4acl to the autoconf build

Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agovfs: Add new VFS module vfs_nfs4acl_xattr to use nfs4acl.idl
Andrew Bartlett [Sun, 14 Apr 2013 10:06:57 +0000 (20:06 +1000)]
vfs: Add new VFS module vfs_nfs4acl_xattr to use nfs4acl.idl

This uses the xattr format used by the patches at http://users.suse.com/~agruen/nfs4acl/

Andrew Bartlett
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agovfs: Remove unused security_info argument in vfz_zfsacl.c
Andrew Bartlett [Sun, 14 Apr 2013 09:22:37 +0000 (19:22 +1000)]
vfs: Remove unused security_info argument in vfz_zfsacl.c

Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agovfs: Fix compile of vfs_gpfs.c.
Alexander Werth [Mon, 15 Apr 2013 14:35:36 +0000 (16:35 +0200)]
vfs: Fix compile of vfs_gpfs.c.

Since the smb4acl is now correctly allocated on mem_ctx and not
the talloc stack frame we can free the stack frame correctly.
And the chmod emulation code now needs the vfs handle since
that is now required by the callback function to set the smb4acl.

Reviewed-By: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agovfs: Allocate SMB4ACL_T on an explict memory context
Andrew Bartlett [Sun, 14 Apr 2013 08:13:42 +0000 (18:13 +1000)]
vfs: Allocate SMB4ACL_T on an explict memory context

This ensures the caller knows exactly what the memory lifetime of this
returned object is.  This makes the NFSv4 ACL code consistent with the
POSIX and NT ACL code, to avoid supprising developers who have worked
on those other parts of the ACL code.

Most of this patch is adding a memory context to the callers and passing it in.

Andrew Bartlett
Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agovfs: Add vfs_handle_struct argument to smb_set_nt_acl_nfs4 and the callback
Andrew Bartlett [Sun, 14 Apr 2013 07:31:42 +0000 (17:31 +1000)]
vfs: Add vfs_handle_struct argument to smb_set_nt_acl_nfs4 and the callback

This allows the callback to call xattr based storage functions that need this argument.

Andrew Bartlett

Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agobuild: Move nfs4acl to the top level
Andrew Bartlett [Fri, 12 Apr 2013 10:24:44 +0000 (20:24 +1000)]
build: Move nfs4acl to the top level

This is to create IDL-stored NFSv4 ACLs, just as we use for posix ACLs
to permit better testing.

Andrew Bartlett

Reviewed-by: Jeremy Allison <jra@samba.org>
10 years agopidl:NDR/Parser: correctly set $ndr->[relative_highest_]offset for relative_short...
Stefan Metzmacher [Wed, 8 May 2013 08:21:15 +0000 (10:21 +0200)]
pidl:NDR/Parser: correctly set $ndr->[relative_highest_]offset for relative_short pointers

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Günther Deschner <gd@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed May  8 20:49:55 CEST 2013 on sn-devel-104

10 years agoRevert "Remove a bunch of "unused variable _relative_save_offset" warnings."
Stefan Metzmacher [Wed, 8 May 2013 08:18:54 +0000 (10:18 +0200)]
Revert "Remove a bunch of "unused variable _relative_save_offset" warnings."

This reverts commit fa5898b6de797431d5ae9d2ce6dcddcb35a60b66.

This is the wrong fix for the warnings, the correct fix will follow.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Günther Deschner <gd@samba.org>
10 years agoRemove a bunch of "unused variable _relative_save_offset" warnings.
Jeremy Allison [Wed, 8 May 2013 00:07:10 +0000 (17:07 -0700)]
Remove a bunch of "unused variable _relative_save_offset" warnings.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
Autobuild-User(master): Richard Sharpe <sharpe@samba.org>
Autobuild-Date(master): Wed May  8 07:47:45 CEST 2013 on sn-devel-104

10 years agoTests processing an oplock break within a compound SMB2 request.
Richard Sharpe [Thu, 2 May 2013 21:36:05 +0000 (14:36 -0700)]
Tests processing an oplock break within a compound SMB2 request.

Signed-off-by: Richard Sharpe <realrichardsharpe@gmail.com>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Richard Sharpe <sharpe@samba.org>
Autobuild-Date(master): Tue May  7 19:45:36 CEST 2013 on sn-devel-104

10 years agoRemove the compound_related_in_progress state from the smb2 global state.
Jeremy Allison [Thu, 2 May 2013 21:16:22 +0000 (14:16 -0700)]
Remove the compound_related_in_progress state from the smb2 global state.

And also remove the restriction that we can't read a new
request whilst we're in this state.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Richard Sharpe <realrichardsharpe@samba.org>
10 years agoThe core of the fix to allow opens to go async inside a compound request.
Jeremy Allison [Thu, 2 May 2013 20:55:53 +0000 (13:55 -0700)]
The core of the fix to allow opens to go async inside a compound request.

This is only allowed for opens that cause an oplock break, otherwise it
is not allowed. See [MS-SMB2].pdf note <194> on Section 3.3.5.2.7.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
10 years agoMove a variable into the area of code where it's used.
Jeremy Allison [Thu, 2 May 2013 20:08:16 +0000 (13:08 -0700)]
Move a variable into the area of code where it's used.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
10 years agoEnsure we don't try and cancel anything that is in a compound-related request.
Jeremy Allison [Thu, 2 May 2013 19:34:54 +0000 (12:34 -0700)]
Ensure we don't try and cancel anything that is in a compound-related request.

Too hard to deal with splitting off the replies.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
10 years agoOnly do the 1 second delay for sharing violations for SMB1, not SMB2.
Jeremy Allison [Thu, 2 May 2013 18:12:47 +0000 (11:12 -0700)]
Only do the 1 second delay for sharing violations for SMB1, not SMB2.

Match Windows behavior.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
10 years agoMakefile: Fix bug 9868 - Don't know how to make LIBNDR_PREG_OBJ.
Volker Lendecke [Tue, 7 May 2013 13:07:34 +0000 (15:07 +0200)]
Makefile: Fix bug 9868 - Don't know how to make LIBNDR_PREG_OBJ.

Thanks to Lucs for finding the issue

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Günther Deschner <gd@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Tue May  7 17:57:57 CEST 2013 on sn-devel-104

10 years agowinbind: Fix bug 9854 -- NULL pointer dereference
Volker Lendecke [Tue, 7 May 2013 10:39:16 +0000 (12:39 +0200)]
winbind: Fix bug 9854 -- NULL pointer dereference

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Autobuild-User(master): Michael Adam <obnox@samba.org>
Autobuild-Date(master): Tue May  7 14:49:07 CEST 2013 on sn-devel-104

10 years agoFix up the man pages to explain that "store dos attributes" overrides them.
Jeremy Allison [Mon, 6 May 2013 21:10:58 +0000 (14:10 -0700)]
Fix up the man pages to explain that "store dos attributes" overrides them.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Tue May  7 01:24:54 CEST 2013 on sn-devel-104

10 years agoAllow "store dos attributes" to override the other "map XXX" parameters.
Jeremy Allison [Mon, 6 May 2013 21:10:15 +0000 (14:10 -0700)]
Allow "store dos attributes" to override the other "map XXX" parameters.

Makes us consistent with what is described in the man pages.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
10 years agobuild: default --with-regedit to "auto" instead of "yes"
Michael Adam [Mon, 6 May 2013 11:16:49 +0000 (13:16 +0200)]
build: default --with-regedit to "auto" instead of "yes"

This means we don't build regedit when there is no ncurses
and this is not an error for the overall build.

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Kai Blin <kai@samba.org>
Autobuild-User(master): Kai Blin <kai@samba.org>
Autobuild-Date(master): Mon May  6 20:11:09 CEST 2013 on sn-devel-104

10 years agobuild: fix --with-regedit to properly honour the yes/no/auto scheme
Michael Adam [Mon, 6 May 2013 12:14:02 +0000 (14:14 +0200)]
build: fix --with-regedit to properly honour the yes/no/auto scheme

I.e. fail configure when ncurses support is not found but
regedit build was requested.

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Kai Blin <kai@samba.org>
10 years agobuild: simplify ncurses checks: --with-regedit does not take a path list
Michael Adam [Mon, 6 May 2013 11:19:24 +0000 (13:19 +0200)]
build: simplify ncurses checks: --with-regedit does not take a path list

--with-regedit is defined using SAMBA3_ADD_OPTION(), and can hence
take the values "yes", "no", and "auto". So it is not possible to
hand in paths to look for ncurses-config via this option.

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Kai Blin <kai@samba.org>