| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix suggested by Andy Hanton <andyhanton@gmail.com>. The LOGIN_CACHE
struct contains two time_t entries, but was being written to and
read from via tdb_pack/tdb_unpack functions using explicit 32-bit int specifiers.
This would break on machines with a 64-bit time_t. Use correct int
sizes for tdb_pack/tdb_unpack.
We have to fix this properly before 2037 :-).
Contains fixes from master 627fb85092f728065b6d772c41aeb75018154e86
and 69fd8461b8792f4fee1b61db03953044565492c6.
Jeremy.
|
|
|
|
|
|
|
|
| |
See bug #6157 for more details.
metze
Signed-off-by: Stefan Metzmacher <metze@samba.org>
|
|
|
|
|
|
|
|
|
| |
Ok, that's a very long-standing one. I finally got around to install a recent
OpenLDAP and test the different variants of setting a NULL password etc.
Thanks all for your patience!
Volker
|
|
|
|
|
|
| |
With the previous code, the cache can never have been hit at all.
Michael
|
|
|
|
|
| |
This patch picks the alphabetically smallest one of the multi-value attribute
"uid". This fixes a regression against 3.0 and also becomes deterministic.
|
|
|
|
|
|
|
| |
This is aiming bug #6351.
Karolin
(cherry picked from commit c94d1cd7b1dc3ff99ae5a1eb9058ed6015fb9749)
|
|
|
|
|
|
|
| |
Invalid pointers were being dereferenced in lookup_sids causing
occasional seg faults.
Signed-off-by: Tim Prouty <tprouty@samba.org>
|
|
|
|
| |
this is a backport of f238809d236443b8968e1b4b197a55935c7c7e85 from master
|
|
|
|
|
| |
Zagrebin <alexz@visp.ru>.
Jeremy.
|
|
|
|
|
|
|
|
|
|
|
| |
This patch makes sure the original and temporary TDBs are closed
_before_ the rename. Originally, the open TDB was renamed, and so
the name passdb.tdb.tmp stayed around in the db context. Hence
upon client connect, the smbd children died because reinit_after_fork()
calling tdb_reopen_all() would try to reopen passdb.tdb.tmp which
existed no longer...
Michael
|
|
|
|
|
|
|
| |
3 but using a different hash calculation than 3.2.x passwd
databases (also version 3). Introduces a minor version
number.
Jeremy.
|
|
|
|
|
|
|
|
|
|
| |
correctly. For the clustering case.
Clustered setups should have only ever used
the unsigned version of TDB_DATA in the
first place so they can't be in this mess :-).
Just do the normal upgrade in the clustered case.
Jeremy.
|
|
|
|
|
|
|
| |
A client sent a SID with authority 0 and 0 sub-authorities. W2k3 replies with
NT_STATUS_INVALID_SID, even if other SIDs in the list are valid.
Thanks to Pavel <wylda@volny.cz> for the bug report!
|
|
|
|
|
|
|
|
|
| |
correctly.
This is a really nasty one to fix as in order to successfully update the
passdb.tdb we must do the equivalent of a tdbbackup to move to the new hash
values before we do the upgrade.
Jeremy.
|
| |
|
|
|
|
| |
Originally removed in be1dfff02d562e42a7847bd02fed8538630d3f41
|
|
|
|
|
|
|
|
|
|
| |
we end up returning the null sid instead of falling back to the legacy
code. Next time through the code we'll hit the negative cache and do
the right thing, but we still fail the first time.
If we fail the winbind id to sid mapping, call the legacy version. This
catches the case where we don't have a negative cache entry for the mapping.
This is better than returning the NULL sid to the caller.
|
|
|
|
| |
Jeremy.
|
|
|
|
| |
Jeremy.
|
| |
|
|
|
|
| |
Jeremy.
|
|
|
|
|
|
|
|
|
| |
winbind_idmap.tdb
This upgrades the TDBSAM_VERSION to 4 and SAMU_BUFFER_V4.
metze
(cherry picked from commit 7516ba860c160e8b971b6f5bc1036370169e6e8a)
|
|
|
|
|
|
|
|
| |
The upgrade is required because of the followin TDBSAM_VERSION
upgrade.
metze
(cherry picked from commit 671469ae7e3013809fe3995ec8f33dda9adda2ed)
|
|
|
|
|
|
|
|
| |
This marshalling isn't specific to tdbsam and it's
ugly to have the related functions in two different files.
metze
(cherry picked from commit 0544734309feea44e8d449737a9934231f63693d)
|
|
|
|
| |
Jeremy.
|
|
|
|
| |
Jeremy.
|
|
|
|
|
| |
attribute warn_unused_result"
Jeremy.
|
|
|
|
|
| |
Guenther
(cherry picked from commit 26139344fd0fac4fdd2a6752628b252fbd9b7450)
|
|
|
|
|
|
| |
This is an orthogonality measure to make clear this pointer now belongs to the
cache.
(cherry picked from commit e6080c6e87d6fe3995b121a772bf3f6343fa666f)
|
|
|
|
|
|
|
|
|
|
| |
talloc_autofree_context() instead of NULL.
Remove the code in memcache that does a TALLOC_FREE on stored pointers. That's a disaster waiting
to happen. If you're storing talloc'ed pointers, you can't know their lifecycle and they should
be deleted when their parent context is deleted, so freeing them at some arbitrary point later
will be a double-free.
Jeremy.
|
|
|
|
|
|
| |
on a problem found by Boyang. Only the pdb_nds backend implements login attempts so this was broken for tdbsam and ldap.
Jeremy.
|
|
|
|
|
|
|
| |
algoritmic allocator unconditionally, this is wrong an inconsistent.
Use samu_alloc_rid_unix() instead.
(If we create a new user we should do it in a way consistent with all
other commands like pdbedit -a and net rpc user add.)
|
|
|
|
| |
Jeremy.
|
|
|
|
| |
Jeremy.
|
|
|
|
|
| |
we just call the legacy function ?).
Jeremy.
|
|
|
|
|
|
| |
mappings.
Jeremy.
|
|
|
|
| |
thinking it was a failure of a transaction cancel
|
| |
|
|
|
|
| |
Karolin
|
|
|
|
| |
Jeremy.
|
|
|
|
| |
Jeremy.
|
| |
|
|
|
|
| |
Karolin
|
|
|
|
|
|
| |
retieve -> retrieve
Karolin
|
|
|
|
| |
This reverts commit cd8e63b2b45402091d6d328b3c6ca593fc19ac92.
|
|
|
|
| |
This reverts commit dcc39ed00453a075b23daece2844ca4817bfbfaf.
|
|
|
|
| |
This reverts commit 454cb852e06fa3d8bdd0eebb3ebdb24b3d74ecd0.
|
| |
|
| |
|
| |
|