Add new functions and tests: str_list_make_empty(), str_list_make_single()

What to do about debugging in a multi-threaded application?

- For now, punt. Any thread that requests debug to stderr will establish that
  for all threads.

Derrell

Thread-safe protection: libsmbclient initialization

- Begin converting init functions to use SMB_THREAD_ONCE. libsmbclient
  module-wide initialization is now moved into a separate function and called
  via SMB_THREAD_ONCE.

- libsmbclient counts users (contexts) so that it can release global resources
  when the last context is closed. That count of contexts is now protected by
  a mutex.

Derrell

Take advantage of the easier-to-use thread macros

- Now that we initialize for the non-thread-safe case in the macro, there's no
  need to do it here too.

Derrell

Make the thread functions a bit easier to use

- Create separate macros for lock and unlock so that it's easier to identify
  which request is being made.

- Initialize *ponce in the SMB_THREAD_ONCE macro in the non-thread-safe case,
  rather than requiring each init function to determine if it's in the
  non-thread-safe case and manually initialize.

Derrell

s3:libsmb: move read_smb_send/recv() static in async_smb.c

metze

s3:libsmb: let cli_smb_chain_send() also return NTSTATUS

metze

s3:libsmb: return NT_STATUS_CONNECTION_INVALID if the fd is -1

This way we can destinguish between requests which failed
because the connection broke after they were triggered
and the requests which are started on an already broken
connection.

This also moves the check to cli_smb_req_iov_send()
where it really belongs.

metze

Revert "s3: checkin .po for pam_winbind" because of license issue.

This reverts commit 8611f03dbb0c45d8755725232508cff258b426d5.

Allow a parameter to smb_thread_once's initialization function

- This should make life easier for ourselves. We're no longer constrained to
  the semantics of pthread_once, so let's allow passing a parameter to the
  initialization function. Some of Samba's init functions return a
  value. Although I haven't searched, I suspect that some of the init
  functions require in input parameters. The parameter added here can be used
  for input, output, or both, as necessary... or ignored, as is now done in
  talloc_stackframe_init().

Derrell

s3-printing: no need to define struct table_node 4 times.

Guenther

s3-net: Fix bug 6340: don't segfault when cleartext trustdom pwd could not be retrieved.

Guenther

s4-selftest: match new name of RPC-SAMR-ACCESSMASK.

Guenther

s4-smbtorture: split RPC-SAMR-ACCESSMASK into several tests.

Guenther

s4-smbtorture: cosmetic cleanup for RPC-SAMR-ACCESSMASK.

Guenther

Remove an unused struct definition

s3: checkin .po for pam_winbind

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

s3: return proper error code in cli_smb_req_send

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

s3/packaging: Fix build on RHEL when ccache is not available.

This fixes bug #5832.
Patch was provided by D.L. Meyer <dlmeyer [at] uiuc.edu>.

Thanks for reporting and providing the patch!

Karolin

Fix broken smb_thread_once function (again)

- It would help if smb_thread_once did, eventually, set the variable that
  prevents the init function from being run again. Sigh. It must be getting
  late.

Derrell

Fix broken smb_thread_once function

- We can't set *ponce=true before running the function because although other
  threads wouldn't re-run the initialization function, they could potentially
  proceed beyond the initialization point while the first thread was still
  running the initialization function. If a second thread gets to an
  SMB_THREAD_ONCE() call while one with the same ponce is running, we need to
  ensure that it enters smb_thread_once() to await the mutex and then recheck
  whether *ponce is set or not.  My original comment about other "once"
  functions possibly being called from within this "once" function is
  irrelevant since those other ones would have their own unique ponce.

Derrell

s3 onefs: Self-contend level2 oplocks on BRL

s3 onefs: Fix ignore sacl parameter

s3: Fix strict locking with chained reads

Move the strict lock/unlock code down a level for reads to avoid
calling chain_reply before the unlock.

s3: Fallback to the legacy sid_to_(uid|gid) instead of returning NULL.

This is very similar to be1dfff02d562e42a7847bd02fed8538630d3f41

s3 auth: Make debug message louder and more useful

Use PYTHON_VER rather than PYTHON_VERSION, as the first is already
commonly used (e.g. in the FreeBSD ports system).

Signed-Off-By: Jelmer Vernooij <jelmer@samba.org>

s3-spoolss: move SPL_ARCH_X defines to IDL.

Guenther

spoolss: add SPOOLSS_ARCHITECTURE_ALL to IDL.

Guenther

s3-build: re-run make samba3-idl.

Guenther

initshutdown: add shutdown reason codes.

Guenther

s3-netdomjoin-gui: prepare to call out for NetRenameMachineInDomain() to rename computers
in domains.

Guenther

