s3:registry: streamline and cleanup regdb_set_secdesc() somewhat.

Michael

s3:registry: refactor deletion of value/secdesc/subkey list tdb records out

of regdb_store_values().

Michael

s3:registry: streamline reg_deletekey() somewhat.

use W_ERROR_... macros and separate assignments from checks

Michael

s3:registry: use create_reg_subkey() in reg_createkey().

Instead of calling store_reg_keys()

On my box, importing 2000 shares into an empty smbconf key
now takes some 32 seconds.

Michael

s3:registry: implement create_subkey for the smbconf backend

by delegating to the db backend

Michael

s3:registry: implement create_subkey for the db backend.

Michael

s3:registry: add a create_subkey method to the backend ops.

This is to provide a more atomic means of adding a subkey of a key.

Michael

source/include/nt_status.h: add macro W_ERROR_NOT_OK_GOTO(x, y)

a goto destination can be specified as a second parameter.

Michael

s3: move definition of W_ERROR_NOT_OK_GOTO_DONE down to nt_status.h

where all the other W_ERROR_xyz macros are found as well.

Michael

s3:net rpc registry: use regsubkey_ctr_init() instead of using talloc directly.

Michael

s3:registry: hash the list of subkeys in the regsubkey_ctr

This removes many loops over all the arrays
(from regsubkey_ctr_key_exists) and thus
reduces "net conf drop" from 1m55 to 48seconds
and "net conf import" from 1m55 to 58 seconds
for 2000 shares on my box.

Michael

s3:registry: remove definition of regsubkey_ctr from the surface.

All access is now through accessor functions in reg_objects.c
This allows for performance tuning under the hood in the next step.

Michael

s3:profiles: use regsubkey_ctr_init() instead of using talloc directly.

Michael

s3:registry: use regsubkey_ctr_init() in reg_eventlog.c

Michael

s3:services_db: use regsubkey_ctr_init() instead of using talloc directly

Michael

s3:registry: use regsubkey_ctr_get_seqnum() in reg_backend_db.c

Michael

s3:registry: use regsubkey_ctr_set_seqnum() in reg_backend_db.c

Michael

s3:registry: use regsubkey_ctr_init() in reg_backend_db.c

instead of using talloc directly.

Michael

s3:registry: use regsubkey_ctr_init() in reg_api.c

instead of using talloc on struct regsubkey_ctr.

Michael

s3:registry: add regsubkey_ctr_get_seqnum() to hide implementation

Michael

s3:registry: add regsubkey_ctr_set_seqnum to hide implementation from caller.

Michael

s3:registry: add a regsubkey_ctr_init function for allocating a regsubkey_ctr

Michael

s3:registry: fix a comment

Michael

s3:registry: don't directly access key->subkeys->subkeys[] in reg_backend_db.c

Use the reg_objects accessor regsubkey_ctr_specific_key() instead.

Michael

s3:registry: don't directly access key->subkeys->subkeys[] in reg_api.c

Use the reg_objects accessor regsubkey_ctr_specific_key() instead.

Michael

s3:registry: don't directly access key->subkeys->num_subkeys in reg_backend_db.

Use the reg_objects api instead.

Michael

s3:registry: don't directly access key->subkeys->num_subkeys in reg_api.c

Use the reg_objects api instead.

Michael

s3:registry: replace typedef "REGSUBKEY_CTR" by "struct regsubkey_ctr"

This paves the way for hiding the typedef and the implementation from the
surface.

Michael

Revert "Fix a O(n^2) algorithm in regdb_fetch_keys()"

This reverts commit a13f065bad0f4d21a67e68b743f17f45bf0a4691.

This fix is reverted, because the speedup is going to move
further down into reg_objects.c. The unsorted list of subkey names
is going to be indexed: This O(n^2) search bites us in more places.
This re-establishes the abstraction of reg_objects.c.

Michael

errors: add WERR_NOT_FOUND.

Guenther
(This used to be commit b9ac03bdfa5763c713674acd966ab5d4371992a5)
(cherry picked from commit d0411c19dc847919b2a7941d91d2605c885e2b62)

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

s3:net: wrap net conf import into one big transaction

This speeds up "net conf import" of a file with 2000 shares
from 11 minutes to 1m50s on my box.

Michael

s3:libsmbconf: add transactions to the libsmbconf api

This is useful for wrapping higher level aggregate operations
in transactions. The text backend implementations just return
WERR_OK, the registry backend implementatoins use the
regdb_transaction_start|commit|cancel routines just added.

