r20844: Somewhat radical change - this may break the build (I will
watch carefully - so I'm doing it in one transaction so I can
roll back).

Change check_name(), reduce_name() and dptr_create() to
return NTSTATUS. This helps a lot in error path processing
and especially in reduce_name() allows us to ditch the flaky
and error-prone saving of errno and return errors directly.

Jeremy.

r20843: Get rid of last BOOL ok.
Jeremy.

r20842: Only one more BOOL ok to go...
Jeremy.

r20841: Remove more BOOL ok.
Jeremy.

r20840: Keep removing the old BOOL ok logic.
Jeremy.

r20839: Fix other C++ warnings

r20838: Small fix from Jiri.Sasek@Sun.COM to fix null
pointer deref.
Jeremy

r20837: Use real type name, to fix compilation with -WC++-compat

r20833: Reformatting

r20832: Remove extra pointers previously added to unique [out] pointers. Instead,
add [ref] pointers where necessary (top-level [ref] pointers,
by spec, don't appear on the wire).

This brings us closer to the DCE/RPC standard again.

r20828: Fix the build with libnscd.

Guenther

r20827: Increment the interface version.

Wilco: Existing modules should continue to work, they just must be recompiled.

Volker

r20824: Send access to the trusted domain passwords through the pdb backend, so that
in the next step we can store them in LDAP to be replicated across DCs.

Thanks to Michael Adam <ma@sernet.de>

Volker

r20820: merge from samba4:

- include system/aio.h
- use full prototype for main
- use ifdef instead if

metze

r20817: sync lib/replace with samba4

metze

r20811: fix comment

metze

r20796: Fix the same problem Jeremy has fixed (improper handling of deferred opens)
for delete_driver_files. Proper fix pending... :-)

Jeremy, please check.

Volker

r20789: merge from samba4:

- remove the epoll configure checks from libreplace

metze

r20781: Fix a couple of paths found by Volker where we weren't
correctly handling deferred opens.
Jeremy.

r20774: I thought I committed this before Xmas holidays ...
This change is needed to make it possible to not expire
caches in disconnected mode.

Jerry, please can you look at this and confirm it is ok?

Simo.

r20759: Dummy checkin to let the build farm pick up the new smbtorture4

r20745: Commit Finnish transalation of SWAT after review

r20744: Fix the build (I missed some chkpth -> checkpath renames).
Jeremy.

r20743: Remove another BOOL ok from reply_getatr.
Jeremy.

r20742: Rename chkpth -> checkpath for sanity's sake :-).
Start removing unneeded "BOOL ok" from this reply.c
(this logic is old, old, old..... :-).
Jeremy.

r20740: Get rid of some if-statements

r20738: Remove unused variable

r20725: Get rid of a bool passed down -- gd, please check

r20724: Get closer to passing RAW-OPEN. If the client is using
create disposition of FILE_CREATE then there's no need
to check can_delete_file_in_directory().
Jeremy.

r20722: RAW-CHKPATH should now pass, build farm should
go back to normal. Sorry about that.
Jeremy.

r20721: Fix the search unix_convert error returns. Only open
to go...
Jeremy.

r20720: Fix the chkpath problem, still looking at findfirst.
Jeremy.

r20718: Sync up the filename path parsing changes from SAMBA_3_0_24.
The only difference between the two trees now w.r.t file
serving are the changes to smbd/open.c in this branch I need
to review.
Jeremy.

r20717: Merge sharemode patch from Mathias Dietz <MDIETZ@de.ibm.com>.

The patch fixes the behaviour of GPFS sharemodes when
the access mask is no_access.

r20707: Clean up pdb_interface.c a bit -- patch from Michael Adam <ma@sernet.de>

r20700: In the main processing loop we select() on the fam socket. If it fires, it
might be possible that we hang in the receive_smb() although that socket is
not the reason for the select() to return.

This immediately reacts to the fam socket to become readable, and goes into
the select loop again. This fixes delays in files showing up in Windows.

Jeremy, James please review this and merge to 3_0_24 if appropriate.

Thanks,

Volker

r20699: This is an attempt to fix bug 4326.

James, can you please confirm this and merge it to 3_0_24? For me it fixes the
segfault.

Thanks,

Volker

r20695: Remove duplication of constants.
Jeremy.

