Merge branch 'master' of ssh://jht@git.samba.org/data/git/samba

[SAMBA 4 / NETLOGON] Modify type of SAM contexts

In the SAMBA 4 DCE/RPC NETLOGON server the SAM context references have generally
the type "void *". But we know that those context objects are based on the "struct
ldb_context" type. We've always to cast for using a SAM/LDB call.
This I didn't find very appealing and so I assigned the right (detailed) type to each "sam_ctx".
Therefore, the casts could disappear.

Also this change is only cosmetic.

frstrans.idl: add new DFS-R FrsTransport interface with dummy functions

metze

frsrpc.idl: add definition for frsrpc_FrsStartPromotionParent()

metze

s3: Change unix_convert to use an smb_filename struct internally

This allows SMB_VFS_[L]STAT to be called directly.  Additionally, I
changed NTSTATUS result to be named status for consistency.

I also removed the stat_cache_add() from build_stream_path() because
stat_cache_lookup() is never actually called on a file with a stream.
There is no reason why the stat cache couldn't be consulted for
streams in the future.

Jeremy/Volker, please take a look at this one when you get a chance.

s3: Convert some callers of vfs_lstat_smb_fname to SMB_VFS_LSTAT()

s3: Convert some callers of vfs_stat_smb_fname to SMB_VFS_STAT()

librpc: fix the merged build of ndr_frsrpc.c

metze

Revert "net: Use samba default command line arguments."

This reverts commit fb262f79fab00374023e59476e8d05a1015a7041
and related commits c36031778e1983ddb11d3e1fcab35e738dbf94bc
72fd5fa6bb78a054fad5e5ebe19a0c0387a7d45b and
38cd0e086f50ce54d88a19aa5a6803469af90489

This change caused more trouble than it solved. We need to do this differently.
Reverting so we don't accidently release this.

s4:provision Fix provision on FreeBSD

We were missing the 'cn' attribute, which we then prepare a sorted
list based on.  On Linux, strcmp(NULL, NULL) does not segfault, where
it does on FreeBSD.

Reported by Timur I. Bakeyev <timur@com.bat.ru>

Andrew Bartlett

s3: plumb smb_filename through some of the trans2 posix_* functions

s3: Remove unnecessary callers of get_full_smb_filename

This often times means explicitly denying certain operations on a stream
as they are not supported or don't make sense at a particular level.  At
some point in the future these can be enabled, but for now it's better to
remove ambiguity

s3: Remove the now unused fname parameter from filename_convert()

s3: Fix RENAME_FLAG_RENAME path to stop calling unix_convert twice

s3: Remove unnecessary fname argument from callers of filename_convert

s3: Plumb smb_filename through map_open_params_to_ntcreate

frsrpc.idl: make the chunk array in frsrpc_CommPktChunkCtr dynamic

We add an extra num_chunks to the frsrpc_CommPktChunkCtr structure
and use hand modified ndr_push/pull functions to let it not appear
on the wire.

metze

frsrpc.idl: almost complete the idl for the frsrpc_FrsSendCommPkt() function

TODO: The amount of chunks is dynamic, we need to fix that

metze

frsapi.idl: fill the frsapi_WriterCommand() function

metze

frsapi.idl: fix some unknown field names

metze

do not log chdir with level 0 if reason is access denied

this changes the level of logs caused by users trying to access shares
or subdirectories for which they do not have access to in the ACL

this can fill up the samba log even with log level 0 and is more an
expected kind of logs that IMHO should not be logged with such a high
level.

All other errors while chdir() will still be logged with level 0

Signed-off-by: Christian Ambach <christian.ambach@de.ibm.com>

s3:dbwrap: use the transaction wrapper in dbwrap_trans_store().

Now dbwrap_util.c contains only one call to each of
transaction_start, transaction_commit and transaction_cancel.

Michael

s3:dbwrap: use the transaction wrapper in dbwrap_trans_delete().

Michael

tdb: fix locking error

54a51839ea65aa788b18fce8de0ae4f9ba63e4e7 "Make tdb transaction lock
recursive (samba version)" was broken: I "cleaned it up" and prevented
it from ever unlocking.

To see the problem:
$ bin/tdbtorture -s 1248142523
tdb_brlock failed (fd=3) at offset 8 rw_type=1 lck_type=14 len=1
tdb_transaction_lock: failed to get transaction lock
tdb_transaction_start failed: Resource deadlock avoided

