tsocket_bsd: return -1 and set errno to ENAMETOOLONG if the unix path is too long

metze

tsocket: remove prototype of non-existing tsocket_address_inet_set_broadcast()

metze

Fixed some missing flags and bugs in the security creation.

Also, added some logging. It needs improvement, possibly ability to
turn in on and off via configuration file.

Fixed a bug in object specific access checks.

s3: Remove debug_ctx()

smbd just crashed on me: In a debug message I called a routine preparing a
string that itself used debug_ctx. The outer routine also used it after the
inner routine had returned. It was still referencing the talloc context
that the outer debug_ctx() had given us, which the inner DEBUG had already
freed.

Added some dn to the info in the log messages.

Removed the default DACL from token, as we will not be using it.

Convert from numbers to correct SMB_FIND_XX constant names.
Jeremy.

s3:registry: add an extra check for dsize==0 to regdb_fetch_keys_internal()

Don't only rely on dptr == NULL.
I stumbled over this one when rewriting some of the dbwrap_ctdb code.

Michael

s3:registry: add safety check for return value of tdb_unpack to regdb_fetch_keys_internal()

Prevents segfaults in some situations.

(For a non existent or empty record, we sometimes rely on the fetch operation
 to return dsize==0 and sometimes we rely on dptr==NULL.)

Michael

s3:dbwrap_ctdb: add debug message to transaction_fetch_start()

for the case that another local process has started a transaction
bewteen releasing the transaction_lock record and starting the
transaction.

Michael

s3:dbwrap_ctdb: split combined check in two and add descriptive debug

in db_ctdb_transaction_fetch_start() for error conditions when re-fetching
the transaction_lock record inside the transaction

Michael

s3:dbwrap_ctdb: fix race condition with concurrent transactions on the same node.

In ctdb_transaction_commit(), when the trans2_commit control fails, there
is a race condition in the 1 second sleep between the local transaction_cancel
and the call to ctdb_replay_transaction(): The database is not locked, and
neither is the transaction_lock record. So another client can start and possibly
complete a new transaction in this gap, but only on the same node: The locking
of the transaction_lock record on a different node which involves migration of
the record to the other node has been disabled by introduction of the
transaction_active flag on the db which closes precisely this gap from the start
of the commit until the call to TRANS2_FINISH or TRANS2_ERROR.
But this mechanism does not cover the case where a process on the same node
tries to start a transaction: There is no obstacle to locking the transaction_lock
record because the record does not need to be migrated.

This commit closes this race condition in ctdb_transaction_fetch_start()
by using the new ctdb_ctrl_transaction_active() call to ask the local
ctdb daemon whether it has a transaction running on the database.
If so, the check is repeated until the running transaction is done.

This does introduce an additional call to the local ctdbd when starting
transactions, but it does close the (hopefully) last race condition.

Michael

s3:configure: add a check for the new CTDB_CONTROL_TRANS2_ACTIVE

Michael

s3:dbwrap_ctdb: add new db_ctdb_transaction_active() that calls CTDB_CONTROL_TRANS2_COMMIT

Michael

s3:dbwrap_ctdb: fix a race in starting concurrent transactions on a single node

There are two races in concurrent transactions on a single node.
One in starting a transaction and one with replay during commit.

This commit closes the first race by storing the client pid in the
transaction-lock record and comparing the stored pid against its own
pid after releasing the lock and refetching the record inside the
transaction.

Michael

s3:dbwrap_ctdb: use db_ctdb_ltdb_fetch() inside db_ctdb_transaction_fetch_start

Michael

s3:dbwrap_ctdb: use db_ctdb_ltdb_fetch() inside db_ctdb_transaction_fetch()

Michael

s3:dbwrap_ctdb: add a function db_ctdb_ltdb_fetch()

This fetches a record from the db and splits out the ctdb header.

Michael

s3:dbrwap_ctdb: add a function db_ctdb_ltdb_store()

and use it in db_ctdb_store() and db_ctdb_transaction_store().

Michael

s3:dbwrap_ctdb: reformat a comment slightly to enhance clearness.

Michael

Fix bug 6867 - trans2findnext returns reply_nterror(req, ntstatus) In a directory with a lot of files.
Jeremy.

s3: Fix a 100% CPU loop when ctdbd dies during a traverse

s3-gencache: restore gencache_get behavior with NULL args (with torture test).

Without this, we panic in wins_srv_is_dead() and fail to start nmbd with
wins support.

Volker, please check.

Guenther

s3: Make "debug hires timestamp" default to true

It does not cost much and can help a lot when debugging

s4:dsdb Fix up after the MAP_ constants became LDB_MAP_

s4:provision Remove LDB backend files in provision

Rather than try and remove the records in the LDB files, make the
provision remove the whole file.  This also removes the need to try
and carry forward the old ldb filenames.

Andrew Bartlett

s4:provision Split ProvisionBackend out of the main provision script

This splits the code, while keeping the original behaviour.  The
provision.py file had become just too long.

Andrew Bartlett

