| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
This fixes bug #6417
Michael
(cherry picked from commit e381c13b023f2b512b3f6aec133db9f323bc8132)
(cherry picked from commit 4ca03e3bb96518665c296ba2cf5aa1d91916897e)
|
|
|
|
|
|
| |
Michael
(cherry picked from commit 3fe9859342c28fe9da7011fb18a5fb5de8b29fa6)
(cherry picked from commit df4a0fabff06ea31149aac45d6477564cf96179b)
|
|
|
|
|
|
|
|
| |
This fixes bug #6416
Michael
(cherry picked from commit e12670a1053edf57af137026bd3fdb9fc7dfb0b2)
(cherry picked from commit b369902cddd55fab74ca6e0743e15e0f8cbfc4cc)
|
|
|
|
|
|
|
|
| |
This fixes bug #6415
Michael
(cherry picked from commit 3d3f39838261ddc401053dadcc5bd8e6317a3a8e)
(cherry picked from commit 34500d59b6f35de2c3d273d3523708ec22df59ce)
|
|
|
|
|
|
|
|
|
|
|
| |
When we run out of file descriptors for some reason, every new
connection forks a child that immediately panics causing smbd to
coredump. This seems unnecessarily harsh; with this code change we
now catch that error and merely log a message about it and exit
without the core dump.
Signed-off-by: Tim Prouty <tprouty@samba.org>
(cherry picked from commit 1c8f9892010ce8cc754089b25313c6bc8e622165)
|
|
|
|
|
|
|
|
|
|
| |
With the current infrastructure, we should not return error on
duplicate mappings but just warn instead (because an error would
trigger the attempt to create yet another mapping).
Michael
(cherry picked from commit 3111d78001f458cfcaf81123a1d1c23d5927a6c2)
(cherry picked from commit 5328f600bbc6535d8880b1b0c74bcfbd9b7a162a)
|
|
|
|
|
|
|
|
|
|
| |
With the current infrastructure, we should not return error on
duplicate mappings but just warn instead (because an error would
trigger the attempt to create yet another mapping).
Michael
(cherry picked from commit 35c3f4162d15f9846a645444e623178b78c52994)
(cherry picked from commit 751b6b07c5ea25809b1766a01fc859d580304ae9)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In winbind, we do multiple events in one select round. This needs fixing, but
as long as we're still using it, for efficiency reasons we need to do that.
What can happen is the following: We have outgoing data pending for a client,
thus
state->fd_event.flags == EVENT_FD_WRITE
Now a new client comes in, we go through the list of clients to find an idle
one. The detection for idle clients in remove_idle_client does not take the
pending data into account. We close the socket that has pending outgoing data,
the accept(2) one syscall later gives us the same socket.
In new_connection(), we do a setup_async_read, setting up a read fde. The
select from before however had found the socket (that we had already closed!!)
to be writable. In rw_callback we only want to see a readable flag, and we
panic in the SMB_ASSERT(flags == EVENT_FD_READ).
Found using
bin/smbtorture //127.0.0.1/tmp -U% -N 500 -o 2 local-wbclient
Volker
(cherry picked from commit bfeab3a0f621dbea50f43c98ba70b0ccd8323bff)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This fixes a crash bug hit when multiple mappings were found by
the ldap search. This crash was caused by an ldap asssertion
in ldap_next_entry because was set to NULL in each iteration.
The corresponding fix was applied to the idmap_ldap_sids_to_unixids()
by Jerry in 2007 (b066668b74768d9ed547f16bf7b6ba6aea5df20a).
This fixes the crash part of bug #6387.
There is a logic part, too:
The problem currently only occurs when multiple mappings are found
for one given unixid. Now winbindd does not crash any more but
it does not correctly handle this situation. It just returns the
last mapping from the ldap search results.
This needs fixing.
Michael
(cherry picked from commit e9010fa366746ec1ae948dbcf3493d446e23b14c)
Signed-off-by: Michael Adam <obnox@samba.org>
(cherry picked from commit 2b6dbddb9cc723fcbd2e4e22a9404d6b4ff805d7)
|
|
|
|
|
|
| |
Should make Solaris 10 builds look cleaner.
Jeremy.
(cherry picked from commit b5947b876f3c777e075879d305b6672a6c8d8abd)
|
|
|
|
|
|
|
|
|
|
| |
probing for them as shared modules.
idmap-gurus of the world, please check.
Guenther
(cherry picked from commit 67588ca80d654183b8b7b062b9660a506a825f94)
(cherry picked from commit 76e2e56e256695c1d52bd67d1b07bce0c7e0f37c)
|
|
|
|
|
|
| |
Guenther
(cherry picked from commit b5bec1a6d73f5939b306e157937d027a7286163c)
(cherry picked from commit c7d5e0a19057b6ad7301a390fc766bb438967eb2)
|
|
|
|
|
|
| |
Guenther
(cherry picked from commit d71dec9259366e99beca69fcd9397bd38ed82c71)
(cherry picked from commit 4e32c424a00bc29e63176af9773f9ffa7aa09bc2)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also remove ads_memfree(), which was only ever a wrapper around
SAFE_FREE, used only to free the DN from ads_get_ds().
This actually makes libgpo more consistant, as it mixed a talloc and a
malloc based string on the same element.
Andrew Bartlett
Signed-off-by: Günther Deschner <gd@samba.org>
(cherry picked from commit 20501876735fc4656083747ba8aa0de9da8fb820)
(cherry picked from commit 6673ad24d48ad44fa76497f0327b13e82721d0a1)
|
|
|
|
|
| |
(cherry picked from commit f4aec7b5907065a2f48d1b19065c6673aff6a190)
(cherry picked from commit 11b3f0d8deb629489de4de29cedf518adc86d196)
|
|
|
|
|
|
| |
Guenther
(cherry picked from commit 531af136f9dd5c6050f78948837294aed02de440)
(cherry picked from commit 91216ffa95b4ed53e54b11665e96b911cb4e4ab5)
|
|
|
|
|
|
| |
metze
(cherry picked from commit 93c2057c8b5a3976cda65a9d27dc4dbb9c5c550a)
(cherry picked from commit 00b65cc6ebcb4c67259e7fa618f6d961f7117396)
|
|
|
|
|
|
| |
metze
(cherry picked from commit 3b8dd79f2bc775ed94130565ec2c4383a4864348)
(cherry picked from commit ca31920af281ba9ea8f466ed0264118737a1b055)
|
|
|
|
|
|
| |
metze
(cherry picked from commit 0685031ccfc09feb0ad070df1c1a1d0cef5874f2)
(cherry picked from commit e270b4f0f62699469cbaad024c893fe84142a256)
|
|
|
|
|
| |
Thanks to François Legal <devel@thom.fr.eu.org> for reporting this bug
(cherry picked from commit a1cf98a2458a7f7a08531832bd29da3962ddafc9)
|
|
|
|
|
|
| |
Found in "make test" -- if we can't connect at all, "cli" is uninitialized
(cherry picked from commit 7735650f2ed74a72ddee272baf281e3fd9e7c42b)
(cherry picked from commit a6b0963d718d792f7b4cb218f6a070715ec7a091)
|
|
|
|
|
|
| |
password talloc'ed strings within the cli_struct.
Jeremy.
(cherry picked from commit 6be4bf17de47d1b468389de1225a72daa94e0119)
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
cli_setup_signing_state() with Undefined is a noop.
metze
|
| |
|
| |
|
| |
|
|
|
|
| |
Jeremy.
|
|
|
|
| |
Michael
|
|
|
|
|
|
| |
* Adds wbcGetSidAliases that calls the lookup_useraliases function.
* Updates wbinfo and winbind_util.c to call the new function.
* Also added winbind_get_groups helper function.
|
|
|
|
|
|
|
|
| |
* Adds the plumbing required to lookup users by sid into winbind, wbinfo
and smbd helper lib (winbind_util.c).
* Removes some double declarations of winbind_util.c functions.
* Bumps the winbind protocol version to 21 and the minor version of
wbclient to 3.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
necessary."
This reverts commit 487f5e7b4768cfe9e511b0ba56f16c411e21f702.
I was confused about the real meaning of find_domain_from_name_noinit()
vs. find_domain_from_name(). We don't need the connection established
here, just the domain struct which gets initialized by rescan_trusted_domains().
Sorry for the noise.
Michael
|
|
|
|
|
|
|
| |
Calling find_domain_from_name_noinit() might not be enough here.
This makes winbindd_getpwent() behave the same as winbindd_getgrent().
Michael
|
|
|
|
|
|
|
| |
find_domain_from_name_noinit() is no longer called only for
name alias support.
Michael
|
|
|
|
|
| |
This also switches wb_reqtrans to use wbcErr instead of NTSTATUS as it would
be pointless to convert to errno first and to wbcErr later.
|
|
|
|
| |
Michael, please check!
|
|
|
|
| |
Michael
|
|
|
|
| |
Michael
|
|
|
|
| |
Michael
|
|
|
|
|
|
| |
adding tmp talloc ctx and removing a variable
Michael
|
|
|
|
| |
Michael
|
|
|
|
| |
Michael
|
|
|
|
| |
Michael
|
|
|
|
| |
Michael
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
lp_use_kerberos_keytab parameter.
The first is "kerberos method" and replaces the "use kerberos keytab"
with an enum. Valid options are:
secrets only - use only the secrets for ticket verification (default)
system keytab - use only the system keytab for ticket verification
dedicated keytab - use a dedicated keytab for ticket verification.
secrets and keytab - use the secrets.tdb first, then the system keytab
For existing installs:
"use kerberos keytab = yes" corresponds to secrets and keytab
"use kerberos keytab = no" corresponds to secrets only
The major difference between "system keytab" and "dedicated keytab" is
that the latter method relies on kerberos to find the correct keytab
entry instead of filtering based on expected principals.
The second parameter is "dedicated keytab file", which is the keytab
to use when in "dedicated keytab" mode. This keytab is only used in
ads_verify_ticket.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
"getent group" used to fill the idmap cache with negative
cache entries for unmapped group sids.
Don't pass domain name unconditionally to idmap_sid_to_gid().
idmap_sid_to_gid() only creates new mappings (allocating
idmap backends tdb, tdb2, ldap...) when the domain name passed
in is "".
Note that it is _wrong_ to directly call the idmap_sid_to_gid()
functions here, in the main winbindd. The correct fix would be
to send a sid_to_gid request to winbindd itself, but this needs
more work to prepare the async mechanisms, and we nee a quick
fix for getent passwd now.
Michael
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
"getent passwd" used to fill the idmap cache with negative
cache entries for unmapped user sids.
Don't pass domain name unconditionally to idmap_sid_to_[ug]id().
idmap_sid_to_[ug]id() only creates new mappings (allocating
idmap backends tdb, tdb2, ldap...) when the domain name passed
in is "".
Note that it is _wrong_ to directly call the idmap_sid_to_[ug]id()
functions here, in the main winbindd. The correct fix would be
to send a sid_to_[ug]id request to winbindd itself, but this needs
more work to prepare the async mechanisms, and we nee a quick
fix for getent passwd now.
Michael
|