1
0
mirror of https://github.com/samba-team/samba.git synced 2024-12-27 03:21:53 +03:00
Commit Graph

18274 Commits

Author SHA1 Message Date
Jim McDonough
700c3ab1c9 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.
(This used to be commit 799967c668)
2007-10-10 12:17:04 -05:00
Volker Lendecke
dbea3a2b6f r20707: Clean up pdb_interface.c a bit -- patch from Michael Adam <ma@sernet.de>
(This used to be commit d78c18a9fa)
2007-10-10 12:17:04 -05:00
Volker Lendecke
7a915ec3a6 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
(This used to be commit c846153b2e)
2007-10-10 12:17:04 -05:00
Volker Lendecke
9343350102 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
(This used to be commit a841b7659b)
2007-10-10 12:17:04 -05:00
Jeremy Allison
b86d6b75c0 r20695: Remove duplication of constants.
Jeremy.
(This used to be commit 66cca893e1)
2007-10-10 12:17:03 -05:00
Jeremy Allison
ec43849134 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.
(This used to be commit 31f8e0edc0)
2007-10-10 12:17:03 -05:00
Jeremy Allison
5a2585416c 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.
(This used to be commit f4e898b622)
2007-10-10 12:17:03 -05:00
Andrew Tridgell
fc86057354 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.
(This used to be commit ab8d410533)
2007-10-10 12:17:02 -05:00
Günther Deschner
0d538f7370 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
(This used to be commit 668b278653)
2007-10-10 12:17:02 -05:00
Jelmer Vernooij
b79629fcc4 r20674: Regenerate NDR parsers after pidl changes. This should fix a couple of
coverity warnings.
(This used to be commit 5f86d006de)
2007-10-10 12:17:02 -05:00
Lars Müller
c7f59d48c1 r20668: Also do not install the smbumount man page if we don't build smbumount.
(This used to be commit 82ce272b24)
2007-10-10 12:17:02 -05:00
Lars Müller
1e8cbe46e0 r20666: Also do not install the smbmnt man page if SMBMOUNT_PROGS is not set.
(This used to be commit aa122be8f7)
2007-10-10 12:17:01 -05:00
Lars Müller
99c1e8f262 r20664: Set the environment variable _before_ calling the shell.
(This used to be commit 9cecd62654)
2007-10-10 12:17:01 -05:00
Lars Müller
83a41eb3a7 r20663: Do not install smbmount man page if we didn't called configure
--with-smbmount.
(This used to be commit ee12348cf1)
2007-10-10 12:17:01 -05:00
Volker Lendecke
b5e262dbd0 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
(This used to be commit b6347751d8)
2007-10-10 12:17:01 -05:00
Günther Deschner
062a1a4904 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
(This used to be commit 22d7923712)
2007-10-10 12:17:01 -05:00
Volker Lendecke
69272c9b04 r20644: Make some functions static, thanks to Michael Adam <ma@sernet.de> for the hint.
(This used to be commit f1f5d035db)
2007-10-10 12:17:01 -05:00
Volker Lendecke
7050da81fe r20636: Remove notify leftover
(This used to be commit a174b49173)
2007-10-10 12:17:00 -05:00
Jeremy Allison
b538c2b86b 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.
(This used to be commit 06e20c2878)
2007-10-10 12:17:00 -05:00
Volker Lendecke
46fdae1b6b 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
(This used to be commit f146a85e74)
2007-10-10 12:17:00 -05:00
Volker Lendecke
87bd4d2723 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
(This used to be commit 58b8a242a7)
2007-10-10 12:17:00 -05:00
Günther Deschner
7f88a7c948 r20630: Fix some trivial memleaks.
Guenther
(This used to be commit 3bb94a0818)
2007-10-10 12:17:00 -05:00
Volker Lendecke
45e7b954cd r20628: Looks bigger than it is. This is just re-indenting the if (dirname) that we've
taken care of above.

Volker
(This used to be commit 5e1da363bb)
2007-10-10 12:16:59 -05:00
Volker Lendecke
b5cad2c2a7 r20627: orig_name is not needed anymore, and slightly simplify logic by doing early
returns.

