r24981: - Use the formal syntax for calling functions through pointers.  I've wanted
  to make this change for ages, but now with the issue of "open" requiring it,
  this is the time to just do all of them.

Derrell

r24978: move domain_init_recv() near the functions that uses it

metze

r24977: Ensure negative caching for name2sid, sid2name and rids2names
mappings.

rafal

r24974: catch SIGINT and SIGQUIT like SIGTERM

metze

r24970: Polish some comments and add my (C).

Michael

r24969: Fwd port "open" patch

r24955: Reformat some object lists in Makefile.in to minimize the diff.

r24954: Fix the build without krb5.

Guenther

r24952: Set the kdc locator env vars. This makes the krb5 locator plugin fully
operational (from within winbindd and outside).

Guenther

r24951: Warn if "os level" > 255

Thanks to Karolin Seeger <ks@sernet.de>

r24949: Remove some static buffers

r24943: Some stackframes

r24920: Reformatting.

Guenther

r24903: One more tick in #if 0 code

r24902: DEBUG might use talloc_tos() itself...

r24879: Activate the winbindd cache-validation message handler.
Now the winbindd cache can be checked at runtime by
calling "smbcontrol winbindd validate-cache".

For the execution of the validation code, I fork a child
and in the child restore the default SIGCHLD handler in
order for the fork/waitpid mechanism of tdb_validate to work.

Michael

r24877: Don't panic in tdb validation code when the fork or waitpid fails.
Return error instead.

Michael

r24864: - Correct failure of libsmbclient against a version of Windows found on a NAS
  device.  The device resets a NBT connection on port 139 when it receives a
  NetBIOS keepalive request.  That request should be supported when NetBIOS is
  in use; Windows is behaving badly.

  libsmbclient needs a way to determine if a connection is still alive, and
  was using a NetBIOS keepalive request if port 139 was in use (on the
  assumption that it was probably NBT), and getpeername() when port 139 was
  not being used (assuming naked transport).

  This patch simplifies the code by exclusively using getpeername() to check
  whether a connection is still alive.  The NetBIOS keepalive request is
  optional anyway (with preference being given to using TCP mechanisms for the
  same purpose), so this should be both simpler and more reliable.

Derrell

r24853: Rename function as Jerry asked.
s/net_use_upn_machine_account/net_use_krb_machine_account/

rafal

r24848: Make tdb_validate() take an open tdb handle instead of a file name.
A new wrapper tdb_validate_open() takes a filename an opens and closes
the tdb before and after calling tdb_validate() respectively.

winbindd_validata_cache_nobackup() now dynamically calls one of
the above functions depending on whether the cache tdb has already
been opened or not.

Michael

r24847: Change standard failure return code of tdb_validate from "-1" to "1".
(This is more safely used with casts from int to uint8, e.g.)

Michael

r24845: Fix a segfault in smbcontrol when called with parameters but no extra args.

Michael

r24843: Add a "validate-cache" control message to winbindd.
So there is a new subcommand "smbcontrol winbindd validate-cache" now.

This change provides the infrastructure:
The function currently returns "true" unconditionally.
The call of a real cache validation function will be incorporated
in subsequent changes.

Michael

r24842: Fix build warning.

Guenther

r24841: The locator has no dependencies to ldap libs anymore. Also fix the build.

Guenther

r24838: Now for real: build the locator, if we can.

Guenther

r24836: Initialize some uninitialized variables.
This prevents a segfault when get_kdc_ip_string() is called
with sitename == NULL.

Michael

r24833: Move locator to nsswitch (does not belong to libads anymore).

Guenther

r24832: In the winbind-locator recursion case, try to pick up the kdc from the
environment.

Guenther

r24830: Add a winbindd cache validation function that does not do
backup and corrupt file handling. (To be used in subsequent
changes.)

r24829: Make use of a variable, that is available... :-)

r24828: Give smbcontrol an initial talloc stackframe.

r24827: Give smbstatus an initial talloc stackframe.
Rewrite main() so as to exit only at a single point
where the stack frame is freed, too.

Michael

r24826: Fix two memleaks in idmap_cache.c, bug 4917

Thanks again to Patrick Rynhart for persisting :-)

Simo, please check!

r24825: Give testparm an initial talloc stackframe, so it does not complain
it is leaking.

r24824: Create an initial talloc stackframe for the net binary.

r24813: Reformat to 80 columns.
Jeremy.

r24811: Simple reformatting to fit the 80 columns rule.
Jeremy.

r24809: Consolidate the use of temporary talloc contexts.

This adds the two functions talloc_stackframe() and talloc_tos().

 * When a new talloc stackframe is allocated with talloc_stackframe(), then
 * the TALLOC_CTX returned with talloc_tos() is reset to that new
 * frame. Whenever that stack frame is TALLOC_FREE()'ed, then the reverse
 * happens: The previous talloc_tos() is restored.
 *
 * This API is designed to be robust in the sense that if someone forgets to
 * TALLOC_FREE() a stackframe, then the next outer one correctly cleans up and
 * resets the talloc_tos().