r20694: To get this right we need to do signed 64-bit
comparisons here, not unsigned as we're eventually
casting into what it normall a signed 32 bit
value. Guenther please check (but I think I'm right here).
Jeremy.

r20692: Fix bug found by Guenther -

Just try to log on in offline mode without the fix: all accounts are expired,
although they are set to never expire in the PAC/info3.

NTTIME "Never" needs to get (time_t) -1.

We were casting a uint64 to time_t before
comparing, and we should have been doing it
the other way around.

Guenther please check this fixes things.

Jeremy.

r20690: fix a bug that causes smbd to 'hang' intermittently.

The problem occurs like this:

  1) running smbd as a domain member without winbindd

  2) client1 connects, during auth smbd-1 calls update_trustdom_cache()

  3) smbd-1 takes the trustdom cache timestamp lock, then starts
     enumerate_domain_trusts

  4) enumerate_domain_trusts hangs for some unknown reason

  5) other clients connect, all block waiting for read lock on trustdom
     cache

  6) samba is now hung

The problem is the lock, and really its just trying to avoid a race
where the cure is worse than the problem. A race in updating the
trutdom cache is not a big issue. So I've just removed the lock.

It is still an open question why enumerate_domain_trusts() can
hang. Unfortunately I've not in a position to get a sniff at the site
that is affected. I suspect a full fix will involve ensuring that all
the rpc code paths have appropriate timeouts.

r20687: Implement grace logons for offline authentications in pam_winbind.

In case a user authenticated sucessfully and his password just expired
while beeing disconnected, we should allow a user to logon (given a
clear warning). We currently forced the user into a password change
dialogue in that scenario; this did not make much sense while offline.

Guenther

r20674: Regenerate NDR parsers after pidl changes. This should fix a couple of
coverity warnings.

r20668: Also do not install the smbumount man page if we don't build smbumount.

r20666: Also do not install the smbmnt man page if SMBMOUNT_PROGS is not set.

r20664: Set the environment variable _before_ calling the shell.

r20663: Do not install smbmount man page if we didn't called configure
--with-smbmount.

r20654: Fix from Karolin Seeger <ks@sernet.de>: Accept "forever", "off" and "never" as
key words for -1 as policy values for net sam policy.

Volker

r20651: Fix "password expires soon" warning message for pam_winbind.

We were incorrectly calculating the days until the password expires and
we also need to look at the info3 pass_must_change_time for expiry
calculation.

Guenther

r20644: Make some functions static, thanks to Michael Adam <ma@sernet.de> for the hint.

r20636: Remove notify leftover

r20635: Add placeholder to ensure we don't go into the
wcard unlink if bad_path was set. The error
returned here is almost certainly incorrect
and will need testing properly with smbtorture,
but I don't want to forget about this path
(yes Volker I know this is currently incorrect :-).
Jeremy.

r20634: A *LOT* more work is necessary before touching notify remotely starts to make
sense. Until then, remove it from the tree to keep the diff between 3_0_24 and
3_0 small.

Volker

r20632: The extended RAW-SAMBA3BADPATH test led me to some wrong assumptions, in
particular the NT_STATUS_INVALID_PARAMETER thing was badly wrong. Remove the
changes based on it. Using gentest is much more effective in this respect, but
it will take a while to figure out the wildcard error handling of W2k3.

Volker

r20630: Fix some trivial memleaks.

Guenther

r20628: Looks bigger than it is. This is just re-indenting the if (dirname) that we've
taken care of above.

Volker

r20627: orig_name is not needed anymore, and slightly simplify logic by doing early
returns.

Volker

r20626: Fix a memleak found by the IBM checker -- not on 3.0.24

r20619: bad_path is handled somewhere else, so can_delete does not need it anymore

r20618: Fix a bug in bad_path handling that also exists in 3.0.23: For reply_unlink
under Linux we returned NT_STATUS_NOT_A_DIRECTORY. This is because in the
bad_path==True condition lstat(2) returns ENOTDIR and not ENOENT.

Not sure if we want to necessarily replicate the INVALID_PARAMETER here, but
this is what W2k3 does.

Jeremy, I tried to call you, but you were not around. So I'll leave it up to
you to merge this.

Volker

r20617: Do not notify if the delete failed

r20614: Dummy-checkin to trigger the build

r20612: A checkin I've wanted to do for ages: For bad_path==False set_bad_path_error
is equivalent to UNIX_ERROR. Expand the last reference. Lets see if I can kill
that as well :-)

Volker

r20611: All but one callers to set_bad_path_error have had the bad_path==True argument
being handled further up.

