s4 torture: Make RAW-SEARCH pass against win7

Signed-off-by: Tim Prouty <tprouty@samba.org>

s4 torture: Parameterize WRITE_AND_CLOSE support

Signed-off-by: Tim Prouty <tprouty@samba.org>

s4 torture: Add lockread_supported based off of CAP_LOCK_AND_READ

Signed-off-by: Tim Prouty <tprouty@samba.org>

s4 torture: Allow READ to be parameterized, add more readx tests

Signed-off-by: Tim Prouty <tprouty@samba.org>

s4-torture: Allow the legacy DENY_FCB/DENY_DOS share modes to be ignored

Signed-off-by: Tim Prouty <tprouty@samba.org>

s4 selftest/quicktest: Update known failures for the sfileinfo and oplock changes

Two new samba4 bugs have been filed for the two corresponding known
failures.  For the short term raw.sfileinfo had to be removed from
quicktest.  It is no longer an individual test, but a test-suite and
quicktest can only run top level tests.

s4 torture: Add a new RAW-OPLOCK test: BATCH26

Try a rename with a wide-open share mode on an already open file
and the there is still share mode contention.  For the reason why
see:

http://social.msdn.microsoft.com/Forums/en-US/os_fileservices/thread/3ca14dc9-da1f-4786-a8f7-a86e9903db0c

Msft's anser:

   After further review, The reason for server to fail with sharing
   violation is that the windows server that executes a path-based
   rename request opens the file for DELETE access, but only with
   FILE_SHARED_READ as ShareAccess .  Therefore, the existing
   open(frame 76), which has shared read/write/delete , is compatible
   with the Windows servers access mode (DELETE), but Windows servers
   open is not compatible with access mode in existing open.

   Note that it is correct to state that the logic in Windows server
   could have been written to allow shared read/write/delete in which
   case it would succeed as you mention. The behavior here is
   historical based on the existing implementation.

s4 torture: Close the third handle in RAW-OPLOCK-BATCH22

s4 torture: Convert an error to a warning in RAW-OPLOCK-BATCH22

Some servers choose to mark a client as bad if they fail an oplock
break request by timing out (win7 is an example).  Once the client is
marked as bad, future oplock requests will timeout instantly.  This
causes subsequent runs of this test to fail, so rather than erroring
out as a failure, a warning is printed instead.

There is also a bug in w2k3 where it was incorrectly returning
contending a share mode lock.  It worked in XP and has been re-fixed
in win7.

This can also now be run against samba3.

s4 torture: Add a few more windows target types

s4 torture: Convert to a more modern version of read in RAW-OPLOCK-BATCH4

s4 torture: Allow some implementations to break from level1 to none with two breaks

Make sure we're not using the old smb_fname data.

Jeremy.

Remove smb_fname duplicates that just keep the same information as in fsp->fsp_name.

Moving towards making VFS_OPEN/VFS_MKDIR/VFS_RMDIR
handle based...
Jeremy.

s3-spoolss: handle SEC_FLAG_MAXIMUM_ALLOWED in spoolss_OpenPrinterEx.

In case someone (smbtorture4) requests SEC_FLAG_MAXIMUM_ALLOWED, translate it
to a request of PRINTER_ACCESS_ADMINISTER.

Guenther

s4-smbtorture: check more WERRORs in RPC-SPOOLSS.

Guenther

s4-smbtorture: call test_EnumPrinterDrivers with architecture in RPC-SPOOLSS.

Guenther

Refactor reply_rmdir to use handle based code. All
calls are now handle based. Put rmdir into close.c
and make it private.
Jeremy.

s4:operational LDB module - Fix usage of LDB constants

s4:ntvfs/posix/pvfs_streams - Fix "discard const" warning

I removed one "const" in front of a string declaration to achieve this.

Signed-off-by: Andrew Tridgell <tridge@samba.org>

s4:ntvfs/posix/pvfs_resolve - Fix "discard const" warning

I removed two "const"s in front of string declarations to achieve this.

Signed-off-by: Andrew Tridgell <tridge@samba.org>