The original motivation for this patch was to get rid of the
sid_string_static & friends buffers. Explicitly passing talloc context
everywhere clutters code too much for my taste, so an implicit
talloc_tos() is introduced here. Many of these static buffers are
replaced by a single static pointer.

The intended use would thus be that low-level functions can rather
freely push stuff to talloc_tos, the upper layers clean up by freeing
the stackframe. The more of these stackframes are used and correctly
freed the more exact the memory cleanup happens.

This patch removes the main_loop_talloc_ctx, tmp_talloc_ctx and
lp_talloc_ctx (did I forget any?)

So, never do a

tmp_ctx = talloc_init("foo");

anymore, instead, use

tmp_ctx = talloc_stackframe()

:-)

Volker

r24808: Fix the same problem Volker noticed.

For some funny reason us4/gcc seems to fall over the '

Jeremy.

r24807: Add WINBINDD_LOCATOR_KDC_ADDRESS env which will be used for the case when the
locator gets called from within winbindd.

Guenther

r24806: Fix the build, sorry...

Guenther

r24805: When we can build the locator, build it.

Guenther

r24804: As a temporary workaround, also try to guess the server's principal in the
"not_defined_in_RFC4178@please_ignore" case to make at least LDAP SASL binds
succeed with windows server 2008.

Guenther

r24803: For some funny reason us4/gcc seems to fall over the '

r24802: Activate new tests

RAW-SAMBA3CASEINSENSITIVE and RAW-SAMBA3POSIXTIMEDLOCK

r24801: When told to ignore the winbind cache also do so while trying to store entries.
Thanks Michael for pointing this out.

Guenther

r24799: Prefer IP address to dns name when replying in winbindd dsgetdcname.

Guenther

r24797: Fix logic in dsgetdcname().

Guenther

r24791: Fix logic error in timeout of blocking lock processing found by
Ronnie. If a lock timeout expires, we must check we can get the
lock before responding with failure. Volker is writing a torture test.
Jeremy.

r24789: Add implementation of machine-authenticated connection to netlogon
pipe used when connecting to win2k and newer domain controllers. The
server may be configured to deny anonymous netlogon connections which
would stop domain join verification step. Still, winnt domains require
such smb sessions not to be authenticated using machine credentials.
Creds employed in smb session cannot have a username in upn form, so
provide the separate function to use machine account.

rafal

r24788: Cleanup some linking msgs and remove references to libmsrpc
in SAMBA_3_2_0

r24786: Fix another build warning.

Guenther

r24785: Put checks in parentheses.

Guenther

r24784: Initialize uninitalized data to prevent segfaults.
Thanks to Volker for the hint!

Michael

r24783: Remove unused off_t type.

Guenther

r24782: Fix C++ warnings

r24781: Fix build warning.

Guenther

r24778: Make sure krb5 locator requests go to a separate locator winbind child.

Guenther

r24776: Remove accidentially commited flag checks.

Guenther

r24773: Fix a ctdb connection lockup

The lockup could happen when packet_read_sync() gets two packets in a row, the
first one being an async message, and the second one being the response to a
ctdb request.

Also add some debug msg to ctdb_conn.c, and cut off the "locking key" messages
to only dump 20 hex chars at debug level 10. >10 will dump everything.

r24771: Use infolevel 25 to set the machine account's password (just like winxp).
This correctly updates pwdLastSet field on win2k3 server.

rafal

r24769: Merge error handling for locator plugin.

Guenther

r24764: Fix second TALLOC_SIZE definition. Still watching the
build farm to see I didn't stuff this up...
Jeremy.

r24762: Fix the build, missed TALLOC_SIZE -> talloc_named_const.
Jeremy.

r24759: Comment out the _nonnull calls for 3.2.x, as agreed with tridge.
Leaving the commented out code for now, in case I need to re-test
some stuff.
Jeremy

r24752: Make sure to return properly when the locator is called from within winbindd.

Guenther

r24750: Fix one more caller of  name_resolve_bcast().

Michael

r24749: Increase debuglevel.

Guenther

r24748: Remove all dependencies to samba internals and convert the krb5 locator plugin
into a tiny winbindd DsGetDcName client. This still does not solve the case of
using the locator from within winbindd itself but at least gencache.tdb and
others are no longer corrupted.

Guenther

r24747: Add WINBINDD_DSGETDCNAME call.

Guenther

r24746: As the winbindd pipe is officially broken since a while: split out request
specfic and generic flags in a winbindd_request.

It turns out that the WBFLAG_RECURSE flag is the only non-PAM specific flag we
put into the "flags" field of a winbind request anyway. Now each request
command can use the entire space of the "flags" field.

