dbwrap ctdb: fix a DEBUG message.

Michael

idmap tdb2: fix broken logic in tdb2_delete_bystring().

1. use the return value that idmap_tdb2_open_perm_db() gives us
2. don't delete frep the local db if deleting from the perm db failed.
3. fix wrong interpretation of return value of the local delete

Michael

idmap tdb2: fix broken logic in tdb2_store_bystring().

1. use the return value that idmap_tdb2_open_perm_db() gives us
2. don't write to the local db if writing to the perm db failed.
3. fix wrong interpretation of return value of the local store

Michael

dbwrap ctdb: don't retry when tdb_store failed in db_ctdb_persistent_store().

Only retry when ctdbd_persisten_update() failed.

Michael

dbwrap ctdb: add a partial mapping from tdb_error to NTSTATUS and use it for store.

Michael

packaging(RHEL-CTDB): Bump release number to ctdb.41 for the next RPM.

Michael

dbwrap ctdb: add db_ctdb_delete_persistent() and use it for persistent DBs

as delete_rec operation from fetch_locked()

Michael

dbwrap ctdb: call db_ctdb_store() in db_ctdb_delete().

to reduce code duplication.

Michael

dbwrap ctdb: add a retry loop to the persistent store operation.

This is because ctdbd can fail in performing the persistent_store
due to race conditions, and this does not mean it can't succeed
the next time.

To not loop infinitely, this makes use of a new parametric option:
"dbwrap ctdb:max store retries" (integer) which defaults to 5
and sets the upper limit for the number or repeats of the
fetch/store cycle.

Michael

dbwrap ctdb: release the lock before calling ctdbd_persistent_store()

in the persistent db_ctdb_store operation.

This is to prevent deadlocks in db_ctdb_persistent_store().

There is a tradeoff: Usually, the record is still locked
after db->store operation. This lock is usually released
via the talloc destructor with the TALLOC_FREE to
the record. So we have two choices:

- Either re-lock the record after the call to persistent_store
  or cancel_persistent update and this way not changing any
  assumptions callers may have about the state, but possibly
  introducing new race conditions.

- Or don't lock the record again but just remove the
  talloc_destructor. This is less racy but assumes that
  the lock is always released via TALLOC_FREE of the record.

I choose the first variant for now since it seems less racy.
We can't guarantee that we succeed in getting the lock
anyways. The only real danger here is that a caller
performs multiple store operations after a fetch_locked()
which is currently not the case.

Michael

dbwrap ctdb: remove erroneously duplicated comment.

Michael

packaging(RHEL-CTDB): Bump release number to ctdb.40 for the next RPM.

Michael

packaging(RHEL-CTDB): Bump release number to ctdb.39 for the next RPM.

Michael

Use transaction start/cancel for persistent writes to avoid leaving the database in an inconsistent state if we crash during the operation

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

packaging(RHEL-CTDB): bump Release number to ctdb.37 for next rpm.

Michael

packaging(RHEL-CTDB): Bump release number to ctdb.37 for the next RPM.

Michael

packaging(RHEL-CTDB): bump Release number to ctdb.36 for next rpm.

Michael

packaging(RHEL-CTDB): Bump release number to ctdb.35 for the next RPM.

Michael

Add a gpfs_prefetch module

This can not go upstream yet because it uses the non-GPL libgpfs. So it will
not be compiled by default and will not be included in the SOFS RPMs. But upon
Sven's request, we include it in the git tree and the source RPMs, so that it
can be built for in-house tests.

packaging(RHEL-CTDB): bump Release number to ctdb.34 for the next RPM.

Michael

packaging(RHEL-CTDB): update the build instructions in the README.

Michael

packaging(RHEL-CTDB): bump Release to ctdb.33 for the next build

Michael

packaging(RHEL-CTDB): remove the libnss_wins modules to reduce deps.

Michael

bump version number to 32

Correctly return NULL from messaging_ctdbd_connection if messaging_init failed

Exit smbstatus cleanly if messaging_init fails

Package 32-bit versions of libtalloc into the winbind rpm

Fix README: DOCS_DIR is replaced by DOCS_TARBALL

Bump up version number

packaging(RHEL-CTDB): use /usr/%{_libarch} instead of %{_libdir}

To fix creation of 32bit packges on x86_64

Michael

packaging(RHEL-CTDB): remove duplicate block from spec file.

Michael