s3: Fix crash in winbindd;(bug#6879).

Signed-off-by: Bo Yang <boyang@samba.org>

Remove unneeded argument from can_set_delete_on_close(). Ensure
can_set_delete_on_close() is correctly called before any setting
of the disposition bit (clean up the do_unlink() call).
Jeremy.

s4/selftest: knownfail some locking tests until bugs are fixed

ѕ3: remove superfluous option check

this function is only called when strict alloc is set, no reason to check that twice.

s4/torture: add multiple lock cancel test

See what happens when we have multiple outstanding lock requests and
we try to cancel both of them within a single LockingAndX.

On Windows, it seems only the first lock in the array is cancelled,
and the second is left pending.  Though, this behavior goes against
the MS-CIFS spec.

s4/torture: add addition multiple lock tests

* test that 2 locks in a single LockAndX are transactional
* test that 1 unlock and 1 lock in a single LockAndX are not
  transactional
* test that SMB2 doesn't like mixed lock/unlock in a single
  PDU

s4/torture: fix build warnings by removing unecessary const

s4/torture: Add target functionality parameters to SMBv1 BRL tests

Abstract the server requirements to pass some BRL tests.

* The new default for >64bit lock tests, is that the server should
  return STATUS_INVALID_LOCK_RANGE.
* Add parameter for targets that don't implement DENY_DOS

s3-selftest: run LOCAL-NDR when running make selftest.

Guenther

s4-smbtorture: be very strict in checking spools_EnumPrinterKey results.

Guenther

spoolss: hand-marshall the spoolss_StringArray2 struct for spoolss_EnumPrinterKey.

This call is just driving me nuts :-)

Guenther

s4-drs: fixed UDV and overlapping sync calls in DRS

When windows abandons a DRS sync, it will sometimes re-use the same bind handle for
a new sync. This means we need to check the DN of the sync and blank the getnc_state
if the DN has changed.

This also fixes the UDV to use the highest uSN for the partition, not for
the whole SAM.

s4-drs: fixed updating of uSNChanged in replmd_modify

Updating of uSNChanged broke in a recent change

s3:build: remove redundant qnx block size definition

Restructure the ACL code some more, get the internal semantics
right. The previous bugs were due to the fact that get_nt_acl_internal()
could return an NTSTATUS error if there was no stored ACL blob, but
otherwise would return the underlying ACL from the filysystem. Fix
this so it always returns a valid acl if it can, and if it does not
its an error to be reported back to the client. This then changes
the inherit acl code. Previously we were trying to match Windows
by setting a minimal ACL on a new file that didn't inherit anything
from a parent directory. This is silly - the returned ACL wouldn't
match the underlying UNIX permissions. The current code will correctly
inherit from a parent if a parent has any inheritable ACE entries
that apply to the new object, but will return a mapping from the
underlying UNIX permissions if the parent has no inheritable entries.
This makes much more sense for new files/directories.
Jeremy.

s3:build: fix shared library build on QNX

Fixes #6860. Thanks to Matt Kraai!

Ensure check_parent_acl_common() only looks at stored
blobs - returns NT_STATUS_OK if there aren't any.
Jeremy.

s3: prefer posix_fallocate for doing "strict allocate"

posix_fallocate is more efficient than manual zero'ing the file. When
preallocation in kernel space is supported it's extremely fast. Support for
preallocation at fs layer via posix_fallocate and fallocate at kernel site
can be found in Linux kernel 2.6.23/glibc 2.10 with ext4, XFS and OCFS2. Other
systems that I know of which support fast preallocation in kernel space are
AIX 6.1 with JFS2 and recent Solaris versions with ZFS maybe UFS2, too.

People who have a system with preallocation in kernel space might want to set
"strict allocate = yes". This reduces file fragentation and it's also safer for
setups with quota being turned on.

As of today most systems still don't have preallocation in kernel space, and
that's why "strict allocate = no" will stay the default for now.

Ensure get_nt_acl_internal() only looks at the ACL blobs, not
reads off the underlying filesystem. Ensure that vfs_acl_tdb.c
returns NT_STATUS_NOT_FOUND, not NT_STATUS_OBJECT_NAME_NOT_FOUND
when it can't find a blob matching the file.
Jeremy.