s4:provision Inline 'ldap_backend_shutdown' for clarity

s4:provision Fix samdb test with new provision code

s4:provision Move 'Schema' into it's own file

s4:provision Make 'linked_attributes' and 'dnsyntax_attributes' a property of the Schema

s4:provision Rework provision to always have a ProvisionBackend

Rather than treat the LDAP backend as a special case, treat all
backends the same, with different callbacks.

Andrew Bartlett

s4 - SID allocation using FDS DNA plugin

s4:dsdb - Removed redundant domain SID filter.

s4:dsdb - Store SID as string in FDS.

s4 - Mapped AD schema to existing FDS schema.

s4:dsdb - Fixed attribute dereferencing for FDS

Remove special case logic in 'samdb_relative_path'.

While this logic (avoiding to prefix a non-filename with a path) is
important in the code this was copied from (private_dir()), none of
the callers of this function need it.

Andrew Bartlett

s4:dsdb Revert back to using DN:filename in the partitions record

This allows us to change the escaping function without breaking
existing installs.  The new escaping function (used for new databases)
is RFC1738 URI encoding, except for the trivial cases without special
characters.

The new databases are also placed in a subdirectory, sam.ldb.d per an
earlier suggestion by metze.

Andrew Bartlett

lib/util Use rfc1738.c from Squid for all our URL encode/decode needs.

Andrew Bartlett

lib/util Add rfc1738 escape/unescape code from Squid

This is intended to replace our rfc1738_unescape(), and give us an
rfc1738_escape implementation (and hopefully is better tested and more
secure).

Andrew Bartlett

s4:credentials Put the 'secrets.keytab' in the same directory as secrets.ldb

This avoids trouble when the secrets.ldb is updated with ldbedit but
an smb.conf is not specified.

Andrew Bartlett

s4: Create a script for updating a running provision with change introduced since the initial provision

s4:torture/raw/samba3misc - Add "discard_const_p" macro before a string

s4:ldb Remove debug traces duplicated by the new generic trace code

s3:ldap: don't search when no values where found

uuid.c: Remove some dead code

s3: Fix the RPC server SUBSYSTEM declaration

If I read SMB_SUBSYSTEM right then the 2nd argument needs to be the file where
the static_init_rpc (in this case) is defined. This seems to have moved from
server.c to process.c.

Jelmer, please check!

Volker

s3:Makefile: add LIBREPLACE_LIBS for talloc, tdb and wbclient

ѕ3:buildsystem: fix depenencies for libreplace

This problem became visible after adding the picky -z defs linker option: On
Solaris libreplace had unresolved symbols, which showed up in the libtalloc
build. PAM_WINBIND_EXTRA_LIBS and WINBIND_NSS_EXTRA_LIBS had been workarounds
to make things work at two placeѕ. These variables have been obsoleted now.
This patch introduces LIBREPLACE_LIBS which contans the linker flags needed for
linking anything using libreplace.

ѕ3:ldap: search for account policies in objectclass sambaDomain, not *

s3/docs: Fix typo.

Fix bug #4250. Thanks to TAKAHASHI Motonobu <monyo@samba.gr.jp> for reporting!

Karolin

s3/docs: Remove reference to nonexistent file.

Fix bug #4252. Thanks to TAKAHASHI Motonobu <monyo@samba.gr.jp> for reporting!

Karolin

s4:nbt_server/wins/winsdb - Fix "const" warning

s4-smbtorture: skip bigendian tests against s3 in RPC-LSA-SECRETS.

Guenther

s3-rpcclient: add deletetrustdom command.

Guenther

s3-rpcclient: add createtrustdom command.

Guenther

s3-lsa: expand struct lsa_info to carry name and sd.

Guenther

s3-lsa: use switch in _lsa_QuerySecurity().

Guenther

s4-smbtorture: fix warning in RPC-WKSSVC torture test.

Guenther

s4:kdc/hdb-samba4 - Remove unused variable

Change RAW-STREAMS for s3 to allow rename over an existing stream,
we need to file a CAR to find out why Windows gives INVALID_PARAMETER
for this.
Jeremy.

Start fixing the RAW-STREAMS test - ensure that the xattr
used to store the stream info in streams_depot.so is not
seen in when enumerating EAs.
Jeremy.

s3: Fix incorrect rc check of nscd_flush_cache.

At least this only resulted in an incorrect debug message.

s3-secrets: use autogenerated code for TRUSTED_DOM_PASS struct parsing from a tdb.

Guenther

s3: add secrets.idl.

Guenther

tdb: update README a bit

While studying tdb, I've noticed a couple of mismatches between readme
and actual code:

- tdb_open_ex changed it's log_fn argument to log_ctx
- there is now no tdb_update(), which it seems was transformed into
  non-exported tdb_update_hash()

There were other mismatches, but I don't remember them now, sorry.

Signed-off-by: Kirill Smelkov <kirr@mns.spb.ru>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

tdb: add tests for double .close() in pytdb