packaging(RHEL-CTDB): fix placement and link direction of new shared libs

libwbclient, libtalloc, libtdb, and libnetapi are now installed
into /usr/lib{,64} by their SONAME. The symlink is made from
the .so files to point to the SONAME files (e.g., libtdb.so.1).

Michael

packaging(RHEL-CTDB): bump rpm release to the next version ctdb.29

Michael

packaging(RHEL-CTDB): add "service winbind condrestart" to "%post common"

Michael

packaging(RHEL-CTDB): use "/sbin/service smb" instead of "%{initdir}/smb"

Michael

packaging(RHEL-CTDB): ignore errors from init scripts in pre/post scripts.

Michael

packaging(RHEL-CTDB): in %preun, first stop the service then deactivate

Michael

packaging(RHEL-CTDB): use condrestart instead of restart in %post script

Michael

packaging(RHEL-CTDB): move service smb restart from %postun to %post

What would be the use of restarting a service that has just
been uninstalled...

Michael

packaging(RHEL-CTDB): move "chkconfig --del winbind" to "%preun common"

remains of winbind are being removed from the main package.
also stop the winbind service in the "%preun common" script.

Michael

packaging(RHEL-CTDB): remove winbind init script and its linke from main pkg

Winbind along with its libraries and init script is packaged in the
samba-common rpm.

Michael

packaging: bump the rhel-ctdb rpm releas number to ctbd.28

Michael

Do not call testparm in /etc/init.d/smb

For SOFS, we do know that we have netbios disabled, and testparm in the ctdb
shutdown event locks up due to a locked registry.tdb

keep compatibility with v3-0-ctdb name for fileid:mapping option

update configure.rpm for clustered Samba usage

Add winbind:online check timeout parameter

This is a band-aid for the rather convoluted offline/online mess in winbind
right now. Winbind re-uses the offline functionality that is targeted at domain
client installations on laptops to not overload disfunctional DCs. It uses the
winbind cache timeout as the retry timeout after a DC reboot.

I am using a parametric options because when this mess is cleaned up, that
parameter needs to go away again.

I'd recommend to use something like

winbind:online check timeout = 30

in typical LAN environments. This means a reconnect is attempted every 30
seconds.

Volker

Attempt to fix bug 5684

With the ctdb checkin dde9f3f006 tdb optimized out write lock checks for
write-enabled transaction. Sadly, this also removed the possibility to ever
remove dead records left over from tdb_delete calls within a transaction.

Tridge, please check this! Did dde9f3f006 have any reason beyond performance
optimizations?

Thanks,

Volker

Attempt to fix the build on Irix CC
(cherry picked from commit 666bf8456ac44cbbbd5524af2bf4fd89e18ddf62)

Merge from Herb. print correct test status

Fix bug 5686 - libsmbclient segfaults with more than one SMBCCTX.
Here is a patch to allow many subsystems to be re-initialized. The only
functional change I made was to remove the null context tracking, as the memory
allocated here is designed to be left for the complete lifetime of the program.
Freeing this early (when all smb contexts are destroyed) could crash other
users of talloc.
Jeremy.

use variables for files used in multiple places
(cherry picked from commit cee044bc42d955c535dbb6bb372af01089d37756)

Make sure to always set errno on error path in OpenDir (and hence scan_directory).

Michael

Fix unix_convert() for "*" after changing map_nt_error_from_unix().

map_nt_error_from_unix() now assumes that it is called in
an error path and returns an error even for a given errno == 0.
The original behaviour of unix_convert() used the mapping
of errno == 0 ==> NT_STATUS_OK to return success through
an error path.

