s3-smbd: Call sys_acl_get_perm() directly rather than via the VFS

This will allow us to remove the struct smb_acl_t manipuations from the VFS layer,
which will be reduced to handling the get/set functions.

Andrew Bartlett

s3-smbd: Move smb_acl_t declaration to smb_acl.idl

This will allow us to marshall this into and from an NDR blob on disk, which will
allow us to fake up ACL support during make test, and to test the NT ACL emulation
using python bindings via the VFS.

Andrew Bartlett

pidl: Add mode_t as an alias so we can marshall posix ACL structures

s3-smbd: Change allocation of smb_acl_t to talloc()

The acl element is changed to be a talloc child, and is no longer one element
longer than requested by virtue of the acl[1] base pointer.

This also avoids one of the few remaining cases of over-allocation of a structure.

Andrew Bartlett

libwbclient: Add test for wbcPingDc2

The internal domain used in 'make test' does not report a DC name, so
just add tests similar to the old wbcPingDc call.

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

wbinfo: Improve output of wbinfo --ping-dc

Use wbcPingDc2 to get the DC name and print it.

Cleanup error messages: Remove "Could not ping our DC", there is always
a more specific message.  Avoid printing "failed to call wbcPingDc" in
case the ping has been attempted and it returns an error, the error is
already printed.

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

libwbclient: Add wbcPingDc2

Add wbcPingDc2 that optionally returns the DC that was attempted to
ping. wbcPing is implemented as a wrapper around wbcPingDc2.

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

s3-winbind: Return the DC name from DC_PING

The DC that was attempted to ping is useful for troubleshooting. Return
the DC name in the response to the wbclient.

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

s3-winbind: Pass ping-dc result to client

The client checks for an error code in response.data.auth.nt_status,
make sure the result is stored there.

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

selftest: Add knownfail for samba3.winbind.wbclient.wbcPingDc2

The soon-to-be-added command also fails against the s4 winbind.

Andrew Bartlett

s4:dsdb/repl: fix the usage of 'GC/' prefixed principal names

The "serverReference" attribute is available on the "server" object
not on the "nTDSA" object.

This allows connections to RODCs, as they don't have a
E3514235-4B06-11D1-AB04-00C04FC2DCD2/${NTDSGUID}/${DNSDOMAIN}
principal.

Pair-Programmed-With: Björn Baumbach <bb@sernet.de>

metze

Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Tue Aug 14 18:57:41 CEST 2012 on sn-devel-104

s4:samba-tool/drs: print the dns name of the server belonging to a connection

Pair-Programmed-With: Björn Baumbach <bb@sernet.de>

metze

s4:ntp_signd: fix SEGV if SID cannot be found

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Tue Aug 14 17:16:54 CEST 2012 on sn-devel-104

s3-passdb: Silence scary DEBUG(0) message on first use of secrets.tdb databases

When pdb_samba4 first opens this databse, this message is printed.

Andrew Bartlett

s4-dsdb: Use samdb_dn_is_our_ntdsa()

This uses a GUID based comparison, and avoids re-fetching the
samdb_ntds_settings_dn each time.

Andrew Bartlett

s4-dsdb: Add samdb_dn_is_our_ntdsa()

This is like samdb_reference_dn_is_our_ntdsa but without the attribute de-reference.

Andrew Bartlett

s4-dsdb: Use samdb_reference_dn_is_our_ntdsa()

s4-repl: Use samdb_reference_dn_is_our_ntdsa()

s4-dsdb: Add helper function samdb_reference_dn_is_our_ntdsa()

We often want to know if we own an FSMO role (for example).  This tries to be more
efficient by comparing the GUID, rather than the string DN, as this does not need
to be re-fetched each time.

Andrew Bartlett

s4-dsdb: Use ldb_dn_copy() rather than talloc_reference()

As the normal case (outside provision) uses a copy, this avoids a case
where a caller might modify a global variable accidentily.

As suggested by metze.

Andrew Bartlett

s4-libnet: Prepare libnet_BecomeDC for samdb_reference_dn() returning an extended DN

Remote LDAP servers will not accept an extended DN with other components.

Andrew Bartlett

s4-libnet: Improve debugging of libnet_BecomeDC LDAP errors

s4:dsdb/repl: ldb_errstring() takes a 'struct ldb_context' not 'int'

metze

Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Tue Aug 14 13:58:31 CEST 2012 on sn-devel-104

s4:dsdb/repl: make sure instanceType_e is not changed by a reallocation

Pair-Programmed-With: Björn Baumbach <bb@sernet.de>

metze

s4:dsdb/repl: avoid reallocation of msg->elements

