Merge branch 'v3-3-test' of ssh://jra@git.samba.org/data/git/samba into v3-3-test

Fix a segfault if ? is there but the options are NULL. This is the case if SMBC_parse_path is called by SMBC_stat_ctx.

"First thing, kill all the language lawyers" :-). Ensure possible insane compilers
can't kill us later.
Jeremy.

ads_connect: Return immediately on a failed GC connection.

ads_connect_gc() feeds an explicit server to ads_connect().  However, if the
resulting connection fails, the latter function was attempting to find a DC
on its own and continuing the connection.  This resulting in GC searches being
sent over a connection using port 389 which would fail when using the base
search suffix outside of the domain naming context.

The fix is to fail immediately in ads_connect() since the GC lookup ordering
is handled already in ads_connect_gc().

(was commit 073e9f42f0c5f5de5d736ec7843d80a274c891ce in master)

Fix bug #6040 -  Missing <th> header in Status page.
Based on a patch from pkg-samba-maint@lists.alioth.debian.org.
Jeremy.

Merge branch 'v3-3-test' of ssh://jra@git.samba.org/data/git/samba into v3-3-test

Allow reinit_after_fork to be called safely from within swat and other binaries that don't have
an event context or a msg context. Fixes crash bug in swat.
Jeremy.

packaging(RHEL-CTDB): export CC - really use ccache...

metze

Signed-off-by: Michael Adam <obnox@samba.org>

Fix bug #6040 - Calling Samba print server with an aliased DNS-name fails.
Jeremy.

Fix bug #6035 - Possible race between fcntl F_SETLKW and alarm delivery.
Jeremy.

Fix bug in get_dc_name_via_netlogon(), null pointer refrence.

Don't send message to any other child in child process.

From boyang - ensure we never "return" from a forked child, always _exit().
Jeremy.

Fix bug #6019 File corruption in Clustered SMB/NFS environment managed via CTDB
Jeremy.

Deprecate the "share modes" parameter to address bug #6024, swat disagrees with smbstatus as to share mode with share modes = No set in samba.
Jeremy.

s3:libsmb: handle the smb signing states the same in the krb5 and ntlmssp cases

SMB signing works the same regardless of the used auth mech.

We need to start with the temp signing ("BSRSPYL ")
and the session setup response with NT_STATUS_OK
is the first signed packet.

Now we set the krb5 session key if we got the NT_STATUS_OK
from the server and then recheck the packet.

All this is needed to make the fallback from krb5 to
ntlmssp possible. This commit also resets the cli->vuid
value to 0, if the krb5 auth didn't succeed. Otherwise
the server handles NTLMSSP packets as krb5 packets.

The restructuring of the SMB signing code is needed to
make sure the krb5 code only starts the signing engine
on success. Otherwise the NTLMSSP fallback could not initialize
the signing engine (again).

metze
(cherry picked from commit 7d9fd64f38aa5821b38c1223cf87979fc87bfb71)

Even for srclen == 0 we have to return something

This fixes a regression reported by Corinna Vinschen <corinna@vinschen.de>

Thanks,

Volker

Don't set child->requests to NULL in parent after fork

s3/smbpasswd: Check if Unix account exists before asking for the password.

Admins shouldn't have to type in the password twice when the passdb account
cannot be created because the Unix account is missing.

Karolin

docs: Improve man mount.cifs.

Add hint about specifying the workgroup in the credentials file.
Whitespace cleanup.

This fixes bug #4541.
Patch based on proposed patch from Christian Perrier <bubulle@debian.org>.
Thanks for reporting and providing a patch!

Karolin
(cherry picked from commit 1f7e09ea542df3a2f5f553c0cb11a39c74712950)

s3: Remove a few unnecessary checks from the streams depot module and fix to work with NTRENAME

Handling of error codes when renaming a file to a stream and a stream
to a file is now done in rename_internals_fsp.

The NTRENAME stream path only passes in the stream name, so the new
base can now be different from the old base.

s3: Remove a few unnecessary checks from the streams xattr module

Handling of error codes when renaming a file to a stream and a stream
to a file is now done in rename_internals_fsp.