Guenther

r24745: Merge Simo's shared lib build fix from svn r22842 that was lost
somehow.  Don't include the PIE_FLAGS when building shared libs.

r24744: Increase length by what we got from recv, not from ioctl

r24743: Fix build warning.

Guenther

r24742: Add experimental DsGetDcName() call (will be used by krb5 locator for fine
grained KDC DNS queries).

Guenther

r24740: Fix the build.

Guenther

r24739: With resolve_ads() allow to query for PDCs as well.
Also add dns query functions to find GCs and DCs by GUID.

Guenther

r24738: Fix one more use of pwrite in tdb code in the spirit of r23972 and r23977.

Michael

r24737: Remove older TODO: Convert internal_resolve_name() and friends to NTSTATUS.

Guenther

r24734: Move nss_err_str() to a more public place.

Guenther

r24733: Add support for storing DATA_BLOBs in gencache.tdb (including torturetest).

Mimir, please have a look. DATA_BLOBs will now just show up as "DATA_BLOB"
values with "net cache list".

Guenther

r24722: Squashed commit of the following:

commit fb52f971986dd298abbcd9745ddf702820ce0184
Author: Gerald Carter <coffeedude@plainjoe.org>
Date:   Mon Aug 27 13:50:26 2007 -0500

    Check correct return type for pam_winbind_request_log() wnibind_upn_to_username

    which is an int and not NSS_STATUS.

commit 7382edf6fc0fe555df89d5b2a94d12b35049b279
Author: Gerald Carter <coffeedude@plainjoe.org>
Date:   Mon Aug 27 13:30:26 2007 -0500

    Allow wbinfo -n to convert a UPN to a SID

commit 8266c0fe1ccf2141e5a983f3213356419e626dda
Author: Gerald Carter <coffeedude@plainjoe.org>
Date:   Fri Aug 3 09:53:16 2007 -0500

    Merge some of Guenther UPN work for pam_winbind.c (check the winbind separator

    and better pam logging when converting a upn to a username).

commit 15156c17bc81dbcadf32757015c4e5158823bf3f
Author: Gerald Carter <coffeedude@plainjoe.org>
Date:   Fri Aug 3 08:52:50 2007 -0500

    Include Universal groups from the cached PAC/SamLogon info when

    generating the list of domain group SIDs for a user's token.

commit 979053c0307b051954261d539445102c55f309c7
Author: Gerald Carter <coffeedude@plainjoe.org>
Date:   Thu Aug 2 17:35:41 2007 -0500

    merge upnlogon patch from my tree

r24714: Fix confusing indent.

Guenther

r24713: Fix obvious error in enum_dom_groups. We were returning NT_STATUS_OK when the realloc failed.

Guenther

r24711: Remove unused talloc context from query_user_list rpc.

Guenther

r24702: Remove the old API pointers

r24701: Fix the swat build

Swat has not been built by default for a while, so I did not notice that
the _ macro is actually used. Re-add the lang_msg_rotate function, this
time only to swat so that this is the only binary that has to take the
16k penalty.

r24699: Actually write 24 zeros instead of zero 24's...

Jeremy, please check ;-)

r24661: Fix some obvious diffs between 3_2 and 3_2_0

Jeremy, there are two remaining diffs in sesssetup.c which I don't really
know which one is right. Can you take a look?

Thanks,

Volker

r24659: Some formatting changes helping to minimize the 3_2_0 diff

r24654: Adapt to coding conventions.

Guenther

r24649: Attempt to fix bug 4917. Simo, please check!

Thanks Patrick Rynhart for reporting this.

r24639: Add parameter "directory name cache size" - parameterize
use of directory name cache, 100 by default. Will be needed
to turn this off for *BSD systems.
Jeremy.

r24638: Remove redundent setting of vuid.
Jeremy

r24637: In order for "net rpc registry" to be able to write to
Samba's own registry, the access mask for opening the
registry for the write operations needs to be
SEC_RIGHTS_MAXIMUM_ALLOWED instead of REG_WRITE: we can
not open e.g. HKLM read write explicitly, since we can
not write to this virtual part of the registry, only
to the subkeys like 'HKLM\Software\Samba\smbconf' that
are stored on disk.

Note that MAXIMUM_ALLOWED is also what windows' regedit
passed to the open calls.

Michael

r24636: Raise one debug level, since the callers (that want to do so)
issue a level 0 debug message.

Michael

r24632: Fix build warnings.

Guenther

r24630: Store Samba configuratin options only under the default name, not as aliases.
This prevents creation of problematic configurations from registry editors
like regedit or "net rpc registry".

I will refactor the code to be somewhat more concise,
but I wanted to have this in the tree, now I got it working... :-)

Michael

r24629: Make read_sock return the total number of bytes read instead
of the number of bytes read in the last of possibly several
read calls.

This was noted by Metze.

Michael