The index into the elements needs to match between
msg->elements and md->ctr.ctr1.array, which means we should
pre-allocate them with the same size.

Pair-Programmed-With: Björn Baumbach <bb@sernet.de>

metze

s4-dsdb: Add mem_ctx argument to samdb_ntds_settings_dn

As this value is calculated new each time, we need to give it a context to live on.

If the value is the forced value during provision, a reference is taken.

This was responsible for the memory leak in the replication process.  In the
example I was given, this DN appeared in memory 13596 times!

Andrew Bartlett

Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Tue Aug 14 10:05:14 CEST 2012 on sn-devel-104

s4-dsdb: Improve memory handling in dsdb_schema_from_ldb_results() by adding a tmp_ctx

s4-dsdb: Improve memory handling in kccsrv_add_connection()

s4-dsdb: Improve memory handling in kccsrv_find_connections() by adding a tmp_ctx

s4-dsdb: Add const

VERSION: Move on to beta6!

Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Tue Aug 14 04:05:06 CEST 2012 on sn-devel-104

VERSION: Mark as the beta6 release

WHATSNEW: prepare for 4.0 beta6

s3-vfs: Put vfs_aixacl_util.c helper functions into a header file

Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Tue Aug 14 02:16:11 CEST 2012 on sn-devel-104

s4:kdc/wdc-samba4.c - fix user logins on specific workstations

The decrement operation has been missing.

Problem found by Mohammad Ebrahim Abravi <lamp.mia@gmail.com>

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

s4-classicupgrade: Tests if sam policies exist before trying to import them.

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

s3-selftest: Add smbclient tarmode test

(With small changes to have test complete by Andrew Bartlett)

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

s3-selftest: Fix copy/paste error in test usage string

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

Fix smbclient/tarmode panic on connecting to Windows 2000 clients.

  'Freed frame ../source3/libsmb/clilist.c:934, expected ../source3/client/clitar.c:821'
  Cause: (strequal(finfo->name,"..") || strequal(finfo->name,"."))
    evaluates to true, do_tar returns without freeing ctx

Signed-off-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Mon Aug 13 23:12:50 CEST 2012 on sn-devel-104

Ensure we update last_access on the winbindd child struct on each request.

s3: skip loading vfs modules for printer connections

Autobuild-User(master): Björn Jacke <bj@sernet.de>
Autobuild-Date(master): Sun Aug 12 23:40:23 CEST 2012 on sn-devel-104

s4-dsdb: Take more care in handling of global schema memory

This reworks dsdb_replicated_objects_commit() to have a proper local tmp_ctx and
to be more careful about what schema is set (only setting a global schema if
the original schema was global).

In particular, the new working_schema is not given a talloc reference
to the old schema.  This ensures that the old schema can go away when
no longer used.

Andrew Bartlett

Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Sat Aug 11 10:31:57 CEST 2012 on sn-devel-104

s4-dsdb: Remove support for per-partition sequence numbers

These sequence numbers were only used for telling if the schema was
changed, and are no longer directly related to the replication USN.

The per-partition replication USN can be obtained from the
@REPLCHANGED record on the per-partition database, and this is done
with an ldb_search().

Andrew Bartlett

s4-dsdb: Use only the replication USN for schema reload.

This way we do not track both the partition seq number and the
replication USN for schema reload purposes.

We only need one indication of actual data change, and the replication
per-partition sequence number is no more expensive to obtain than the
ldb per-partition sequence number.

Andrew Bartlett

s3-nfs4acls: Remove lookup_sid and sidmap from NFSv4 ACL mapping and check gid first

By checking just the IDMAP, and by removing the sidmap and lookup_sid calls, we support
IDMAP_BOTH.  This is because by checking for a mapping to a GID first, we can rely on
the fact that IDMAP_BOTH will resolve to a GID.

If the sidmap idea is valued - it allows multiple SIDs to map to a single unix ID, this should
be done in the IDMAP layer.

Andrew Bartlett

Signed-off-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Sat Aug 11 01:17:36 CEST 2012 on sn-devel-104

s3-smbd: Merge ACE entries based on mapped UID/GID not SID

As the test for a valid posix ACL is based on the unix uid/gid only appearing once in the ACL
the merge process also needs to be UID/GID based.

This is a problem when we have multiple builtin groups mapped to the same POSIX group
as happens in a Samba4 provision.

Andrew Bartlett

Signed-off-by: Jeremy Allison <jra@samba.org>

s3-smbd: Convert posix_acls.c to use struct unixid internally

This is consistent with the rest of Samba which uses this structure to represent
a unix uid or gid.