Fix bug #6837 - "Too many open files" when trying to access large number of files
from Windows 7. Original patch by me fixed up with the correct open files number
by jmaggard10@hotmail.com.
Jeremy.

samba-spoolss: use spoolss_StringArray2 in spoolss_EnumPrinterKey.

This should finally resolve the endian issues we were seeing on sparc and is
much cleaner for spoolss clients and servers.

Guenther

spoolss: add spoolss_StringArray2.

The difference to spoolss_StringArray is that in spoolss_StringArray2 the string
array is put into a subcontext of _ndr_size.

Guenther

s3-spoolss: move MAXDEVICENAME to spoolss idl.

Guenther

s3: Cope with older external libtdb

Revert "s3-build: taise tdb version when building against system libtdb library."

This reverts commit ea20678c55fee9f4586630cdb5fe7f35457d309a.

Just one new error code does probably not justify raising the required tdb version.

Guenther

Ensure we don't see the xattr used to store NT security (visible when xattr_tdb
is used). Allows make test to pass with acl_xattr.so prepended to the vfs modules.
Jeremy.

s3: Fix smbd to correctly return INVALID_LEVEL on set_file_end_of_file_info for paths

This allows smbd to pass the freshly updated RAW-SFILEINFO-END-OF-FILE
torture test.

s4 torture: Change oplock to use passthrough for exclusive3/batch11

In light of the INVALID_LEVEL that is seen for RAW_SFILEINFO_END_OF_FILE_INFO
requests on a path, I'm changing these back to using the passthrough
RAW_SFILEINFO_END_OF_FILE_INFORMATION to test the oplock break behavior as
originally intended

s4 torture: Update RAW-SFILEINFO-END-OF-FILE to test some additional corner cases

It turns out setting the end-of-file with Trans2SetPathInfo using the
snia spec's info level will attempt to open the file, enforcing share
modes, but then subsequentlys fail the setpathinfo with a dos error of
INVALID_LEVEL.  Doing a Trans2SetFileInfo with either end-of-file info
level succeeds as expected.

Revert "s4 torture: Allow onefs to be checked like samba3 and samba4"

This reverts commit 98f595036e196dd61340fef0faf63ca762a25307.

No longer necessary

s4 torture: Change RAW-SFILEINFO-END-OF-FILE to check for share modes by default

Since the windows behavior appears to be a bug, only check for
the windows-style share mode bug if target=<windows variant> is
specified

s4 torture: Move target macros to a common header instead of redefining them in multiple files

s3:build: fix detection of CTDB headers on systems without system-libtalloc

Michael

s3:build: fix the build when no external talloc and tdb are installed.

The check for the external libs and the addition of the include paths
to the CPPFLAGS was too late in configure.

This patch moves the whole subsystem/library section up right below
the detection of "BLDSHARED".
And it updates not only SAMBA_CPPFLAGS but also SAMBA_CONFIGURE_CPPFLAGS
so that many tests that use these flags can now succeed.

Michael

s4:torture/raw/lock - Fix "discard const" warnings by introducing "discard_const_p"s

s4:torture/nbt/wins - Add more casts

s4:torture/nbench/nbench - Add a cast before "str_list_make_shell"

s4:torture/ldap/ldap_sort - Add some casts to suppress warnings

s4:schema - Make some more (result) variables const and fix up warnings with "discard_const_p"

util/util_strlist - add a call "const_str_list" for making "unconst" lists "const"

s4:kdc - Merged kdc_tcp_accept() and kpasswdd_tcp_accept().

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

s4:kdc - Merged kdc_add_kdc_socket() and kdc_add_kpasswd_socket().

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

s4 - Create default modules directory.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

s4:kdc - Disable KDC port when it's set to 0.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

Merge commit 'origin/master' into abartlet-devel

On our way to Samba4 alpha10!

Fix build of Samba4 from tarball generated by mkrelease.sh

Revert "Remove RFC's from the release tarballs to make the lives of the Debian"

This reverts commit eda7f35bc891ca4a7505ec054a2b4591c6edfb38.

These files are essential to the Samba4 build.

Andrew Bartlett

Improve upgrade instructions

s4-ldb: the '1' form of extended_dn search is easier to read

The '1' form gives GUIDs and SIDs in the ascii form as normally used
for display.