The NTRENAME stream path only passes in the stream name, so the new
base can now be different from the old base.

s3: Allow renames of streams via NTRENAME and fix stream error codes on rename

The test_streams_rename2 test in RAW-STREAMS verifies these changes

Fix bug #6021 - smbclient du command does not recuse properly
Jeremy.

Fix race condition in alarm lock processing noticed by Richard Sharpe <realrichardsharpe@gmail.com>.
"It seems to me that if the lock is already held by another process when we
enter this code, there is a race between the timeout and the granting. If
the lock is subsequently granted, the process releasing the lock will signal
the wait variable (or whatever) and our process will be scheduled. However,
if the timeout occurs before we are scheduled, the timeout will be delivered
first.

We will have the lock but will forget we have the lock, and never release
it."
Jeremy.

Fix bug #6016 - Alternate Data Streams / Extended Attributes seem to conflict.
Jeremy.

refresh sequence number as soon as possible when domain->sequence_number == -1 or domain->last_status is not ok.

Make winbindd_cm.c use winbindd_reinit_after_fork().
Jeremy.

Fix bug #6017, - magic script does not work. based on a patch from monyo@samba.gr.jp.
Jeremy.

Add winbindd_reinit_after_fork(), cleaning out all possible events
in a forked child.
Jeremy.

Factor out lots of common code into a function.
Jeremy.

s3:events: change event_add_timed() prototype to match samba4

metze

s3/swat: Fix creation of the first share using SWAT.

This fixes bug #5965.

docs: Improve description of the share commands in man smb.conf.

-Correct the parameter names.
-Fix typos.
-Fix related parameters.

This fixes bug #6008.
Thanks to TAKAHASHI Motonobu <monyo@samba.gr.jp> for reporting!

Karolin
(cherry picked from commit 8b7f66c548ed170d2dab0c91ccff4aca00f4b52d)

docs: Fix example in man vfs_recycle.

This fixes bug #6001.
Thanks to TAKAHASHI Motonobu <monyo@samba.gr.jp> for reporting!

Karolin
(cherry picked from commit 9f44a17c3bf36067d04cec036c3e529798932cd7)

set entry->refresh_time to make ccache_regain_all_now() work correctly.

s3:events: get rid of the stupid set_event_dispatch_time() function

metze

s3:events: get rid of the stupid cancel_named_event() function

metze

s3:events: move prototypes to event.h

metze

s3:events: remove unused stuff

metze

s3:winbindd: regain tickets for all ccache entries, when we go online

set_event_dispatch_time() is stupid by design and only handles
the first event with a given name.

metze

s3:winbindd: cancel all ccache entry events and not just one

cancel_named_event() is stupid by design and also only cancels
one single event.

metze

s3:winbindd: recreate the per domain check_online_event without relying on global state

set_event_dispatch_time() is stupid by design and just picks the first
event with the given name.

metze

s3:winbindd: rename fd_event => winbindd_fd_event

It's really confusing to have two versions of 'fd_event'

metze

s3:winbindd: move WINBINDD_CCACHE_ENTRY and WINBINDD_MEMORY_CREDS to winbindd.h

metze

Fix broken krb5 refresh chain

Signed-off-by: Stefan Metzmacher <metze@samba.org>

clean event context after child is forked.

Signed-off-by: Stefan Metzmacher <metze@samba.org>

Attempt to fix crash seen with new CUPS async printcap loading code.
Karolin please pull for 3.3.0.
Jeremy.

examples: Avoid bashism in perfcount.init.

This fixes bug #6000.
Thanks to the Debian Samba package maintainers for providing the patch!
Remove trailing whitespaces.

Karolin
(cherry picked from commit 2e09746a4db4186c9d648370b9004971bc18e5c9)

Revert "examples: Avoid bashism in perfcount.init."

This reverts commit e273c07e94ee607bbf05b6fa66cf3cea13fe4502.

WHATSNEW: Update changes since 3.3.0rc2.

Karolin

Fix bug 6014 -- mget shouldn't segfault without arguments

Thanks to kevin.paulus@mtm.kuleuven.be for the bug report

Volker