My testcase relied on the *count* being correct, which it was.  Fixing that
now.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Michael Adam <obnox@samba.org>

s3: Add some asserts to the filename util functions

In the smb_filename struct stream_name must equal NULL if there
is no stream name.  These asserts should catch any future offenders
of this invariant early.

s3: Plumb smb_filename through open_fake_file

s3: Separate out a new file: filename_utils.c

This is to ease the linking pain of everything that links LOCKING_OBJ

s3: Move is_ntfs_stream*() to filename.c

s3: Finish plumbing the fsp->fsp_name smb_fname conversion through the modules.

s3: Change fsp->fsp_name to be an smb_filename struct!

s3 onefs oplocks: Replace static fstring with talloc'd dbg_ctx()

s3: Change file_structs to be allocated with talloc instead of malloc

lib util: Fix const warning

Make cli_send_mailslot() static. Preparing to do away with unexpected.tdb....
Jeremy.

Make tdb transaction lock recursive (samba version)

This patch replaces 6ed27edbcd3ba1893636a8072c8d7a621437daf7 and
1a416ff13ca7786f2e8d24c66addf00883e9cb12, which fixed the bug where traversals
inside transactions would release the transaction lock early.

This solution is more general, and solves the more minor symptom that nested
traversals would also release the transaction lock early.  (It was also suggestd in
Volker's comment in 6ed27ed).

This patch also applies to ctdb, if the traverse.c part is removed (ctdb's tdb
code never received the previous two fixes).

Tested using the testsuite from ccan (adapted to the samba code).  Thanks to
Michael Adam for feedback.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Michael Adam <obnox@samba.org>

Re-add accidently removed shares test.

s4:kdc Add in a simple check for constrained delegation to self

To do this properly, we must use the PAC, but for now this is enough
to check that we are delegating to another name on the same host
(which must be safe).

(Windows 7 does this a lot, also noted in bug 6273)

Andrew Bartlett

[SAMBA 4 directory] Refactoring and clean up of directory structure

- Adds more system objects which make sense to have them in SAMBA 4 also to
  have them when we add more and more services related to the directory (volume
  support, DFS, replication service, COM...)
- Make sure that "isCriticalSystemObject" and "showInAdvancedViewOnly" attributes
  are set correctly on each object

pidl: Print actual type names in errors about unknown origins rather than 'HASH(0x....)'.

pidl: Print proper errors when arrays don't have a specified size rather than spewing pages of perl warnings.

Remove unused parameter setting.

Remove unnecessary imports.

Merge branch 'master' of git://git.samba.org/samba

Include ntsvc NDR in libndr-standard.

Remove unused variable.

source4/lib/registry/patchfile.c(reg_diff_load): fixed possible resource leak.

File descriptor leaks when write(2) fails and we are returning from
function.

Found by cppcheck:
[./source4/lib/registry/patchfile.c:319]: (error) Resource leak: fd

lib/util/util_file.c(file_save): fixed file descriptor leak when read(2) fails.

Found by cppcheck:
[./lib/util/util_file.c:383]: (error) Resource leak: fd

Fix broken password quality check

This fixes broken password tests when the passwords contain non ASCII characters
(e.g. accentuated chars like ('e, `e, ...)

Fix typo

[SAMBA 4] Some cosmetic changes for the LDB modules

Some corrections which make the code a bit more readable (no functional changes here)

include unix.h if it's available

Cosmetic correction

Changes the order of two commands. First set up the "priv" structure, then
assign it to the "ntvfs" structure.

Merge more ndr interfaces into libndr-standard.

librpc: Integrate some subsystems now part of libndr-standard and integrate ndr_dfs in libndr-standard.

Add missing includes.

provision: Remove unused imports.

Create libndr-standard, which includes the NDR marshalling code for
the "standard" set of DCE/RPC interfaces found on Windows.

Remove unnecessary include, update README.

Remove outdated status file about samba3 -> samba4 upgrade project.

Fix a few type errors in VFS modules

VFS examples: Update copy of config.{guess,sub}.

python: Set right ldb modules directory when using system ldb.

Actually fill in ldb modules directory.

Remove pyldb_util and simply duplicate the 5-line function it contains,
rather than creating a separate shared library for it.

ldb: Display SHLD_FLAGS when building.

Fix Coverity CIDs 887, 888. Don't pass NULL's to functions
that deref them.
Jeremy.

Fix Coverity CID 928. Removed check as this can never be null.
Jeremy.

Fix Coverity CID 929. Potential NULL deref after code refactoring.
Jeremy.

Fix coverity CID 930. Pointer check can never be null here.
Jeremy

Fix coverity CID 931. This check is redundent as smb_fname can
never be null in this function.
Jeremy.

Fix a typo reading uninitialized memory. Caught by valgrind.
Jeremy.

Move the initialization of smbd_server_conn from smbd/process,
after the accept and fork, to smbd_init_globals(), so it's
done immediately on server startup. This is needed as some
messages are sent to all active smbd processes (including
the master listening daemon). If it gets a message that
forces it to scan it's current connections (ie. conn_find())
then it discovers that sconn->smb1.tcons.Connections dereferences
null (as sconn == NULL in the parent) and crashes. Yes,
I could fix all cases where sconn is used and explicitly
check for NULL but this fix is easier. It means that
the smbd_event_context() is initialized in the master
daemon and then re-initialized after fork, but that
should be being done correctly in every fork call anyway.
Without this change the previous fix 6a9e0039100b57f9626e87defec6720c476b9789
still panics in the reproducible test case for bug
6564, as this is one case where such a message
(MSG_SMB_CONF_UPDATED) is sent to the parent. Metze
please check. This change passes valgrind.
Jeremy.

Fix bug #6564 - SetPrinter fails (panics) as non root.
Missing become_root()/unbecome_root() around reload_services.
Jeremy.

3.4 is released...

s4-spoolss: fix the build after OsVersion changes.

Guenther

spoolss: fill in some unknowns in spoolss_OSVersion and spoolss_OSVersionEx.

Guenther

s3: add failure check.

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

s3-rpcclient: add createprinteric command.

Guenther

spoolss: fill in spoolss_Create/DeletePrinterIC + rerun make samba3-idl.

Guenther

s3-lsa: let _lsa_Delete return NT_STATUS_NOT_SUPPORTED as w2k3 does.

Guenther

s3-lsa: Fix access_mask calculation for new handle in _lsa_CreateAccount().

Guenther

s3-lsa: add (not yet activate) level specific access checks for _lsa_QueryInfoPolicy.

Guenther

s3-lsa: also implement level 13 in lsa_QueryInfoPolicy.

Guenther

s3-lsa: Fix policy handle memleak and handle type check in _lsa_DeleteObject().

Guenther

s3-rpcclient: add LSA storeprivatedata command.

Guenther

s3-rpcclient: add LSA retrieveprivatedata command.

Guenther

s3-rpcclient: add LSA setsecret command.

Guenther

s3-rpcclient: add LSA querysecret command.

Guenther

s3-rpcclient: add LSA deletesecret command.

Guenther

s3-rpcclient: add LSA createsecret command.

Guenther

lsa: fill in lsa_StorePrivateData and lsa_RetrievePrivateData and rerun make samba3-idl.

Guenther

s3: compile warning and upn handling

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

s3: don't do this, upper callbacks will check it

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

s3: Fix crsh in net usershare list

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

More conversions of NULL -> talloc_autofree_context()
so we at least know when we're using a long-lived context.
Jeremy.

Replace more long-lived contexts with talloc_autofree_context().
Jeremy.

Replace short-lived NULL talloc contexts with talloc_tos().
Jeremy.

Remove an unused talloc context.
Jeremy.

Revert "s4:heimdal_build: predefine GSSAPI_DEPRECATED depending on the compiler version"

This is now handled correctly in the newly imported Heimdal

This reverts commit 4a754d029b0eb229b23980aa4a80dae2b485a302.

s4:heimdal: import lorikeet-heimdal-200907162216 (commit d09910d6803aad96b52ee626327ee55b14ea0de8)

This includes in particular changes to the KDC to resolve bug 6272,
originally by Matthieu Patou <mat+Informatique.Samba@matws.net>.  We
need to sort the AuthorizationData elements to put the PAC first, or
else WinXP breaks when browsed from Win2k8.

Andrew Bartlett