r20329: Fix a winbindd crash bug. If someone pulls
the network cable out of the machine *exactly*
after the init_dc_connect() call in cm_connect_sam()
or cm_connect_lsa() call succeeded but before any
of the other calls fail, and they have debug level
10 set in the log, then we'd crash due to dereferencing
a now NULL pointer (conn->cli gets set to NULL when
the init_dc_connect() call called from cm_get_schannel_dcinfo()
fails). Yes, before you ask this *did* happen on a
customer site :-).
Jeremy.

r20313: Make trans2_mkdir also go through open_directory for all the fancy inherit_acl
stuff & friends.

Survives raw-mkdir. I'll activate that tomorrow when all build farm machines
have picked up the changed samba4 torture test.

Volker

r20312: Fix a memleak

r20310: We've checked for EA's and support for them further up already.

Volker

r20304: Smaller fixes for pam_winbind:

* fail on invalid credential flags in pam_sm_setcred
* parse config file for pam_sm_acct_mgmt and pam_sm_open_session

Guenther

r20296: If we're going to overwrite krb5.conf
only do it for our primary domain.
Jeremy.

r20290: Remove unused call

r20289: IDMAP is part of winbind but not the main process.
Make sure we route all request to remote DCs via the main process
so that IDMAP can correctly reuse DC connections and use the
async interface.

This fixes also idmap_nss so that it is able to resolve local
group names (requires patch on the samba dc earlier committed
to SAMBA_3_0 to make it resolve both the mapped and the unmapped
name).

Simo.

r20287: Allow a NULL sid when setting up the list of trusted domains.

r20279: Fix winbind segfault in winbindd_getsidaliases.

Jeremy: sidstr formerly could be NULL (when num_aliases was 0), since we
strdup here it needs to exist.

Guenther

r20273: Map KRB5_KDCREP_SKEW to NT_STATUS_TIME_DIFFERENCE_AT_DC.

This gives much nicer error messages when failing to join due to clock
skew.

Guenther

r20271: Fix the build

r20270: Even with the dual daemon mode the parent winbindd
still needs to contact the DC's for non async requests
like enumerate users/groups etc. Now that online
DC detection is tied to async events we must enable
the processing of events in the main loop of winbindd.
Finally got rid of the last hard coded domain->initialized = 1
code in init_child_recv() - now all domain->initialized = True
gets done only in the connection manager code when either
we're online and have spoken to the DC or are offline and
we know we can't talk to the DC.
Jeremy.

r20269: merge -r20264:20267 from SAMBA_3_0_24
more no previous prototype warnings

r20268: merge -r 20261:20263 from samba_3_0_24
get rid of previous prototype warnings

r20264: Two lines above this we just checked if (fsp->is_directory) ...

r20261: merge 20260 from samba_3_0_24
clean up a bunch of no previous prototype warnings

r20257: Hey, change_owner_to_parent is now static :-)

r20255: Activate BASE-SAMBA3ERROR in the build farm

r20254: The pam_chauthtok needs to go through the async interface as well.
This fixes pam password changes in the online case.

Guenther

r20253: Reduce some code duplication, make reply_mkdir go through the same code paths
ncreate does. This is a bit slower (about 10-20%), because it goes touches the
share mode db, but I think not having to call change_owner_to_parent and
friends in fewer places outweighs this. And, mkdir is not the way current
Windows boxes create directories, they do it via the ncreate call.

Volker

r20252: Revert back to const, but I have a fleble feeling we should go the ther way.

r20250: If we've come from being globally offline we
don't have a check online event handler set.
We need to add one once we're been asked to
go back online as this is the only way to actually
go into the online state. Doh ! :-).
Jeremy.

r20249: Fail when parsing invalid options in _pam_parse.

Guenther

r20248: merge 20247 from samba_3_0_24
get rid of warning: value computed is not used

r20245: merge 20244 from samba_3_0_24
get rid of more nested extern declarations warnings

r20243: Make lookup_name resolve both the mapped and the real unix group name

r20242: these are not really const as we set them in the function

r20241: Slightly improve readability of the pam_vsyslog replacement function.

Guenther

r20240: Be a little more verbose about the credential flags when the debug flag
is set.

Guenther

r20239: Parse the configfile for pam_sm_setcred as well
(e.g. to get the debug flag)

Guenther

r20237: Replace exit_server with exit_server_cleanly where appropriate. All
send_smb failures should be clean exits. All times when we exit as
a matter of policy should also be clean exits.

