[SCM] Samba Shared Repository - branch master updated
The branch, master has been updated via 07fc59f Tests for bug #8972 - Add permission checks to run_simple_posix_open_test(). via 27bb016 Tests for bug #8972 - run the POSIX tests against the posix_share. via f57ca68 Tests for bug #8972 - Add a posix_share definition to s4 so we can do raw posix permissions checks. via acab344 Tests for bug #8972 - Add a posix_share definition to s3 so we can do raw posix permissions checks. from d6cd48f wintest: Fix test-s3.py to use smbclient3 under the name smbclient http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master - Log - commit 07fc59f78d816c4496623cc37026b82fd6b0bed3 Author: Jeremy Allison Date: Mon Jun 4 20:45:34 2012 -0700 Tests for bug #8972 - Add permission checks to run_simple_posix_open_test(). This requires a share with : create mask = 0777 force create mode = 0 directory mask = 0777 force directory mode = 0 set so we don't mess with requested permissions. Autobuild-User: Jeremy Allison Autobuild-Date: Tue Jun 5 08:20:03 CEST 2012 on sn-devel-104 commit 27bb01691ca71854ba146fb62b1114e6b612f562 Author: Jeremy Allison Date: Mon Jun 4 20:44:56 2012 -0700 Tests for bug #8972 - run the POSIX tests against the posix_share. commit f57ca68b1c26d2a93de4a6dc415817b7babcd87f Author: Jeremy Allison Date: Mon Jun 4 20:44:08 2012 -0700 Tests for bug #8972 - Add a posix_share definition to s4 so we can do raw posix permissions checks. commit acab34484718780dea8818ac0be5d6db78a06275 Author: Jeremy Allison Date: Mon Jun 4 20:43:09 2012 -0700 Tests for bug #8972 - Add a posix_share definition to s3 so we can do raw posix permissions checks. --- Summary of changes: selftest/target/Samba3.pm |9 + selftest/target/Samba4.pm |8 source3/selftest/tests.py | 10 -- source3/torture/torture.c | 27 +++ 4 files changed, 52 insertions(+), 2 deletions(-) Changeset truncated at 500 lines: diff --git a/selftest/target/Samba3.pm b/selftest/target/Samba3.pm index 098f532..a84b4e1 100755 --- a/selftest/target/Samba3.pm +++ b/selftest/target/Samba3.pm @@ -984,6 +984,15 @@ sub provision($$) comment = smb username is [%U] create mask = 777 force create mode = 777 +[posix_share] + path = $shrdir + comment = smb username is [%U] + create mask = 0777 + force create mode = 0 + directory mask = 0777 + force directory mode = 0 + vfs objects = $vfs_modulesdir_abs/xattr_tdb.so + [print\$] copy = tmp "; diff --git a/selftest/target/Samba4.pm b/selftest/target/Samba4.pm index 97cdbf2..954cf9c 100644 --- a/selftest/target/Samba4.pm +++ b/selftest/target/Samba4.pm @@ -745,6 +745,14 @@ sub provision() create mask = 777 force create mode = 777 +[posix_share] + path = $ctx->{share} + read only = no + create mask = 0777 + force create mode = 0 + directory mask = 0777 + force directory mode = 0 + [test1] path = $ctx->{share}/test1 read only = no diff --git a/source3/selftest/tests.py b/source3/selftest/tests.py index c131ca2..0d3bf04 100755 --- a/source3/selftest/tests.py +++ b/source3/selftest/tests.py @@ -85,8 +85,7 @@ tests=[ "FDPASS", "LOCK1", "LOCK2", "LOCK3", "LOCK4", "LOCK5", "LOCK6", "LOCK7", "OPEN", "XCOPY", "RENAME", "DELETE", "DELETE-LN", "PROPERTIES", "W2K", "TCON2", "IOCTL", "CHKPATH", "FDSESS", "CHAIN1", "CHAIN2", "CHAIN3", -"GETADDRINFO", "POSIX", "UID-REGRESSION-TEST", "SHORTNAME-TEST", -"POSIX-APPEND", +"GETADDRINFO", "UID-REGRESSION-TEST", "SHORTNAME-TEST", "CASE-INSENSITIVE-CREATE", "SMB2-BASIC", "NTTRANS-FSCTL", "SMB2-NEGPROT", "CLEANUP1", "CLEANUP2", @@ -97,6 +96,13 @@ for t in tests: plantestsuite("samba3.smbtorture_s3.crypt(s3dc).%s" % t, "s3dc", [os.path.join(samba3srcdir, "script/tests/test_smbtorture_s3.sh"), t, '//$SERVER_IP/tmp', '$USERNAME', '$PASSWORD', smbtorture3, "-e", "-l $LOCAL_PATH"]) plantestsuite("samba3.smbtorture_s3.plain(dc).%s" % t, "dc", [os.path.join(samba3srcdir, "script/tests/test_smbtorture_s3.sh"), t, '//$SERVER_IP/tmp', '$USERNAME', '$PASSWORD', smbtorture3, "", "-l $LOCAL_PATH"]) +posix_tests=[ "POSIX", "POSIX-APPEND"] + +for t in posix_tests: +plantestsuite("samba3.smbtorture_s3.plain(s3dc).%s" % t, "s3dc", [os.path.join(samba3srcdir, "script/tests/test_smbtorture_s3.sh"), t, '//$SERVER_IP/posix_share', '$USERNAME', '$PASSWORD', smbtorture3, "", "-l $LOCAL_PATH"]) +plantestsuite("samba3.smbtorture_s3.crypt(s3dc).%s" % t, "s3dc", [os.path.join(samba3srcdir, "script/tests/test_smbtorture_s3.sh"), t, '//$SERVER_IP/posix_share', '$USERNAME',
[SCM] Samba Shared Repository - branch master updated
The branch, master has been updated via d6cd48f wintest: Fix test-s3.py to use smbclient3 under the name smbclient via 1df3982 WHATSNEW: mention FreeBSD getaddrinfo() issue via b7754f4 s3:build(autoconf): fix "no AC_LANG_SOURCE call detected" warnings via d99bbb5 WHATSNEW: Mention trouble with DNS replication via e3d1d6c WHATSNEW: Note FreeBSD improvements via dc799d4 s4-provision: Test for posix ACLs in the actual sysvol path via d2b635c s4-provision: Place xattr.tdb in the actual state directory via 68769c2 WHATSNEW: Update for beta1 via d561ef2 VERSION: prepare for beta1 by setting and parsing the beta version via 1a2b65c Add '--use-ntvfs' option to 'samba-tool domain join' via 9db04c4 s3-smbd: Remove support for protocols before LANMAN1 via 3a13581 Remove Manifest. The information here was outdated via edd7251 libreplace: Add copyrights to ease tracking of this file in future via 5f58359 libreplace: Remove autoconf build system from 4accd23 Fix bug #8972 - Directory group write permission bit is set if unix extensions are enabled http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master - Log - commit d6cd48fca6bc69e7c8c5070790ec5d48a6dcf3f6 Author: Andrew Bartlett Date: Tue Jun 5 11:51:18 2012 +1000 wintest: Fix test-s3.py to use smbclient3 under the name smbclient Autobuild-User: Andrew Bartlett Autobuild-Date: Tue Jun 5 06:24:39 CEST 2012 on sn-devel-104 commit 1df39825e4e5e0db3654d1c2eac4697c30457ab2 Author: Andrew Bartlett Date: Tue Jun 5 11:41:52 2012 +1000 WHATSNEW: mention FreeBSD getaddrinfo() issue commit b7754f4bf7f1204e02baf6086c3cbbd7c6ebf1d9 Author: Martin Schwenke Date: Fri Dec 16 12:43:34 2011 +1100 s3:build(autoconf): fix "no AC_LANG_SOURCE call detected" warnings Autoconf 2.68 NEWS says: ** The macros AC_PREPROC_IFELSE, AC_COMPILE_IFELSE, AC_LINK_IFELSE, and AC_RUN_IFELSE now warn if the first argument failed to use AC_LANG_SOURCE or AC_LANG_PROGRAM to generate the conftest file contents. A new macro AC_LANG_DEFINES_PROVIDED exists if you have a compelling reason why you cannot use AC_LANG_SOURCE but must avoid the warning. Signed-off-by: Martin Schwenke commit d99bbb52e7763601a638ae24033e4b46691582be Author: Andrew Bartlett Date: Tue Jun 5 11:15:21 2012 +1000 WHATSNEW: Mention trouble with DNS replication commit e3d1d6c03c04b096b1c4e19eadeb58cf8bdba135 Author: Andrew Bartlett Date: Tue Jun 5 10:34:27 2012 +1000 WHATSNEW: Note FreeBSD improvements commit dc799d4b42551842a9c1dc863568611d82c6168d Author: Andrew Bartlett Date: Tue Jun 5 09:58:41 2012 +1000 s4-provision: Test for posix ACLs in the actual sysvol path This avoids issues in the previous location where lp may not be initialised at this point and instead simply waits until we have a known sysvol path, and test for ACL support there. Andrew Bartlett commit d2b635cc2f031364be1f25a7f8fb897a78764e5c Author: Andrew Bartlett Date: Tue Jun 5 09:56:53 2012 +1000 s4-provision: Place xattr.tdb in the actual state directory This patch does two things: it fixes up the spelling of "state dir" to "state directory" so that we actually find the smb.conf parameter, and we move it to after we process the global settings in case this is changed in the future. Andrew Bartlett commit 68769c26ff1bf7d6b36820c64e71169b35434061 Author: Andrew Bartlett Date: Tue Jun 5 09:21:15 2012 +1000 WHATSNEW: Update for beta1 commit d561ef20b5f87bd3a77939de7175daca45679e29 Author: Andrew Bartlett Date: Tue Jun 5 09:07:44 2012 +1000 VERSION: prepare for beta1 by setting and parsing the beta version commit 1a2b65cd31243f36f2a0a63bdad7f6d6f39afc1d Author: Andriy Syrovenko Date: Tue Jun 5 01:32:42 2012 +0300 Add '--use-ntvfs' option to 'samba-tool domain join' commit 9db04c481d981ee0ab285e0e34f32c49a7e84641 Author: Andrew Bartlett Date: Wed May 30 12:32:56 2012 +1000 s3-smbd: Remove support for protocols before LANMAN1 This falls out of the removal of security=share, because we now require that a session setup has been performed before (essentially) all other operations. Andrew Bartlett commit 3a1358121ab0d0d950252a0ec770a08bdda90809 Author: Andrew Bartlett Date: Mon May 28 13:08:45 2012 +1000 Remove Manifest. The information here was outdated commit edd725125792e848a9c1230ae4e57246dcda4352 Author: Andrew Bartlett Date: Tue Jun 5 08:00:00 2012 +1000 libreplace: Add copyrights to ease tracking of this file in future As GIT didn't realise this was a copy out of lib/system.c, this should make it easier to track the copyright holders on this file. Herb's name wasn't on the original file, but was
[SCM] Samba Shared Repository - branch master updated
The branch, master has been updated via 4accd23 Fix bug #8972 - Directory group write permission bit is set if unix extensions are enabled from 0110623 libreplace: Ensure we link xattr.c on MacOS http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master - Log - commit 4accd2396e8a9ac4a7b2191a1d3bc9c84d2e0497 Author: Jeremy Allison Date: Mon Jun 4 17:34:54 2012 -0700 Fix bug #8972 - Directory group write permission bit is set if unix extensions are enabled We can't manipulate file_attributes if it's a posix call. I'll look at adding a test for this asap. Autobuild-User: Jeremy Allison Autobuild-Date: Tue Jun 5 04:26:11 CEST 2012 on sn-devel-104 --- Summary of changes: source3/smbd/open.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) Changeset truncated at 500 lines: diff --git a/source3/smbd/open.c b/source3/smbd/open.c index 17721c6..a012675 100644 --- a/source3/smbd/open.c +++ b/source3/smbd/open.c @@ -2714,8 +2714,10 @@ static NTSTATUS open_directory(connection_struct *conn, return NT_STATUS_NOT_A_DIRECTORY; } - /* Ensure we have a directory attribute. */ - file_attributes |= FILE_ATTRIBUTE_DIRECTORY; + if (!(file_attributes & FILE_FLAG_POSIX_SEMANTICS)) { + /* Ensure we have a directory attribute. */ + file_attributes |= FILE_ATTRIBUTE_DIRECTORY; + } DEBUG(5,("open_directory: opening directory %s, access_mask = 0x%x, " "share_access = 0x%x create_options = 0x%x, " -- Samba Shared Repository