Volker
(This used to be commit 614651c6a7)
2007-10-10 12:16:59 -05:00
Volker Lendecke
0e4ab9a5ac r20626: Fix a memleak found by the IBM checker -- not on 3.0.24
(This used to be commit 9d0a789e32)
2007-10-10 12:16:59 -05:00
Volker Lendecke
dbac39f74e r20619: bad_path is handled somewhere else, so can_delete does not need it anymore
(This used to be commit 1798987128)
2007-10-10 12:16:59 -05:00
Volker Lendecke
1b50bcc9c8 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
(This used to be commit b1edc3d053)
2007-10-10 12:16:59 -05:00
Volker Lendecke
a9e3624bbc r20617: Do not notify if the delete failed
(This used to be commit c86fbdf8cc)
2007-10-10 12:16:59 -05:00
Volker Lendecke
5740f6910e r20614: Dummy-checkin to trigger the build
(This used to be commit 7e444e1612)
2007-10-10 12:16:58 -05:00
Volker Lendecke
d10af02226 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
(This used to be commit 794d5d38f1)
2007-10-10 12:16:58 -05:00
Volker Lendecke
283c690e6e r20611: All but one callers to set_bad_path_error have had the bad_path==True argument
being handled further up.

Volker
(This used to be commit 358673c366)
2007-10-10 12:16:58 -05:00
Volker Lendecke
c78623e905 r20610: Fix a Solaris warning
(This used to be commit 60d7b349db)
2007-10-10 12:16:58 -05:00
Volker Lendecke
90149315ea r20608: ENOENT->NT_STATUS_OBJECT_NAME_NOT_FOUND is covered in the standard unix error mapping table
(This used to be commit 816c19aaeb)
2007-10-10 12:16:58 -05:00
Volker Lendecke
72ecfaf32c r20607: Next micro-step for reply_setatr
(This used to be commit cea9a01434)
2007-10-10 12:16:58 -05:00
Volker Lendecke
21cc389545 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
(This used to be commit c16775486b)
2007-10-10 12:16:57 -05:00
Volker Lendecke
ccc54da805 r20605: Simplify logic in reply_setatr slightly
(This used to be commit f19db8f33e)
2007-10-10 12:16:57 -05:00
Volker Lendecke
395d304f22 r20604: Fix two memleaks, Coverity ID 337, merge to 3_0_24
(This used to be commit 44f9d25a90)
2007-10-10 12:16:57 -05:00
Volker Lendecke
3ab9f619d8 r20603: Slightly simplify logic
(This used to be commit e4dea0e647)
2007-10-10 12:16:57 -05:00
Volker Lendecke
c79be5713a r20602: Fix coverity ID 339. SMB_ASSERT does not panic without -DDEVELOPER.
Volker
(This used to be commit 33b5950bec)
2007-10-10 12:16:57 -05:00
Volker Lendecke
b2efff8fe5 r20597: Survive some of the notify mask tests.
(This used to be commit e4a2e63272)
2007-10-10 12:16:57 -05:00
Volker Lendecke
bb357a1617 r20589: Re-add lost bad_path handling, sorry.
(This used to be commit 244f96388c)
2007-10-10 12:16:56 -05:00
Jeremy Allison
0a8e3fd236 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.
(This used to be commit 27dc5bebda)
2007-10-10 12:16:56 -05:00
Volker Lendecke
18bf0b29b4 r20546: Fix some IBM checker errors
(This used to be commit 8f67efd582)
2007-10-10 12:16:56 -05:00
Volker Lendecke
1307337aaf 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
(This used to be commit 5ebdf02ba1)
2007-10-10 12:16:56 -05:00
Volker Lendecke
84da72860e r20542: Make close_remove_share_mode retun NTSTATUS. Not that anybody cares yet...
(This used to be commit c899577153)
2007-10-10 12:16:55 -05:00
Volker Lendecke
f933a9abc2 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
(This used to be commit 36f60407ae)
2007-10-10 12:16:55 -05:00
Günther Deschner
f3a85fb152 r20536: In the offline PAM session close case the attempt to delete a
non-existing krb5 credential cache should not generate an error.

Guenther
(This used to be commit 11c6f573af)
2007-10-10 12:16:55 -05:00
Jeremy Allison
b1fed47440 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.
(This used to be commit 7dcb61d1b2)
2007-10-10 12:16:55 -05:00
Jeremy Allison
2b1760297f r20532: From Volker...
"smbcli_setattr("") always returns ACCESS_DENIED, test extension to follow.

Merge to 3.0.24?"

Yep (with slight optimization).

Jeremy.
(This used to be commit dcc7bca1ad)
2007-10-10 12:16:55 -05:00
Jeremy Allison
e5e0f51b83 r20530: Don't want this call to get inadvertently re-added :-).
Jeremy.
(This used to be commit 68c4fbcf33)
2007-10-10 12:16:54 -05:00