I found lots of places where we assume error will be set when calling
one of our virtualised functions, such as db_open(), but error is only
set when a system call fails, and it is not uncommon for us to fail a
function internally without ever making a system call. That led to us
passing back success when a function had in fact failed.

I found two places where we relied on map_nt_error_from_unix()
returning success when errno==0, but lots and lots of places where we
relied on the reverse, so I fixed those two places.

map_nt_error_from_unix() will now always return an error, returning
NT_STATUS_UNSUCCESSFUL if errno is 0
(cherry picked from commit 69d40ca4c1af925d4b0e59ddc69ef8c26e6501d1)

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

Fix unix_convert() for "*" after changing map_nt_error_from_unix().

map_nt_error_from_unix() now assumes that it is called in
an error path and returns an error even for a given errno == 0.
The original behaviour of unix_convert() used the mapping
of errno == 0 ==> NT_STATUS_OK to return success through
an error path.

I think this must have been an oversight, and unix_convert() worked
only by coincidence (or because explicitly using the knowledge
of the conceptually wrong working of map_nt_error_from_unix().

This patch puts this straight by not interpreting errno == 0
as an error condition and proceeding in that case.

Jeremy - please check!

Michael

Make sure to always set errno on error path in OpenDir (and hence scan_directory).

Michael

Merge branch 'v3-2-test' of ssh://git.samba.org/data/git/samba into my32-test

print correct test status

Fix bug 5686 - libsmbclient segfaults with more than one SMBCCTX.
Here is a patch to allow many subsystems to be re-initialized. The only
functional change I made was to remove the null context tracking, as the memory
allocated here is designed to be left for the complete lifetime of the program.
Freeing this early (when all smb contexts are destroyed) could crash other
users of talloc.
Jeremy.

fix build warning.

Guenther

WHATSNEW: Update changes since 3.2.1.

Karolin

fix smb_len calculation for chained requests

I think chain_reply() is one of the most tricky parts of Samba. This recursion
needs to go away, we need to sequentially walk the chain list.

Fix andx offset calculation for more than 2 chained requests

Untested code is broken code.... Test follows later, it's quite an intrusive
change to libsmb/

One more build fix. Ensure we have KRB5_AUTH_CONTEXT_USE_SUBKEY defined before we compile the new code.
Jeremy.

Try and fix the build for systems that don't have krb5_auth_con_set_req_cksumtype().
Jeremy.

Add Derrick Schommer's <dschommer@F5.com> kerberos delegation patch. Some
work by me and advice by Love.
Jeremy.

using NGROUPS_MAX instead of 32 for the max group value in rep_initgroups() subroutine in lib/replace/replace.c

Fix bug #5675 with a varient of Tim Waugh's patch,
as proposed by James Peach.
Jeremy.

Solve an IBM XL C/C++ compiler error encountered in get_exit_code() auth_errors array initialization in client/smbspool.c

fixed permissions on ctdb databases
(cherry picked from commit 123fc3980a83d956bffaa689f3af81bbf81ce1c1)

fixed a fd leak when trying to regain contact to a domain controller
in winbind

When a w2k3 DC is rebooted the 139/445 ports come up before the
udp/389 cldap port. During this brief period, winbind manages to
connect to 139/445 but not to udp 389. It then enters a tight loop
where it leaks one fd each time. In a couple of seconds it runs out of
file descriptors, and leaves winbind crippled after the DC does
finally come up
(cherry picked from commit 57187cafbcc053e75bb54750494df9feabe3a738)

dbwrap: add comment describing behaviour of dbwrap_change_int32_atomic().

Michael
(cherry picked from commit f8f21c8e3922806230e240cb54205fc2db7a3619)

secrets: fix replacemend random seed generator (security issue).

This is a regression introduced by the change to dbwrap.
The replacement dbwrap_change_int32_atomic() does not
correctly mimic the behaviour of tdb_change_int32_atomic():
The intended behaviour is to use *oldval  as an initial
value when the entry does not yet exist in the db and to
return the old value in *oldval.

The effect was that:
1. get_rand_seed() always returns sys_getpid() in *new_seed
   instead of the incremented seed from the secrets.tdb.
2. the seed stored in the tdb is always starting at 0 instead
   of sys_getpid() + 1 and incremented in subsequent calls.

In principle this is a security issue, but i think the danger is
low, since this is only used as a fallback when there is no useable
/dev/urandom, and this is at most called on startup or via
reinit_after_fork.

Michael
(cherry picked from commit bfc5d34a196f667276ce1e173821db478d01258b)

dbwrap: add comment describing behaviour of dbwrap_change_uint32_atomic().

Michael
(cherry picked from commit 7edfb54c865ddcfd5cdcc8c2184b96aaac2d2ec0)

idmap_tdb2: fix a race condition in idmap_tdb2_allocate_id().

The race is a regression introduced by the change to dbwrap.
It might have led to two concurrent processes returning the same id.

This fix is achieved by changing dbwrap_change_uint32_atomic() to
match the original behaviour of tdb_change_uint32_atomic(), which
is the following: *oldval is used as initial value when
the value does not yet exist and that the old value should be
returned in *oldval.

dbwrap_change_uint32_atomic() is used (only) in idmap_tdb2.c,
to get new ids.

Michael
(cherry picked from commit 72bd83fea7572a6202027b200d192c05023aa633)

Without stdlib.h we don't get a prototype for free().

This test fails if GCC emits any warnings (presumably to detect the
function propertly), but unless we include this message then free()
fail.  Why we need to call free in a configure test is probably
something to blame on valgrind...

Andrew Bartlett
(cherry picked from commit d013f6fadc3e80fabb4a1784207dabc84f9b7dc2)

WHATSNEW: Start WHATSNEW for 3.2.2.

Karolin

VERSION: Raise version number up to 3.2.2.

Karolin

man pages: Improve description of boolean values in smb.conf.5.

This fixes bug #5378.
Thanks Morton K. Poulsen <morten+bugzilla.samba.org [at] afdelingp.dk>
for reporting!

Karolin
(cherry picked from commit 8195ca2132cbdba396dc35e9d04d4bdc3a8a666c)

man pages: Add documentation about smbclient command "rename".

This fixes bug #5268.
Thanks to Alexander Franz <a.franz [at] gmx.net> for reporting!

Karolin
(cherry picked from commit 0a93fd2dedfa7fed1ad0b8a5e079bf7be72a4bd5)

WHATSNEW: Update changes since 3.2.0.

Karolin

Compile error on AIX 6.1

Samba3 HowTo: Fix duplicate chapter id.

Karolin
(cherry picked from commit f67b6fd97e177a527e896861f337c2e70541f697)

WHATSNEW: Update changes since 3.2.0.

Karolin

Fix uninitialized variable.
Jeremy.

rpc_client: Bug 5616 - fix session keys also in rpccli_netr_LogonSamLogonEx wrapper.

Guenther
(cherry picked from commit fef58091408cce0d7870c86f28f78cf9400cf2b6)

winbindd: handle trusted domains without sid.

Guenther
(cherry picked from commit 0c1efc6c89b1a51a94d10971bf0fc515416709b3)

libwbclient: let wbcStringToSid handle the global NULL sid.

Guenther
(cherry picked from commit 09fed085bea9dae5bb8aacd986deed3d458e3574)

man pages: Fix typos in man smbclient.

Karolin
(cherry picked from commit db843d2faafb3487e3c7df4a3a52d6c71f369e1d)

Attempt to fix bug 5616

We were calculating the session key but did not return it to the caller...

WHATSNEW: Update changes since 3.2.0.

Karolin

Fix an incompatible pointer warning

Fix bug creating files using DOS clients with mixed
case files. Reported by Daniel Johnson <Progman2000@usa.net>.
The smb_set_file_time() call to set the filetimes is failing
because it's using the unmodified name passed in by the
client, not the modified name (matching case on the
disk) that comes out from create_file().
Jeremy.

WHATSNEW: Update changes since 3.2.0.

Karolin

Fix a race condition in winbind leading to a crash

When SIGCHLD handling is delayed for some reason, sending a request to a child
can fail early because the child has died already. In this case
async_main_request_sent() directly called the continuation function without
properly removing the malfunctioning child process and the requests in the
queue. The next request would then crash in the DLIST_ADD_END() in
async_request() because the request pending for the child had been
talloc_free()'ed and yet still was referenced in the list.

This one is *old*...

Volker

talloc_string_sub2: Don't return NULL if src is empty.

This fixes BUG #5635.

Finished print jobs were not removed from the $PRINTER.tdb file if
"printing=cups".

In print_queue_update, talloc_string_sub2 is used to assemble the
"lprm command". In the case of using "printing=cups", the default
"lprm command" is an empty string. talloc_string_sub2 is called with
this empty string and returns NULL which leads to exiting
print_queue_update without doing the actual print queue update.

Signed-off by Michael Adam <obnox@samba.org>
(cherry picked from commit 03d66554d1bbd9d6c72a3dd5203e5305343c76b8)

If we're not allowing streams on this conn ptr,
then don't allow create_file() to call down to
create_file_unixpath() with a stream name.
Jeremy.

Remove trailing withespace from wbinfo -m which breaks gdm auth.

Signed-off-by: Andreas Schneider <anschneider@suse.de>

Remove test changes added in error.
Jeremy.

Use sys_getgrnam not getgrnam. Pointed out by Herb.
Jeremy.

pdb_ldap: Raise level for debug message to avoid log file flooding.
(cherry picked from commit 9b863a10da8762f715c16f147d6cd1e79422d248)

get rid of unused variable warnings when HAVE_IPV6 not defined

man pages: "printcap name" is a global parameter, not a share specific one.

Karolin
(cherry picked from commit 9cde2525560536e16e2dbf991dbeb446260dd0df)

Update Roadmap

testsuite: fix expr error on Tru64, triggered by uninitialized failed variable.

Michael
(cherry picked from commit 25cd227aaecf91dfa3114ae0fd2b47c783434aad)

testsuite: fix another expr syntax error. in tests_all.sh.

Michael
(cherry picked from commit 486b1bc5b5021565ec443fd463c8287c02c686b3)

testsuite: fix expr syntax error in net registry test.

Pointed out by Björn Jacke <bj@sernet.de>.

Michael
(cherry picked from commit 63e7d9adb7a51b3790eb431615cb6e450391a894)

Fix link to Using Samba

Fix the build of vfs_zfsacl.c

WHATSNEW: Update changes since 3.2.0.

Karolin

Document the streams modules

Change the directory parameter in streams_depot to become consistent

This is not built by default and not documented in 3.2.0, so I doubt this will
break any setups :-)