This is alpha9

s4:setup Adjust upgradeprovision blackbox test now we don't have --targetdir

s4: Remove targetdir as it can cause some trouble and can be replaced by an adapted -s smb.conf

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

s4: Handle the case in secrets.ldb without name attribute

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

s4: don't forget to update defaultSecurityDescriptor

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

s4: load the domain level of the current provision and create a provision with the same domain level

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

s4:ldap.py Add test of namingContext behaviour after tridge found a bug

Tridge found that the partitions.c module was being initialised twice,
and setting the partitions into the rootDSE twice.

Andrew Bartlett

s4:WHATSNEW Nadezhda's acl module handles modifies

Fix DEBUG 0 -> DEBUG 10 (left over code).
Fix opendir status return if access denied.
Jeremy.

Fix bug 6938 : No hook exists to check creation rights when using acl_xattr module
Fix ACL modules to test for permissions on open/mkdir/opendir.
Ensure that underlying ACLs are returned for directories/files with
no Windows xattr or tdb acls stored.
Jeremy.

s3-build: taise tdb version when building against system libtdb library.

Try to fix the build on "buildsamba02". At least fixes the build on fedora12
with libtdb-devel-1.1.5-2.fc12.x86_64 installed.

Volker, please check.

Guenther

s3:add split_tokens, a cmdline tool to test next_token_talloc()

Michael

s3:fix linking of the t_str* test programs

Michael

s3:fix linking of libbigballofmud (on my box at least)

Michael

s3:torture: make t_stringoverflow.c compile at least (pstring is long gone)

I don't know whether these t_* programs are used at all any more, but this
one was using pstrcpy...

Michael

Restructure the connect function code to always call
down to NEXT-> before initializing. This allows us to
do cleanup (by calling DISCONNECT) if initialization
fails. Also fix vfs_acl_xattr which was failing to
call the NEXT connect function.
Jeremy.

drsuapi: fix build warning of NDR size calculation of drsuapi_DsReplicaObjectIdentifier3Binary.

The generated size actually does not include the size of the binary itself.

Guenther

s3-spoolss: fix fstrings in convert_devicemode() function.

Guenther

Fix bug 6546: Avoid accessing buf[-1] if NUL byte comes from fgets

s3:docs: Fix typo in man mount.cifs.

Fix bug #6844 (wrong credential file format in mount.cifs manpage).
Thanks to the Debian Samba package maintainers for reporting!

Karolin

s3:docs: Document "aio write behind".

Part of a fix for bug #6890 (Some smb.conf parameters are undocumented).

Karolin

s3:docs: Document "ldap page size".

Part of a fix for bug #6890 (Some smb.conf parameters are undocumented).

Karolin

s3: Fix crash in free_file_list().

Signed-off-by: Bo Yang <boyang@samba.org>

s3:docs: Document "enable core files".

Part of a fix for bug #6890 (Some smb.conf parameters are undocumented).

Karolin

s4-drs: Test situations for runtime constructed parentGUID

Includes the following verifications for the constructed parentGUID:
- Checks if it returns nothing when there is no parent object
- Ensures that attributes mentioned after the parentGUID
are returned correctly (this avoid a bug pointed out by Tridge
during sync constructed parentGUID development)

Signed-off-by: Andrew Tridgell <tridge@samba.org>

s4-drs: Using dsdb_msg_add_guid() utility function

Uses the dsdb_msg_add_guid() to add any kind of GUID attribute
to a ldb_message in several places of samba4 code.

Signed-off-by: Andrew Tridgell <tridge@samba.org>

s3:docs: Document "cache directory" and "state directory".

Part of a fix for bug #6890 (Some smb.conf parameters are undocumented).

Karolin

s3: Fix bug 6288

s4-drs: add deletion of old connections

the nTDSConnection objects that are not needed anymore will be deleted.
the function kccsrv_delete_connection wasn't tested yet.

Signed-off-by: Andrew Tridgell <tridge@samba.org>

s4-ldb: changed ldb_msg_add_dn() to ldb_msg_add_linearized_dn()

this makes the usage clearer

Signed-off-by: Andrew Tridgell <tridge@samba.org>