r20235: Don't use realloc directly.

r20229: Avoid a silly function call in mkdir_internal. None of the callers look at
errno, all go straight to ERROR_NT(status).

Volker

r20228: Bring the calling conventions of inherit_access_acl and change_owner_to_parent
a bit closer together: Move the lp_inherit_perms() check into the callers.

Volker

r20227: Simplify logic in vfs_MkDir() a bit

r20218: Same fix as r20217 -- can we consolidate that duplicated code a bit?

r20217: uid_t and gid_t are not necessarily 32 bit. In assignments we get an automatic
conversion, but not when we pass pointers down to other functions.

Simo, please check.

Volker

r20216: Fix fallback code.
A reversed check made it impossile to fallback to the Unix Domain mapping code.
Also fix a potential use of a freed array.

Jerry,
my tests shows that this code now correctly handle the fallback to Unix Domain
when our Domain member is asked for a mapped group that has a unix name different
from the Windows name against a Samba DC and we do not use winbindd but share
users/groups by other means (ldap / sync of passwd and group files)

Immediate Fix would be to discuss if we should answer back when DOMAIN\unixgroup -> SID
is asked for, in the case the unixgroup name is mapped to a different name.
IE: DOMAIN\Domain Admins -> ntadmins

Currently if we are asked for "DOMAIN\Domain Admins" we return the dom admins SID
If we are asked for "DOMAIN\ntadmins we return "not found", but we may consider to
return the Domain admins SID in this case too.

Comments are welcome on this point!

Long term fix I think is the unixinfo pipe and of course an idmap_unixinfo moudle.

Simo.

r20215: Next step trying to fix the Solaris build.

I think "anonimous" is correctly spelled "anonymous". The Solaris compile is
referring to this as "anonymous" in line 814 of smbldap.c. Simo, please check.

Thanks,

Volker

r20214: Fix a type-punned warning. Simo, Jerry, please check

r20212: Attempt to fix the Solaris build

r20211: send_nt_replies does not need the inbuf arg

r20210: registry_fetch_values is not needed anymore, fix typo

r20209: Fix two memleaks

r20208: Change sprintf_append() never to use malloc,
but always use a talloc context.
Thanks to simo for pointing this out.
Jeremy.

r20207: Fix a couple more places where extra_data was
being talloc'ed off the NULL context instead
of being malloced.
Jeremy.

r20206: Start cleaning up the talloc_ctx mess.
child->mem_ctx isn't actually used for
anything, so remove it.
Jeremy.

r20202: fix make clean

r20201: merge makefile changes from 3_0_24

r20197: merge talloc fixes from samba4:

- make most static functions inline
- handle NULL pointers in talloc_parent_chunk()
- use talloc_parent_chunk() in talloc_parent_name()
  to fix a bug found by the IBM checker

metze

r20186: Fix winbind crash bug in WINBIND_GETGROUPS.

response_extra_sent() expects to free a malloced
extra_data.data while the add_XX_to_array functions all return talloced
memory now. Jeremy, please check.

Guenther

r20183: Fix the build.

Guenther

r20181: Improve NT status to pam error mapping.
Jeremy.

r20180: Ensure that pam returns the correct error messages
when offline and or doing password changes.
Jeremy.

r20179: Sync up with Samba4 - remove blank lines at the
end parsing a file.
Jeremy.

r20178: Ensure we allocate the intermediate trans structs
off conn->mem_ctx, not the null context so we can
safefy free everything on conn close. Should fix
possible memleak.
Jeremy.

r20174: If we're only going to call one handler per message
then terminate the traversal once we've done that.
Jeremy.

r20173: DNS update fixes:

* Fix DNS updates for multi-homed hosts
* Child domains often don't have an NS record in
  DNS so we have to fall back to looking up the the NS
  records for the forest root.
* Fix compile warning caused by mismatched 'struct in_addr'
  and 'in_addr_t' parameters called to DoDNSUpdate()

r20171: Don't delete the krb5 credential if others still reference to it.

Guenther

r20170: Fix secure DNS updates to work against
Wnidows 2000 DNS which expects the TKEY payload to
be in the answer section and not in the additional
set of records (like Windows 2003 and the RFC).

r20169: Support for fallback to legacy mapping code was not completely tested.
Add necessary fixes.