s4 config-summary: Fix printing of which libraries are external and
internal.

error-codes: add some more WERRORs to table.

These tables really should be auto-generated at some point...

Guenther

s3-netdomjoin-gui: Fix join prompt dialog test.

Guenther

s3-netdomjoin-gui: use NetServerGetInfo() level 101 to get hostname.

Guenther

s3-netdomjoin-gui: inspect the correct computername string before
enabling/disabling the change button.

Guenther

Clean up assignments to iov_base, ensure it's always cast to void *. This should quieten some warnings with picky compilers on the buildfarm.
Jeremy.

Fix the last few format arg missmatches I missed.
Jeremy.

Convert response.extra_data.data from malloc to talloc

Fix bug 6157

This patch picks the alphabetically smallest one of the multi-value attribute
"uid". This fixes a regression against 3.0 and also becomes deterministic.

s3-printing: Fix vlp testprinter application.

Jeremy, we cannot just access cache_path() here without calling lp_load and
friends as well as parsing configfile from the commandline in order to make
"make test/selftest" find the correct conffile with path, etc.

I just changed it to pass the target tdbfilename as an argument, ok ?

Guenther

s3:build: extend the support fo Debian GNU/kFooBar to kOpenSolaris

As suggested by Robert Millan.

Michael

s3:build: support non-linux gnu ports

This is based on a patch by the debian package maintainers,
adapted for the merged branch.

Michael

s3:loadparm: free the file_list at the start of loadparm

This should reduce the waste of memory when using "config file"
or "config backend". It also reduces the risk of triggering
reloads due to some old unused files being checked.

Michael

s3:loadparm: refactor freeing of file_list out into free_file_lists()

Michael

s3:proto.h: remove old param/params.c prototype

Michael

Fix warning about unused label with no sendfile.
Jeremy.

Fix definition of smb_thread_once - must return int not void as
it's used in a ? : comparison macro.
Jeremy.

Fix a bunch of compiler warnings about wrong format types.
Should make Solaris 10 builds look cleaner.
Jeremy.

Hopefully fix the buildfarm. Add some debug level 10 messages
I used to track down the vlp problem, change the vlp test printer
not to use a static path of /tmp/vlp.tdb for the virtual print
database (as this will eventually fill up). Cause it to use
a virtual print database inside the cachepath.
Jeremy.

Ensure documentation matches the tdb_util function usage.
Jeremy.

s3-samr: implement _samr_RidToSid().

Guenther

s3-samr: Let _samr_TestPrivateFunctionsDomain() return NT_STATUS_NOT_SUPPORTED to make RPC-SAMR happy.

Guenther

s4-smbtorture: skip SetMemberAttributesOfGroup in RPC-SAMR for s3 as well.

Guenther

s4-smbtorture: Skip secdesc set calls for non-user objects in RPC-SAMR for Samba3.

Note that they are skipped for all objects while testing Samba4.

Guenther

s4-smbtorture: Fix order of assert args in test_TestPrivateFunctionsDomain() in RPC-SAMR.

Guenther

s4-smbtorture: switch to use torture_context in RPC-SAMR.

Guenther

s4-smbtorture: remove trailing whitespace in RPC-SAMR.

Guenther

Fix some nonempty blank lines

Correct typos; semicolons instead of commas

s3-samr: Fix Bug #5859, renaming of samr objects failed due to samr setuserinfo access checks.

Torture test to follow...

Guenther

Add checks for non_root mode to help buildfarm spoolss tests.
Reviewed by Guenther.
Jeremy.

s3-lsa: Fix _lsa_LookupNames2() server implementation which always returned a NULL sid_array since 3.2.0.

Found by torture test.

This makes it possible to search for users while adding them to groups via
windows usermanager.

Guenther

s3-selftest: enable RPC-LSA-LOOKUPNAMES against Samba 3.

Guenther

s4-smbtorture: add very basic RPC-LSA-LOOKUPNAMES test.

Guenther

s4-smbtorture: add optional very simple reply checks to lsa lookup name tests.

Guenther

s4:smb_server: return after we have processed an invalid smb2 request

metze

Move SMB2 constants to a separate file

Fix SMB2_CAP_DFS

s4-smbtorture: remove trailing whitespace in RPC-LSA test.

Guenther

s4-smbtorture: use struct torture_context in RPC-LSA.

Guenther

s3-docs: document "net dom renamecomputer".

Guenther

s3-net: add "net dom renamecomputer" to rename machines in a domain.

dmarkey, please test :)

Guenther

Use bool instead of int for a boolean variable

Optimize smb_thread_once usage by not calling it if known already called

- The macro SMB_THREAD_ONCE now tests whether the "once" variable is already
  set, and if so avoids calling smb_thread_once().