Michael

s3:registry: wrap deletekey_recursive in one big transaction.

This speeds up "net conf drop" with 2000 shares on my box from
4m40s to 1m50s, leaving virtually only cpu load.

Michael

s3:registry: provide transaction_start|commit|cancel fns for the registry tdb

Michael

Wrap creating the sorted subkey cache in a transaction

Add a comment describing the sorted subkeys

Add sorted subkey cache

On my box this gets net conf list for 1000 records down to .1 seconds

Add db_tdb_parse

Add dbwrap->parse_record

Speed up "net conf list"

For 1000 shares this speeds up net conf list from .6 to .25 seconds on my box

Speed up "net conf list"

With 1000 shares in the registry, this changed the time of "net conf list" from
1.1 seconds to .6 seconds.

s3:dbwrap: add get_flags() hook to db_context

metze

s3:libsmbconf: use talloc_free instead of TALLOC_FREE in reg backend

Michael

libsmbconf: add method is_writeable() and wrapper smbconf_is_writeable()

This allows for per-config-source checking of write support.

Michael

libsmbconf: add backend_requires_messaging() method to libsmbconf.

In a clustered environment, the registry backend needs messaging
to be set up since ctdb requires this.

Michael

libsmbconf: add "includes" to the forbidden_valnames[].

This removes the warning messages 'Unknown parameter encountered: "includes"'.

Michael
(cherry picked from commit b20019ceaaf4a8964792f6ba37f50f91b6847e7f)

libsmbconf: fastpaths first in smbconf_reg_valname_valid().

Before dropping into lp_parameter_is_valid().

Michael
(cherry picked from commit 59beb7acd777a4c224dce90fbbff7a137d2b89c1)

remove an unused variable

Michael

remove an unused variable

Michael

Fix a O(n^2) algorithm in regdb_fetch_keys()
(cherry picked from commit d98c43982c56ef788144a3fd67cdd65a9f95f2be)

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

Fix a buffer handling bug when adding lots of registry keys

This is *ancient*... From 2002, and nobody noticed until someone added lots of
shares using net conf... :-)
(cherry picked from commit 36ae846d15027df5e3a02ffabb08183dad9f6517)

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

Merge commit 'origin/v3-2-test' into v3-2-ctdb

s3:packaging: fix parsing of include.h in fill-templates

The content has changed slightly in master for the merged build,
and we must be more exact when parsing the lines.

Michael

s3:packaging: fix path to calling update-pkginfo after pushd in fill-templates

Michael

s3:packaging: paths cosmetics in fill-templates

Michael

enable IPv6 support for NetBSD, FreeBSD

most systems include netinet/in.h via netdb.h but *BSD don't.

Merge commit 'origin/v3-2-test' into v3-2-ctdb

s3:build: some implementations of "ln" are picky about the order or arguments

Michael

Fix resume handle for _samr_EnumDomainGroups

Merge commit 'origin/v3-2-test' into v3-2-ctdb

packaging: make update-pkginfo callable from any directory

Michael
(cherry picked from commit 7ea90835ebf6e2f2c6f325760800d8361d087ede)

packaging: use "find" instead of "du" to find files... :-)

Michael
(This used to be commit 768aa82361a2c4b0aab71433b7593723a2868f0a)
(cherry picked from commit fd7635a1161e25d88a8252a5cf9de70ca2fe4232)

packaging: remove commented out stuff from update-pkginfo

Michael
(cherry picked from commit d714e58ec1f74897cd13e7cd9dd9003600052b45)

packaging: don't clutter the output of fillup-templates  with popd output

Michael
(cherry picked from commit 0c711935d30b500a15ef06fd658d382b2c1e2c84)

packaging: fix fill-templates call to update-pkginfo to pushd appropriately first

Michael
(cherry picked from commit c28d4ae537d9fc0b76eae758d32356eaf1113c89)

packaging: add script fill-templates

This is a standalone-wrapper for update-pkginfo, which is
usually called from create-tarball. It basically repeats
some functionality of create-tarball.

Michael
(cherry picked from commit ca23469a906bab690162184e8d3949897f7b5a67)

s3-spoolss: fix memleak in get_remote_printer_publishing_data().

Guenther
(cherry picked from commit 97f1514ce95800f1296b1979f1b34dcc5d56a376)

Merge commit 'origin/v3-2-test' into v3-2-ctdb