r20165: Change messaging subsystem to only allow one message
per type - this is all we use right now and makes
re-entrancy problems with deleting handlers with
a message dispatch loop go away.
Jeremy.

r20160: Fix long-standing (ie. from initial code I think) bug
in tdb message processing. If we're inside a dispatch
function and we delete our own handler we'd walk onto
the next pointer from a deleted memory block. Fixes
crash bug in winbindd (and goodness knows where else).
Jeremy.

r20155: revert, I misread

r20154: Fix nasty typo, hunting another bug

r20151: remove meaningless checks

r20150: better memory handling for some functions, make sure we don't
leak memory by using the wrong(long lived)  mem context

r20148: Forgot to merge. Ensure open_any_socket_out()
is safe from signals.
Jeremy.

r20146: Now online checks are fully async we can do them
every cache timeout times.
Jeremy.

r20140: Make online/offline detection completely asynchronous.
Now I've done this I might be able to reduce the probe
timeout and reduce the backoff algorithm, going back
to checking every cache time seconds (5 mins by default),
as the parent or forked domain child will never block.
Jeremy.

r20136: Fix #4290.  Properly compute time to password expiration in message from
pam_winbind.  Thanks to Andrew Benham <andrew.benham@thus.net>

r20133: get rid of defined but not used warning - static function only used
inside the #ifdef HAVE_NATIVE_ICONV

r20132: get rid of defined but not used warning - static function only used
inside the #ifdef HAVE_KRB5

r20131: get rid of a few no previous prototype warnings

r20124: clean up nested extern declaration warnings

r20119: Update help info indicating how to use separators (forward slash only)
and properly use backslashes in "net ads join computername="

r20118: Fix some more warnings

r20117: 1st Error in the merge

r20116: Start merging in the work done to create the new idmap subsystem.

Simo.

r20110: Fix interaction between paranoid malloc checker
and lib/replace. Found by Herb - thanks !
Jeremy.

r20098: Properly fix issues with create_token_from_username()
reported by James. Ensure that this function allocates
everything on the temporary context except the return
memory. Never call this with a null mem context, and
now use conn->mem_ctx instead in smbd/service.c.
Remove separate free functions for conn->ngroups
and conn->nt_user_token as they are now always
talloc'ed off the conn->mem_ctx. Future optimization
will be to remove conn->mem_ctx and make all objects
pointed to in the conn struct talloc'ed off conn itself.
Easy to free then :-).
Jeremy.

r20097: Ensure found_username is freed correctly when
talloced on the null context.
Jeremy.

r20090: Fix a class of bugs found by James Peach. Ensure
we never mix malloc and talloc'ed contexts in the
add_XX_to_array() and add_XX_to_array_unique()
calls. Ensure that these calls always return
False on out of memory, True otherwise and always
check them. Ensure that the relevent parts of
the conn struct and the nt_user_tokens are
TALLOC_DESTROYED not SAFE_FREE'd.
James - this should fix your crash bug in both
branches.
Jeremy.

r20089: Put gpfs acl function into vfs_gpfs module.  Thanks to Gomati Mohanan
<gomati.mohanan@in.ibm.com>.

Also fix fields for sec_desc differences between 3.0 and 3.0.24 in
nfs4_acls.c.

r20087: Ensure we clean up any random pending events we
may have inherited from our parent in the winbindd
forked child.
Jeremy.

r20082: When fork_domain_child is forked when we're offline
the child inherits *all* active check_online timout handlers.
This is bad when it's not our domain (ie. BUILTIN).
Jeremy.

r20069: Fix logic bug I introduced

r20063: Fix a missing protoype warning in freelistcheck.c

r20060: Fix the timout calculation.
Jeremy.

r20058: Ensure we actually do the increasing time
calculation when in offline mode.
Jeremy.

r20057: Attempt to fix connect timeouts when connected on
a network but not one on which any home DC's can
be found (hotel network problem). Still testing
but this is getting close.
Jeremy.

r20054: Revert my changes, Simo promised to work on it :-)

r20050: Remove a bogus NULL check, LTDB_OBJECTCLASS is a static string

r20049: Reformatting

r20048: Fix vfs_full_audit after Jims kernel_flock addition

r20046: Add ldb_search_exp_fmt and port comment to 4

r20037: Reduce code size slightly by shuffling stuff around

r20036: Merge ldb_search_exp_fmt -- Thanks simo

r20035: Fix obvious horrible bug in falling back to MS-RPC
methods.
Jeremy.