Derrell

Panic upon mutex lock or unlock failure

- It's a serious error if we can't lock or unlock a mutex in
  smb_thread_once(). Panic instead of just displaying a DEBUG message.

Derrell

Provide a libsmbclient interface for programs requiring threads

- This adds two functions: smbc_thread_posix() which provides access to the
  internal threading implementation using pthread; and smbc_thread_impl()
  where the user provides each of the functions required by Samba, to give
  access to the thread implementation's native capabilities.

Derrell

Don't require external use of internal enum smb_thread_lock_type

- Internally, when locking or unlocking a mutex, we'll pass one of the
  values of enum smb_thread_lock_type. That enum is not available to users
  providing a thread implementation. Externally, we'll document the integer
  values which will be passed to their lock_mutex function, but not require
  them to access our internal header file.

Derrell

Replace external thread "once" with an internal implementation

Jeremy, please check...

- I'm in the process of providing an interface in libsmbclient to the
  recently-added threading capabilities. In the process, I discovered that
  different thread implementations have varying types for the variable passed
  to the thread_impl_once() function. pthreads, for example, uses type
  pthread_once_t. Since Samba needs to internally declare these variables, it
  would need to know the exact type required by each thread implementation's
  function. After considering multiple methods of obtaining an appropriately
  sized variable, I decided that for the basic "once" functionality required
  by Samba, it would be much simpler to just implement our own "once"
  functionality. We don't require cancellation points et all. This commit adds
  an smb_thread_once() function that is implemented using an internal
  mutex. The mutex itself uses the implementation's create_mutex
  function. This eliminates the need for the user to provide a smb_thread_once
  function pointer and the entire issue of that function's first parameter.

Derrell

Fix an unitialized variable warning

wb_int_trans -> wb_simple_trans in wbclient.c

Add wb_simple_trans_send/recv

Pass also sername to check password script

Fix the mangle1 test

Do not call SMB_VFS_GET_REAL_FILENAME if the name is mangled

The GPFS get_real_file name does not know about mangled names. Tim, if onefs
does not either, you need this bugfix :-)

In case onefs does 8.3 names, we need to pass the mangled flag down to
SMB_VFS_GET_REAL_FILENAME to give GPFS a chance say ENOTSUPP and do the
fallback.

s3:libsmb: fix layering of cli_ntrename_internal and its callers

It's easier to have cli_ntrename_internal as a semetric async
tevent_req function. cli_ntrename() and cli_nt_hardlink() should
be callers on top of cli_ntrename_internal().

metze

s3: Fix chained sesssetupAndX/tconn messages

A sesssetupAndX chained with a tconn will not correctly set the TID in
the response header.  I'm seeing an XP client send this chained
sesssetup/tconn when samba has security = share.  Samba's current
behavior is to return a TID of 0 in the smb header rather than the
actual TID.  This patch also updates the UID in the header as well.

s4-smbtorture: avoid secdesc test on connect handle in RPC-SAMR-USERS for Samba3.

With this, I think, we pass RPC-SAMR-USERS.

Guenther

s3-samr: Fix SetUserInfo level 16 and 21 w.r.t. ACB_AUTOLOCK acct_flag.

It is not allowed to *set* this flag remotely if it has been not set already.

Found by torture test.

Guenther

s3-samr: Fix SetUserInfo level 7 when there has been no name change.

Found by torture test.

Guenther

s3-selftest: enable running RPC-SAMR-USERS against Samba3.

Guenther

s4-smbtorture: skip unsupported ACB bits for Samba3 in RPC-SAMR-USER.

Guenther

s3-samr: more accurateness in _samr_SetDomainInfo().

Guenther

s4-smbtorture: Support timestamp handling for Samba3 in RPC-SAMR-USERS.

Timestamps in passdb (currently) only have second granularity.

Guenther

Fix bug #6330 - DFS doesn't work on AIX. Jeremy.

Expand the comment explaining why user_in_group_sid is
not reliable for winbindd users from foreign domains.
Jeremy.

s3:smbd: fix posix acls when setting an ACL without explicit ACE for the owner (bug#2346)

The problem of bug #2346 remains for users exported by
winbindd, because create_token_from_username() just fakes
the token when the user is not in the local sam domain. This causes
user_in_group_sid() to give totally wrong results.
In uid_entry_in_group() we need to check if we already
have the full unix token in the current_user struct.
If so we should use the current_user unix token,
instead of doing a very complex user_in_group_sid()
which doesn't give reliable results anyway.

metze

s3:smbd: fix the fix for mapped IPv4 address handling in release_ip().

It was too late... Thanks Metze for noticing.

Michael

s3-selftest: run RPC-LSA-GETUSER against Samba 3.

Guenther