Volker

r20610: Fix a Solaris warning

r20608: ENOENT->NT_STATUS_OBJECT_NAME_NOT_FOUND is covered in the standard unix error mapping table

r20607: Next micro-step for reply_setatr

r20606: bad_path == True has been covered further up in reply_setatr.

I'm checking in micro-steps to make them independently checkable. This code
just very severely needs cleanup, but I don't want to break anything. So, be
patient with me, please :-)

Volker

r20605: Simplify logic in reply_setatr slightly

r20604: Fix two memleaks, Coverity ID 337, merge to 3_0_24

r20603: Slightly simplify logic

r20602: Fix coverity ID 339. SMB_ASSERT does not panic without -DDEVELOPER.

Volker

r20597: Survive some of the notify mask tests.

r20589: Re-add lost bad_path handling, sorry.

r20585: - allocate an OID range for LDB/LDAP extended operations

metze

r20578: - allocate an OID range for samba4 LDB/LDAP Controls
- allocate an OID range for conflicting attributes/classes

metze

r20558: Refactor rmdir_internals to use early termination rather
than lots of indented code. Change recursive_rmdir()
to return the smbd standards of True on success, False
on fail (it was doing the reverse, which was very confusing).
Jeremy.

r20546: Fix some IBM checker errors

r20544: Change copy_file() to return NTSTATUS. This is in preparation of turning
close_file() to NTSTATUS as well.

I'm not sure I got all the error codes right, but as I've never come across a
smb_copy() call in all my Samba work, I'm leaving it at that. If I'm
absolutely bored, I will write a thorough torture test.

As far as I can see, Samba4 even does not have a libcli implementation for
it...  :-)

Volker

r20542: Make close_remove_share_mode retun NTSTATUS. Not that anybody cares yet...

r20541: Fix some obvious glitches in wait_for_aio_completion. Hmmm. It seems
reply_close() at its heart is going to be an async call :-)

Volker

r20536: In the offline PAM session close case the attempt to delete a
non-existing krb5 credential cache should not generate an error.

Guenther

r20535: Fix make install to work when root doesn't have
write access into the source tree (like when your
directory is nfs mounted for example....).
Jeremy.

r20532: From Volker...

"smbcli_setattr("") always returns ACCESS_DENIED, test extension to follow.

Merge to 3.0.24?"

Yep (with slight optimization).

Jeremy.

r20530: Don't want this call to get inadvertently re-added :-).
Jeremy.

r20525: Closing a dir with del-on-close set requires the same as files: Don't actually
unlink/rmdir if another process still has it open.

Jeremy, this is a potential merger to 3.0.24.

Volker

r20524: Simplify logic of close_remove_share_mode().

Volker

r20512: Fix typo.

Guenther

r20507: smbcli_setattr("") always returns ACCESS_DENIED, test extension to follow.

Merge to 3.0.24?

Volker

r20503: And one more 64-bit warning

r20502: Activate RAW-RENAME, my Linux box passes this

r20501: Check in config.h fix from Michael Adam <ma@sernet.de>

r20500: Fix compiler warnings.

r20499: Activate RAW-OPEN in the build farm

r20498: Two 64-bit warnings

r20496: Some changes to make Samba3 the RAW-OPEN test. Checking in to both 3_0 and
3_0_24 because I was explicitly asked to, although this needs close
review. Jeremy, I'm sure you will check this thoroughly :-)

In reply_open_and_X the separate "size" variable kills the calculation of the
SPARSE flag returned to the client in the attrib field. In getpathinfo we do
it correctly, and RAW-OPEN (correctly) complains about the difference.

Add the "set the write time" to mknew and create.

For trans2open we were missing the "ofun == 0" ->
NT_STATUS_OBJECT_NAME_COLLISION case, and we dropped the status returned in
favor of ACCESS_DENIED once too many.

Last change is a fix to trans2open: We were returning the attributes given by
the client, not the attributes of the new file.

Volker

r20489: Missed patch ofthe forest_name patch for lookupname

r20488: When joined to a child domain in a multi-domain/single domain tree,
the child domain cannot always resolve SIDs in sibling domains.
Windows tries to contact a DC in its own domain and then the root
domain in the forest.  This async changes makes winbindd's name2sid()
call do the same.

r20487: Remove the unused dn2ad_canonical() call

r20486: Always upper case the "host/<sAMAccoutnName>" entry in the keytab file
so apps will know which one to look for,