examples: Avoid bashism in perfcount.init.

This fixes bug #6000.
Thanks to the Debian Samba package maintainers for providing the patch!
Remove trailing whitespaces.

Karolin
(cherry picked from commit 73875cd344608b591fa884ab99b5f3a10550c149)

docs: Document the -g option of smbclient.

This fixes bug #6013.
Thanks to the Debian Samba packages maintainers for reporting!

Karolin
(cherry picked from commit 6752d78e946b2c4278e2deba325c76fb7ffbc06a)

Fix a bad memleak in vfs_full_audit

Fix bug 6012: Add "get_real_filename" to full_audit

Thanks to Hodur <coil93@gmail.com> for testing!

Volker

smbd: prevent access to root filesystem when connecting with empty service name

This only applies to a setup with "registry shares = yes"

Michael
(cherry picked from commit 288fa94ac7cfdf7457b5098c33fc840bed3d5410)
(cherry picked from commit e85526b184e66f86b7faa9d0a37288a09c12c19e)

Fix bug #6009 - Setting "min receivefile size = 1" breaks writes.
Karolin please pull for 3.3.0. Thanks !
Jeremy.

Alternative fix for 45db33e73 and 0d443ae7931

Simo is right, we need to ask passdb first. At least this fixes a nasty to find
NT_STATUS_ACCESS_DENIED problem in the build farm for the test run I just did
on host "opi".

Michael, can you re-check if this also fixes the error you found, leading to
the two fixes?

Thanks,

Volker

Revert "Fix uid_to_sid mapping when the idmap cache is empty."

This reverts commit d4bc1f8d79929dc1a1d06e92eaf85923a1848e6f.

Revert "idmap: Remove passd check from idmap_backends_unixid_to_sid()."

This reverts commit 3d22b77f1ac5e8f83d1063d87c68e60655a3ed6b.

packaging(RHEL): fix typo in [u]mount.cifs installation

Michael
(cherry picked from commit 835108bcf84a5f47f46d237c7a93c572348125a1)

Signed-off-by: Michael Adam <obnox@samba.org>
(cherry picked from commit 71c74725bce403e36e4dc38287802c95b8d4de66)

packaging(RHEL): fix build of [u]mount.cifs

mount.cifs is now built by make everything.
And it needs extra objects anyways (mtab.o),
which is why the build of mount.cifs failed.

Michael
(cherry picked from commit 96c811cf222be5463a6705feae73d6215685c5f1)

Signed-off-by: Michael Adam <obnox@samba.org>
(cherry picked from commit 308dbca653ce1d689b75f9b781237c67df4233a8)

WHATSNEW: Update changes since 3.3.0rc2.

Karolin

Happy New Year!

metze

Fix bug 5913.

Never seen this to be a problem, but it doesn't hurt either :-)

Fix warnings in make test code.
Jeremy.

Fix all warnings with gcc4.3.
Jeremy.

Fix more asprintf and "ignoring return code" warnings from gcc 4.3.
Jeremy.

recvfile can't be used for printing so far

s3:winbindd: we don't need to call messaging_reinit() twice

reinit_after_fork() already calls messaging_reinit()

metze
(cherry picked from commit 429276556a4930fa1bc782904447120950073190)

Fix an ancient uninitialized variable read

The callers of open_file_ntcreate expect *psbuf to be filled correctly

Fix more "ignore return value" warnings from gcc 4.3.
Jeremy

s3:cluster: register the client connection via CTDB_CONTROL_TCP_ADD

This means we correctly pass IPv6 addresses too.
(CTDB_CONTROL_TCP_CLIENT was IPv4 only)

metze
(cherry picked from commit 465176b7e2fc1370dded850a80a8fe8519e79db8)

s3:pdb_tdb: store the next rid counter in passdb.tdb instead of winbind_idmap.tdb

This upgrades the TDBSAM_VERSION to 4 and SAMU_BUFFER_V4.

metze
(cherry picked from commit 7516ba860c160e8b971b6f5bc1036370169e6e8a)

s3:passdb: add SAMU_BUFFER_V4 with no changes