s3 build: Fix "assignment discards qualifiers from pointer target type" warnings

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

s3:winbind_user: fix "getent passwd" to allocate new uids.

"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

Fix count in _srvsvc_NetDiskEnum()

Without this, remotely adding a share via MMC fails

s3:winbind_group: fix "getent group" to allocate new gids.

"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
(cherry picked from commit 8c64302915bde8a5400b575389b12e0eaf2cf140)

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

Memory leaks and other fixes found by Coverity

(This is the winbindd_group.c portion of the patch - obnox)

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

Memory leaks and other fixes found by Coverity

(This is the winbindd_user.c portion of the patch - obnox)

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

s3:configure: autodetect ctdb/cluster support

We need to make sure ctdb has transaction support
and we autodetect ipv6 support.

metze
(cherry picked from commit 11163720e6f30e83f523c5009e1e60b5cf28cda6)
(cherry picked from commit f87f4842cc0ffc62d76ee5350b94f7bda735aa71)

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

s3:ctdbd_conn: canonicalize ips before sending them to ctdbd

This makes samba work with older ctdb versions.

metze
(cherry picked from commit 7540d13956a8a69d25f6fbbc4f9a3417e91db1d2)
(cherry picked from commit a7fcaff295c3a36e2b783398ac212500eef33ecb)

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

Merge commit 'origin/v3-2-test' into v3-2-ctdb

s3: reinit_after_fork() should reinit the event context before the
messaging context

Because messaging_reinit() may add events to the event context, which will
removed by event_context_reinit().

metze
(cherry picked from commit 56af6112d67aa9dd3757297fbe29ce1a3eabf00d)

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

Merge commit 'origin/v3-2-test' into v3-2-ctdb

mount.cifs: initialize rc to 0 in main

The value of rc in main() isn't initialized in the declaration. This
wasn't a problem before, but Shirish's fakemount patch can make it so
that we return the uninitialized variable if the -n flag is used.

Fix this by initializing rc to 0.

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

mount.cifs: add fakemount (-f) and nomtab (-n) flags to mount.cifs

...so that these options work correctly when passed in by mount(8).

s3:net conf: remove check for sharename being a usernam in "net conf addshare"

This is useless and can be overriden by "net conf setparm" anyways.

Michael

docs: fix two typos in the mount.cifs manpage

Michael
(cherry picked from commit 145fe37766cf1ecffb16a03b58b44d08f7ed7558)

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

Don't try and delete a default ACL from a file.

Have nmbd check all available interfaces for WINS before failing

When nmbd is acting as WINS, it picks the first interface's IP as WINS
server's IP. If the first interface's IP is zero, we will just quit
(even though we might have other interfaces with valid IPs).

This patch makes nmbd look at all interfaces and pick the first interface
with a valid IP as the WINS server's IP.

umount.cifs: clean-up entries in /etc/mtab after unmount

This patch removes the remaining entry in /etc/mtab after a filesystem
is unmounted by canonicalizing the mountpoint supplied on the command
line.

Please refer to bug 4370 in samba bugzilla.

s3-net: remove unused ENUM_HND.

Guenther
(cherry picked from commit a713060a6e36706cfd5b997aa4cbfabf4e180c84)

Fix bug #6085 - In vfs_default.c change utime( ) call.

Probably fixes a crash during name resolution when log level >= 10
and libc segfaults if printf is passed NULL for a "%s" arg
(eg. Solaris).

Merge commit 'origin/v3-2-test' into v3-2-ctdb

s3: Fix 'net rpc join' for users with the SeMachineAccountPrivilege.

This used to be commit fda8abac in master.

packaging(RHEL-CTDB): makerpms.sh: build winbind-32bit libs in the 64bit build

Michael

packaging(RHEL-CTDB): Build winbind-32bit package in the 64bit build

Michael

WHATSNEW: Update changes since 3.2.7.

Karolin

packaging(RHEL-CTDB): extend makespec.sh to extract VENDOR_PATCH from version.h

Michael

packaging(RHEL-CTDB): The former release number has basically moved into VERSION

Michael

VERSION: add vendor suffix ("ctdb") and vendor patch level

Michael

WHATSNEW: Update changes since 3.2.7.

Karolin

Adjust regex to match variable names including underscores

This is required to get the CIFSUPCALL_PROGS setting extracted from
config.log.

Conditional install of the cifs.upcall man page

Only install the cifs.upcall man page if CIFSUPCALL_PROGS was set while
configure.