smbldap: Fix typo in debug message.

Karolin
(cherry picked from commit 6e9e85b897431d7378396aada2c32ae470af0edd)

From Jim McDonough

In reloading the smb.conf, if a "log file" is specified in smb.conf,
winbind children will overwrite the logfile name to be the same as the
parent.

Jeremy.

gitignore: ignore the generated library symbol files

Michael

allow SID with 0 in subauthority to be converted properly
from string representation.

A brown paper bag bug fix for "net sam rights"
(cherry picked from commit 8a455b012827cfb917473a160c5710a90490451a)

Allow %u parameters for print job username - use advanced sub

man pages: Fix typos and add improvements.
(cherry picked from commit 060ff6d8d4e61f3ce89f980182413852cc5719be)

WHATSNEW: Update changes since 3.2.0.

Karolin

Release still reachable memory if the smbclient context is freed.

Signed-off-by: Andreas Schneider <anschneider@suse.de>

Fix from Volodymyr Khomenko <Volodymyr.Khomenko@exanet.com>. Make ntimes
function more like POSIX and allow NULL arg. Help vfs developers.
Jeremy.

Fix swat. Bug #5613.
Jeremy

testparm: refactor the per-share logic checks out into do_per_share_checks().

Just to enhance clearness of the code.

Michael
(cherry picked from commit cd5d13dec0153c64d541e85bd13138b738cf50af)