I think this must have been an oversight, and unix_convert() worked
only by coincidence (or because explicitly using the knowledge
of the conceptually wrong working of map_nt_error_from_unix().

This patch puts this straight by not interpreting errno == 0
as an error condition and proceeding in that case.

Jeremy - please check!

Michael

I found lots of places where we assume error will be set when calling
one of our virtualised functions, such as db_open(), but error is only
set when a system call fails, and it is not uncommon for us to fail a
function internally without ever making a system call. That led to us
passing back success when a function had in fact failed.

I found two places where we relied on map_nt_error_from_unix()
returning success when errno==0, but lots and lots of places where we
relied on the reverse, so I fixed those two places.

map_nt_error_from_unix() will now always return an error, returning
NT_STATUS_UNSUCCESSFUL if errno is 0
(cherry picked from commit 69d40ca4c1af925d4b0e59ddc69ef8c26e6501d1)

ensure we give an error code to any routines above that are looking
for one

cifs.upcall: negatively instantiate keys on error

When a request-key upcall exits without instantiating a key, the kernel
will negatively instantiate the key with a 60s timeout. Older kernels,
however seem to also link that key into the session keyring. This
behavior can interefere with subsequent mount attempts until the
key times out. The next request_key() call will get this negative key
even if the upcall would have worked the second time.

Fix this by having cifs.upcall negatively instantiate the key itself
with a 1s timeout and don't attach it to the session keyring.

Signed-off-by: Jeff Layton <jlayton@redhat.com>

Remove two unused variables

Do not create a new mapping if a domain with an explicit config fails

Make the docs actually build

Thanks to Karolin for the friendly build service :-)

Document idmap rewrite

Some doxygen comments for idmap

Fix prototypes

Remove "idmap alloc config : range" parameter

This was overwritten by "idmap uid/gid" anyway. These are now the range
parameters for the alloc backend.

idmap rewrite

Remove the multi-ID lookup code and the 3.2.0 version of idmap_cache

Directly call backends from idmap_[ugs]_to_[ugs]id

Move the gid2sid cache to the parent winbind process

Move the uid2sid cache to the parent winbind process

WHATSNEW: fix one occurrence of 3.2.0 to say 3.3.0.

Michael

libnetjoin: support kerberized joining/unjoing (fix #5416).

Guenther

netapi: add NetLocalGroupSetMembers example code.

Guenther

netapi: add NetLocalGroupDelMembers example code.

Guenther

netapi: add NetLocalGroupAddMembers example code.

Guenther

netapi: implement NetLocalGroupSetMembers_r().

Guenther

netapi: implement NetLocalGroupDelMembers_r().

Guenther

netapi: implement NetLocalGroupAddMembers_r().

Guenther

netapi: add NetLocalGroup*Member calls to public headers.

Guenther

netapi: add skeleton for NetLocalGroup*Member calls.

Guenther

re-run make idl.

Guenther

netapi: add remaining NetLocalGroup*Member calls to IDL.

Guenther

netapi: add NetUserModalsGet and NetUserModalsSet tests.

Guenther

netapi: implement NetUserModalsSet_r.

Guenther

netapi: implement NetUserModalsGet_r.

Guenther

netapi: add example code for NetUserModalsGet and NetUserModalsSet.

Guenther

netapi: add NetUserModalsGet and NetUserModalsSet to public headers.

Guenther

netapi: add skeleton for NetUserModalsGet and NetUserModalsSet.

Guenther

re-run make idl.

Guenther

netapi: add NetUserModalsSet and NetUserModalsGet to IDL.

Guenther

doserr: add WERR_MEMBER_IN_ALIAS.

Guenther

netapi: add NetApiBufferAllocate.

Guenther

netapi: add ConvertStringSidToSid().

Guenther

netapi: generate the netapi testsuite makefile.

Guenther

fix build warning.

Guenther

Make events robust against their event_context being freed

fix smb_len calculation for chained requests

I think chain_reply() is one of the most tricky parts of Samba. This recursion
needs to go away, we need to sequentially walk the chain list.

Fix andx offset calculation for more than 2 chained requests

Untested code is broken code.... Test follows later, it's quite an intrusive
change to libsmb/

Remove an unused variable, process.c has its static copy

nmbd: add support for delayed initial samlogon packages.

The hosts or networks configured with "init logon delayed hosts"
have their initial samlogon packages (empty username) delayed
by the value configured with "init logon delay" (defaulting
to 100 milliseconds).

This gives the administrator some control over what clients would
consider the preferred logon server: they choose the server that
repsonds most quickly.

Michael

nmbd_packets: make queue_packet() public.

Michael

loadparm: add two parameters "init logon delay hosts" and "init logon delay"

"init logon delays hosts" takes a list of hosts names or addresses
or networks for which the initial SAMLOGON reply should be delayed
(so other DCs get preferred by XP workstations if there are any).
This option takes the same type of list as "hosts allow" does.

"init logon delay" allows one to configure the delay for the hosts
configured for delayed initial samlogon with "init logon delayed hosts".
The value is interpreted as milliseconds. The default value is 100.

This commit only introduces the parameters.
They will be activated in a subsequent commit.

Michael