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

r24627: merge from SAMBA_4_0:
as TALLOC_ABORT() is defined to abort() by default
wrap it into a function so that the function name
in the backtrace shows what happens.

metze

r24625: add '-D' option to winbindd

TODO: don't allow '-i -D' and '-D -i' on all
      server binaries in the SAMBA_3_2 branch!
      The &server_mode patch makes this difficult to check...

metze

r24621: - deferr calling build_options();exit(0);
- use poptPrintUsage() to give the user more info

metze

r24618: move printing out the version string and exit()
into the popt _POST processing.

Now 'smbd -V --bla' complains about an unknown option

metze

r24616: In set_boolean(), only pass the result back to the caller
if parsing of the boolean string was successful.
Also, initialize the local result variable
(although not strictly necessary anymore, now.)

r24615: Say for which domain the getdcname request failed.

Guenther

r24610: Add a debug message that the workaround has been activated...

r24609: Fix the fix of r23668 for win2k giving one zero byte
instead of a 2-byte zero character. I can't recall what
rode me when I put that "2" there. But now I think I
have got it right... :-)

Michael

r24605: remove some stuff samba3 doesn't need any more

metze

r24603: In case of error we need to free prefix_cache otherwise on re-init the first
statement will return positively but prefix_cache_hashes would be NULL

r24602: Add function lp_string_is_valid_boolean() to check if a string
contains a correct representation of a boolean value (in the
understanding of loadparm.c).

Make set_boolean() catch passing NULL for the boolean target.

Michael

r24601: Fix if statements where we still expected cli_connect() to return BOOL.

r24599: patch from Karolin Seeger <ks@sernet.de>:

smbd, nmbd and winbindd can be started with invalid options currently.
The first patch attached would be a possible solution.
It contains an exit if an invalid option has been used. The main problem
is, that existing setups with wrong options or missing arguments in start
scripts will break (which is the right behaviour from my point of view).

metze

r24597: - talloc request structure for the samba3 server bindings
  and make that the primary context for the request
  which the implementations can also use.
- go via functions pointers in the ndr_interface_table
  instead of calling functions directly.

metze

r24595: Fix Coverity ID 393

In this error case we would have used "start" not having it initialized

r24594: pass down the ndr_interface_table in the samba3 client bindings
instead of the pull and push functions

metze

r24590: Reformatting to coding standards. Added my (C) in places it already should
have been :-).
Jeremy.

r24589: Refactor our vuid code so that we keep the same
vuid that was allocated whilst the connection is
being constructed and after the connection has been set up.
This is what Windows does and at least one client
(and HP printer) depends on this behaviour. As it
depends on the req struct not yet ported to SAMBA_3_2_0
(Volker, hint hint.... :-) I am not yet adding this
to that branch, but will investigate that tomorrow.
Jeremy.

r24584: Merge all pam post-processing code (in particular all extra_data code) into
append_data().

Guenther

r24583: Make sure we don't accept invalid request options.
Thanks to Michael for his bit-magic.

Guenther

r24579: Merge error reporting to the end of winbindd_pam_auth().

Guenther

r24578: Fix build warning.

Guenther

r24573: Fix build without LDAP. Thanks Volker for pointing this out.

Guenther

r24572: Allow for functions map_parameter_canonical() and is_synonym_of() to
be called with inverse == NULL.

Add a new function lp_parameter_is_canonical() to check whether a
parameter name is the canonical name (as apposed to an alias).

Michael

r24571: Only look at errno if the close call actually failed

Patch from Ofir Azoulay <Ofir.Azoulay@expand.com> -- thanks

r24564: regenerate after pidl changes...

metze

r24561: merge from SAMBA_4_0:
rename some DCERPC_ prefixes into NDR_

metze

r24558: merge from SAMBA_4_0:
rename 'dcerpc_table_' -> 'ndr_table_'

metze

r24552: merge from SAMBA_4_0:
rename dcerpc_interface_table -> ndr_interface_table
rename dcerpc_interface_list  -> ndr_interface_list

and move them to libndr.h

metze

r24550: Attempt to fix the non-LDAP build

For example host "bigboy" doesn't have it.

Günther, please fix it properly and merge to 3_2_0

Thanks,

Volker

r24549: Fix unix_convert to return the already converted part

This API will change anyway when moving away from pstrings.

It took so long to fix, because that rename bug I just fixed gave make
test failures that had nothing to do with this one.

I have samba4 tests for both bugs, will check them in when the build
farm has caught up

r24548: Fix the case-changing renames

This was broken when I changed reply_mv to wrap in a open_file_ntcreate
call, unix_convert on the destination was called twice

r24547: Fix #4897, patch from David S. Collier-Brown <davecb@spamcop.net> -- Thanks!

r24543: Apply missing portion of correction for bug 4750

r24541: merge from SAMBA_4_0:
rename struct dcerpc_endpoint_list/struct dcerpc_authservice_list
into ndr_interface_string_array and move it to libndr.h

metze

r24538: merge from SAMBA_4_0:
rename struct dcerpc_interface_call -> struct ndr_interface_call
and move it to librpc/ndr/libndr.h

metze

r24537: regenerate after pidl fixes

metze

r24533: merge from SAMBA_4_0:
rename dcerpc_syntax_id -> ndr_syntax_id

metze