testparm: skip the per-share logic checks when --skip-logic-checks is specified.

Michael
(cherry picked from commit 6cafee7e6ae02a32a9f2ddf313d2a20224fa22fe)

testparm: rename -g|--skip-global-ckecks to -l|--skip-logic-checks

as suggested by Karolin.

That is what it really means. And per-share logic tests will be
disabled by the same switch, too...

Michael
(cherry picked from commit 5b8a4c33482917a1ef7071df224957ba831d853a)

testsuite: fix bug #5594 by using the new --skip-global-ckecks switch

Michael
(cherry picked from commit 32f4b1443af12026c419684be45a7d8b96ccfaac)

testparm: add a switch --skip-global-checks.

This allows for successfully calling testparm without the need
to "make install" before. Without this, testparm fails with
the message that the lock directory does not exist...

Michael
(cherry picked from commit eeb018fad65c19804a4e96ea949fc2c64883439e)

Add fix from Simo for bug #5540 - missing code to substitute
%u. Make this the same as other uses.
Jeremy.

build: use $(PAM_WINBIND_EXTRA_LIBS) instead of @PAM_WINBIND_EXTRA_LIBS@ in rules.

Eliminate more potential traps for picky makes.

Michael

build: use $(WINBIND_NSS_PTHREAD) instead of @WINBIND_NSS_PTHREAD@ in rules.