The upgrade is required because of the followin TDBSAM_VERSION
upgrade.

metze
(cherry picked from commit 671469ae7e3013809fe3995ec8f33dda9adda2ed)

s3:passdb: make marshalling struct samu from and to a buffer more generic

This marshalling isn't specific to tdbsam and it's
ugly to have the related functions in two different files.

metze
(cherry picked from commit 0544734309feea44e8d449737a9934231f63693d)

Fix a memory leak in cups_pull_comment_location

We allocated "request" with ippNew, so we also should ippDelete it.

Fix a spinning smbd when printing

Without this, we end up adding more than one timed event. In the event handler
print_notify_event_send_messages() only one event will be deleted, all others
will fire indefinitely.

Don't increment twice

Thanks metze :-)

Second part of the bugfix for #5933

Incrementing the next vuid did not correctly overflow

Now we survive BENCH-SESSSETUP with -o 100000. Takes a while though :-)

Thanks a lot to Ofer Tal <otsmb@shmoop.org> for reporting #5933

First part of bugfix for #5933

Ofer Tal <otsmb@shmoop.org> fully correctly noted that we're incrementing
num_validated_vuids twice per session setup, but decrement it only once.
Looking at sesssetup.c we always call register_initial_vuid() before
register_existing_vuid(), so there's no point in incrementing it in
register_existing_vuid().

Jeremy, please check!

s3:dbwrap: fix dbwrap_store_uint32() to match dbwrap_store_int32()

All callers expect 0 an success and -1 on error.

metze
(cherry picked from commit a255f9ce713b29552722ec73e4038226959e1c8e)

Add code to test write_data_iov a bit

Attempt to fix bug 5953: Make cli_send_smb_direct_writeX use writev

It seems there are SMB servers around which can't cope with the write header
being sent in a packet of its own. With writev we keep the advantage of direct
writes, giving the kernel the chance to coalesce the write calls.

Make write_data use write_data_iov

Add write_data_iov

Add sys_writev

Fix some nonempty blank lines
(cherry picked from commit 010c7101e59477f0d5f3bf11c17f474ec6f79cc1)

Signed-off-by: Michael Adam <obnox@samba.org>

Fix some C++ warnings
(cherry picked from commit dd9e4e6db04acf20f6ef7705955358c7ca442bbd)

Signed-off-by: Michael Adam <obnox@samba.org>

Fix bug #5990 - strict allocate should be checked before ftruncate
reported by and based on a patch by Yasuma Takeda <yasuma@osstech.co.jp>.
Jeremy.

idmap: Remove passd check from idmap_backends_unixid_to_sid().

As noted by Metzy, it makes no sense here to check id->sid.
What is worse, this might even be passed in uninitialized.

This still fixes the bug for me (of course), but we might need
to check, if another special handling of passdb is needed
(possibly changing from constant return code NT_STATUS_OK...)

Michael
(cherry picked from commit 651b5d6deaab4fb3cb726fafd98171c79777631c)

Signed-off-by: Michael Adam <obnox@samba.org>

Fix uid_to_sid mapping when the idmap cache is empty.

This failed for backends other than passed, since
idmap_backends_unixid_to_sid() always asked passdb first,
which returned Success no matter whether a mapping was
found or not.

One effect wast that getpwuid failed after "net cache flush".
Only after filling the cache with a getpwnam call it succeeded.

This fix makes the behaviour of idmap_backends_unixid_to_sid()
exactly the same as that of idmap_backends_sid_to_unixid()

Michael
(cherry picked from commit da7eb5e22e56ea718e393375085cf66a90afafc8)

Signed-off-by: Michael Adam <obnox@samba.org>

Fix more asprintf warnings and some error path errors.
Jeremy.

More asprintf warning fixes.
Jeremy.

More asprintf warning fixes.
Jeremy.

Fix more asprintf errors and error code paths.
Jeremy.

Fix more ‘asprintf’, declared with attribute warn_unused_result.
Jeremy.

Fix more "warning: ignoring return value of ‘asprintf’, declared with
attribute warn_unused_result"
Jeremy.

Fix another "format not a string literal and no format arguments" warning.
Jeremy