r20402: Fix spelling: samba bug #4292 debian #402392

r20394: This is a *VERY* early start of my work on notify.

Checking in because Jeremy was bugging me. Potentially this becomes quite
intrusive, I'm not sure if I should open a temporary branch for this.

Jeremy, Jerry, do you think 3_0 is the right place for this?

Volker

r20393: Debug level 1 is a bit excessive here

r20389: this needs to be after AC_LIBREPLACE_BROKEN_CHECKS :-)

metze

r20388: add the library that contains dlopen to the global LIBS as it was before

metze

r20386: remove unneeded dlopen related configure checks and includes
this is done by libreplace...

this hopefully fixes the build on HPUX 11.00

metze

r20385: merge from samba4:

only try to find dlfcn.h if the dlopen symbol was found,
it hopefully fixes systems where dlfcn.h but no library with dlopen

metze

r20372: Make NTrename the same as SMBmv w.r.t. wildcards.
Jeremy.

r20371: In renames (via SMBmv) both src and dest can contain
wcards. I had forgotten this and had refused to allow
dest to contain wcards. We now pass all the normal
Samba4 smbtorture RAW-RENAME tests.
Jeremy

r20370: Now we've gone to the trouble of getting an NT status
from open_file_ntcreate make sure we return it on rename
error.
Jeremy.

r20368: Fix valgrind report of uninitialized memory
being returned (false positive). Up debug
level on "open_directory" message.
Jeremy.

r20363: Fix any possible valgrind errors in srvstr_get_XX or srvstr_pull_XX
by ensuring we pass in a valid src_len (or zero when appropriate).
Volker is correct in that this is a *horrible* interface and he is
now free to generally clean it up everywhere :-). Go for it Volker !
Jeremy.

r20362: Fix valgrind issues where we may read params
before checking length. Found by Volker.
Jeremy.

r20361: Prevent strnlen_w reading beyond max. Valgrind
found by Volker.
Jeremy

r20356: Consolidate the calls to parent_dirname() per open to one.

This involved passing the dirname as argument to a few routines instead of
calling parent_dirname() deep down.

Volker

r20355: Fix some C++ warnings

r20345: Simplify call_nt_transact_create by using tmp_talloc_ctx()

r20344: Move the calls to inherit_access_acls and change_owner_to_parent together.

Jeremy, I'm 100% sure you watch me closely here, I count on you :-)

Volker

r20343: Split change_owner_to_parent fd-based (for file opens) and a name-based (for
directory opens) routines.

Volker

r20342: Move the lstat into mkdir_internal, for the FILE_OPEN case in open_directory
we want to follow symlinks.

Volker

r20340: Join vfs_MkDir to its only caller

r20339: Make mkdir_internal static to open.c

r20338: Restructure open_directory a bit. This gets rid of a race condition regarding
error messages: We relied upon a stat that a directory did not exist to later
on then do the mkdir or not. This does the mkdir directly and copes with a
potential error.

The second one is more important: It's possible with Samba 3 to do a
ntcreate&x with NTCREATEX_OPTIONS_DIRECTORY and we happily do a NT_STATUS_OK.

Also move up the use_nt_status() logic a bit. I think this does not belong
into the core routines, the smb server as such should take care of it.

Jeremy, do you think this should go to 3.0.24?

I'll update samba4torture when the build farm has picked up this checkin.

Volker

r20336: Fix some memleaks in trans2open and simplify trans2mkdir a bit: We now have
tmp_talloc_ctx().

Volker

r20335: Port oplock return logic from ntcreate&x to NT_TRANSACT_CREATE. Fixes parts of
raw-open.

Volker

r20334: Activate RAW-MKDIR

r20333: Fix a couple of Coverity errors in one run, this was a potential NULL dereference

r20330: And here's the fix for the parent winbindd crashing
after it's child died unexpectedly whilst the parent
was waiting for a reply. We need to clean up the request
we're not going to service, plus we still need to call
the continuation function with a "False" flag so it
can clean things up. Still testing this, but I think
I'm right.
Jeremy

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>