Eliminate more potential traps for picky makes.

Michael

build: use $(WINBIND_NSS_EXTRA_LIBS) instead of @WINBIND_NSS_EXTRA_LIBS@ in rules.

Eliminate more potential traps for picky makes.

Michael

build: use $(LDAP_LIBS) instead of @LDAP_LIBS@ in rules.

Eliminate more potential traps for picky makes.

Michael

build: use $(LIBNETAPI_LIBS) instead of @LIBNETAPI_LIBS@ in rules.

Eliminate more potential traps for picky makes.

Michael

build: link in $(WINBIND_LIBS) instead of @LIBWBCLIENT_SHARED@

This should fix some more picky makes when not building @LIBWBCLINET_SHARED@.
Moreover, we want to link against -lwbclient, not against bin/libwbclient.so,
anyways.

Michael

build: use $(LIBTDB_LIBS) instead of @LIBTDB_LIBS@ in rules.

Eliminate more potential traps for picky makes.

Michael

build: use $(LIBTALLOC_LIBS) instead of @LIBTALLOC_LIBS@ in rules.

Eliminate more potential traps for picky makes.

Michael

build: use $(POPT_LIBS) instad of @POPT_LIBS@ in rules.

Make picky makes happier.

Michael

build: fix build on systems with picky make and without shared libwbclient

This produces empty lines in Makefile rules.
Replacing @WINBIND_LIBS@ by $(WINBIND_LIBS) where
WINBIND_LIBS=@WINBIND_LIBS@ should fix that.

Michael

Fix typo.

retieve -> retrieve

Karolin
(cherry picked from commit 37c64130701ab13b6f34998ac17fec2d128c2e08)

Fix regression introduced with 4086ef15b395f1a536fb669af2103a33ecc14de4
We must not return an error here just because we are offline.
We must instead fix the mappings to the best of our knowledge
(ie mark as mapped, expired ones, and as unmapped, unknown ones)

build: fix build of libaddns shared lib : it needs socket_wrapper.

Michael
(cherry picked from commit 58bc04b31b15c36505c5ecb2d8373e9033972390)

build: fix build of libaddns shared lib. symbols file was missing.

This symbols file is checked because mksyms script currently does
not support headers without exportet symbols...

Michael
(cherry picked from commit 1fa57b47e8f8da33a5713096c187bafb3a6c5b79)

cleanup some dup code by passing the password to the wbinfo_auth*
functions

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

Fix realpath() check so that it doesn't generate a core() when it fails.

docs: fix the specified default of "password server" - it is *, not "".

Michael
(cherry picked from commit d7f759d302612ef9eb0bba022f3b3e1dd433f843)

Always set "registry shares = yes" for "include = registry"

This fixes the case where "include = registry" is set in the smb.conf text file
and *only* shares (no [global] settings) are present in the registry.

Michael, please check!

Thanks,

Volker
(cherry picked from commit 6c80362e291d8c5e748b987a583e8e32acc36354)

loadparm: fix process_registry_shares() when there is only [global] in registry.

This fixes testparm printing "Error loading services." when there is only
the global section in registry and no other service definition.

Michael
(cherry picked from commit 28d197220038b9a47af177b88a8587f865e45de5)

loadarm: fix testparm with "config backend = registry".

Set the iServiceIndex to "-1" for starup with either config
backend (originally only for text backend). Otherwise,
process_registry_shares() will fail.

Currently, the only user of lp_load_with_registry_shares() is testparm.

Michael
(cherry picked from commit 5e2b925367241c41793b2eb7a628e9fc9f3ac8ff)

testparm: list registry shares with testparm.

I.e., use lp_load_with_registry_shares() instead of lp_load().

Michael
(cherry picked from commit 95339c0c1ecc49049f1fc176f72a1dcac639e06d)

loadparm: add funcion lp_load_with_registry_shares().

This is like lp_load(), except that it will load the
registry shares into the services aarray when registry
shares are activated.

Michael
(cherry picked from commit b387c614ba8d1d3960f6917f03f0fd433cf2b2ee)