World values remain represented by the owner_type being WORLD_ACE in the containing
structure.  A -1 value is filled in to the unixid.id in the same way the .world value
was initialised in the union.

Andrew Bartlett

Signed-off-by: Jeremy Allison <jra@samba.org>

s3-smbd: Create a shortcut for building the token of a user by SID for posix_acls

When a user owns a file, but does not have specific permissions on that file, we need to
make up the user permissions.  This change ensures that the first thing that we do
is to look up the SID, and confirm it is a user.  Then, we avoid the getpwnam()
and directly create the token via the SID.

Andrew Bartlett

Signed-off-by: Jeremy Allison <jra@samba.org>

torture: Reproducer for 64c0367

Signed-off-by: Jeremy Allison <jra@samba.org>

tdb/test: fix build on OSF/1

Autobuild-User(master): Björn Jacke <bj@sernet.de>
Autobuild-Date(master): Fri Aug 10 23:33:20 CEST 2012 on sn-devel-104

Revert "s3:smbd: include smbXsrv.h before smbd/proto.h to have the smbXsrv_ structs available"

This reverts commit 98ccca8dca70b87d04a93c8ef5232a071ab7c2af.

Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Fri Aug 10 17:35:38 CEST 2012 on sn-devel-104

Revert "s3:smbd: Include smbXsrv.h before vfs.h (in smbd.h) so that the smbXsrv structures are available"

This reverts commit e332bfaff51e54638bd37cd1fe08e57608e16e86.

Revert "s3:smb: include "smbXsrv.h" before "vfs.h""

This reverts commit db0c233624e633b3cc1a6e0e44dccc09aaa121f2.

s3:smbXsrv.idl: remove smbXsrv_*0 defines

This makes ctags more usable.

metze

s3:param: fix compiler warnings with FN_GLOBAL_CONST_STRING()

metze

build: rename security → samba-security

there is a libsecurity on OSF1 which clasheѕ with our security lib. see bug #9023.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User(master): Björn Jacke <bj@sernet.de>
Autobuild-Date(master): Fri Aug 10 14:22:21 CEST 2012 on sn-devel-104

nsswitch: add ABI checking and symbol versions to libwbclient

This will ensure that we do not unintentionally break the ABI.

Andrew Bartlett

Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Fri Aug 10 04:08:54 CEST 2012 on sn-devel-104

s4-dsdb: Explain better what records are written during schema set

This is controlled by setting write_indices_and_attributes.

Andrew Bartlett

lib/ldb: Use tdb_parse_record and a callback rather than tdb_fetch()

This avoid allocation at the tdb layer as we will allocate this
with talloc right away anyway.

Andrew Bartlett

Remove smb_panic() from unix_strlower(). Just rely on error code return.

Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu Aug  9 23:52:53 CEST 2012 on sn-devel-104

Correctly check for errors in strlower_m() returns.

Fix strlower_m() to return an error indication.

Check error returns on strnorm().

Check error returns from strupper_m() (in all reasonable places).

Fix missing ads_destroy in error path.

Change strupper_m() to return a value.

Fix bad return in unix_strupper.

Prepare to remove smb_panic() from unix_strlower().

Fix bad return values in unix_strlower/unix_strupper.

s4:torture:basic: check the return status of the last open in deltest16

Autobuild-User(master): Michael Adam <obnox@samba.org>
Autobuild-Date(master): Thu Aug  9 18:01:50 CEST 2012 on sn-devel-104

s4:torture:basic: fix a message typo in the delete17 test

s4:torture:basic: fix abundance of spaces in deltest6

s4:torture:basic:delete: fix 4 vs 8 spc tab formatting in check_delete_on_close()

s3:torture:delete: add a 12th subtest to the delete-on-close tests

test whether second open is possible with initial delete on close
and how setting and unsetting delete on close on the handle affects
the initial delete on close (it does not...)

s3:torture:delete: fix 11th test to work against windows

s3:torture:delete: simplify return code handling, fixing a couple of return codes in error cases

s3:torture:delete: reduce indentation

s3:torture:delete: add a comment

s3:torture:delete: add a comment

s3:torture:delete: add a comment

s3:torture:delete: move the success message for a subtest to the correct place

s3:torture:delete: remove an else, reducing indentation

s3:torture:delete: remove an else, reducing indentation

s3:torture:delete: remove an else, reducing indentation

s3:torture:delete: remove an else, reducing indentation

s3:torture:delete: really fail the test in a failure case

s3:torture:delete: fix a comment

s3:torture:delete: fix a message

s3:torture:delete: fix a message

s3:torture:delete: fix a message

s3:torture:delete: fix a message (counting the opens)

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untanlge function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check