The reason I do it is that when using older python-tdb as shipped in
Debian Lenny, python interpreter crashes on this test:

    (gdb) bt
    #0  0xb7f8c424 in __kernel_vsyscall ()
    #1  0xb7df5640 in raise () from /lib/i686/cmov/libc.so.6
    #2  0xb7df7018 in abort () from /lib/i686/cmov/libc.so.6
    #3  0xb7e3234d in __libc_message () from /lib/i686/cmov/libc.so.6
    #4  0xb7e38624 in malloc_printerr () from /lib/i686/cmov/libc.so.6
    #5  0xb7e3a826 in free () from /lib/i686/cmov/libc.so.6
    #6  0xb7b39c84 in tdb_close () from /usr/lib/libtdb.so.1
    #7  0xb7b43e14 in ?? () from /var/lib/python-support/python2.5/_tdb.so
    #8  0x0a038d08 in ?? ()
    #9  0x00000000 in ?? ()

master's pytdb does not (we have a check for self->closed in obj_close()),
but still...

Signed-off-by: Kirill Smelkov <kirr@mns.spb.ru>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

tdb: reset tdb->fd to -1 in tdb_close()

So that erroneous double tdb_close() calls do not try to close() same
fd again. This is like SAFE_FREE() but for fd.

Signed-off-by: Kirill Smelkov <kirr@mns.spb.ru>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

tdb: fix typo in python's Tdb.get() docstring

It's Tdb.get(), not Tdb.fetch().

Signed-off-by: Kirill Smelkov <kirr@mns.spb.ru>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

tdb: kill last bits from swig

We no longer use swig for pytdb, so there is no need for swig make
rules. Also pytdb.c header should be updated.

Signed-off-by: Kirill Smelkov <kirr@mns.spb.ru>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

Fix map readonly in smb.conf, it is a single word

Add in Metze's new code to ndr encode the user.DOSATTRIB
blobs. Next I'll change the create timestamp and dos attribute
code to use this.

Fix bug 6572 - libsmbclient: unable to access 'msdfs proxy' share.

s3:net: fix output of net rpc trustdom list

Move some messages from stderr to stdout. When some trusting DCs were
unreachable we lacked \n's on stdout which screwed up whole the output.

s3:configure: only check for gpfs_gpl.h

The header is everything we need in order to build vfs_gpfs.

metze

Signed-off-by: Michael Adam <obnox@samba.org>

s3-passdb: move open_schannel_session_store() to passdb/secrets_schannel.c.

Guenther

s3-net: acct_flags are uint32_t in net_sam_set_userflag().

Guenther

adssearch: dump some more nttime timestamps.

Guenther

s3-lsa: add lsa_trusted_domain_mapping.

Guenther

lsa: add LSA_TRUSTED_DOMAIN access masks.

Guenther

s3-passdb: add secrets_delete_generic().

Guenther

s3-lsa: add lsa_secret_mapping.

Guenther

lsa: add LSA_SECRET access masks.

Guenther

s3-lsa: use correct function name in_lsa_RemoveAccountRights().

Guenther

s3-lsa: pure cosmetic indentation fixes.

Guenther

s3-lsa: use enum lsa_LookupNamesLevel in lsa_lookup_level_to_flags().

Guenther

ldb:tdb backend - be also here more careful with the result value

"msg_delete_attribute" doesn't return an LDB result constant.

ldb:python bindings - make the intention by Jelmer ("int" vs. "enum") more clear

Revert "ldb python bindungs - better use the "enum ldb_scope" for the search scope rather than "int""

This reverts commit 4f8826ff7f4789c5b5f363b733a42053f72aa526.

Jelmer pointed out that the "enum"s don't work so well in combination with python.

s3/docs: Document new pdbedit -K option.

Karolin

s3: pdbedit: add option --kickoff-time/-K to set the user's kickoff time

Use "never" as argument to set this to unlimited.

Michael

s4-dsdb: call dsdb_make_schema_global() from ldb_wrap

Calling it from samdb_connect() can cause a stale schema to be put
into the global schema.

Thanks to Andrew Bartlett for spotting this.

Second part of the fix for bug 6828 - infinite timeout occurs when byte lock held outside of samba.
Fixes case where a connection with a pending lock can me marked "idle", and ensures
that the lock queue timeout is always recalculated.
Jeremy.

s4-dsdb: always cancel transactions on all partitions

If we get an error ending a transaction on one partition we need to
continue on the other partitions.

s4-ldb: '+' can also happen in base64 encoded index DNs

s4:ldb Add detail to failures in the indexing code

s4:dsdb Rework partitions module for better tracing

This means we need to create a fake 'module' which only has a 'next'
pointer, so that we can now ldb_next_request() (which incorporates
tracing).  The remainaing stub of partition_request() is retained so
that we can indicate which partition an operation is destined for.

Similar tracing is added to the transaction handlers.

Andrew Bartlett

s4:ldb Add additional tracing of the ldb API

This helps pin down where errors occour, by printing a call stack and
setting error strings and trace messages in the transaction case.

Andrew Bartlett