s3:dbwrap: change dbwrap_store_uint32() to NTSTATUS return type

for consistency and better error propagation

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Tue Oct 11 15:51:00 CEST 2011 on sn-devel-104

s3:dbwrap: change dbwrap_store_int32() to NTSTATUS return type

for consistency and better error propagation

s3:dbwrap: change dbwrap_fetch_uint32() to NTSTATUS return type (instead of bool)

for consistency and better error propagation

s3:dbwrap: convert dbwrap_fetch_int32() to NTSTATUS return code

Return the int32 value retrieved from the db by reference.
Before this, return value "-1" was used as a error indication,
but it could also be a valid value from the database.

s3:idmap_tdb2: fix hwm-handling to use uint32 consistently

The initialization code user int32, later writes used uint32...

s3:idmap_tdb: fix hwm-handling to use uint32 consistently

The initialization code user int32, later writes used uint32...

s3:passdb:account_pol: use dbwrap_fetch_uint32, not dbwrap_fetch_int32 for the DB version

This is also stored as uint32_t.

s3:idmap_autorid: make a debug message more precise

s3:idmap_autorid: untangle function from check and log status in idmap_autorid_db_init()

s3:dbwrap: move definitions of db_record and db_contect structs to dbwrap_private.h

The API and callers now only need the forward declarations.

s3:torture: convert torture.c to only use dbwrap wrapper functions

Avoid direct use of the db_record and db_context structs.

s3:locking: convert posix.c to use dbwrap wrapper functions only

Avoid direct use of the db_record and db_context structs.

s3:locking: convert locking.c to use dbwrap wrapper functions only

Avoid direct use of the db_record and db_context structs.

s3:locking: convert brlock to only use dbwrap wrapper functions

Avoid direct use of the db_record and db_context structs.

s3:messages: convert to use only dbwrap wrapper functions

Avoid direct use of the db_record and db_context structs.

s3:groupdb: convert mapping_tdb.c to use dbwrap wrapper functions only

s3:rpc_server: convert srvsvc to only use dbwrap wrapper functions

Avoid direct use of the db_record and db_context structs.

s3:smbd: convert notify db to use dbwrap wrapper functions

Avoid direct use of the db_record and db_context structs.

s3:smbd: convert session.c to use dbwrap wrapper functions only

Avoid direct use of the db_record and db_context structs.

s3:smbd: convert connections.c to use only dbrwap wrapper functions

Avoid direct use of the db_record and db_context structs.

s3:conn_tdb: convert connections tdb to only use dbwrap wrapper functions

Avoid direct use of the db_record and db_context structs.

s3:vfs: convert the acl_tdb module to use dbwrap wrapper functions.

Avoid direct use of the db_record and db_context structs.

s3:vfs: convert nfs4_acls sidmap db code to use dbwrap_wrapper functions

Avoid direct use of the db_record and db_context structs.

s3:vfs: change the xattr_tdb module to use dbwrap wrapper functions

Avoid direct use of the db_record and db_context structs.

s3:printing: convert printer_list to use dbwrap wrapper functions

Avoid direct use of the db_record and db_context structs.

s3:secrets: convert to use dbwrap wrapper functions only

Avoid direct use of the db_record and db_context structs.

s3:passdb: convert account_pol to use dbwrap wrapper functions only

Avoid direct use of the db_record and db_context structs.

s3:passdb: convert machine_sid to use dbwrap wrapper functions only

Avoid direct use of the db_record and db_context structs.

s3:passdb: convert pdb_tdb to use dbwrap wrapper functions.

Avoid direct use of the db_record and db_context structs

s3:net: convert net idmap check to use dbwrap wrapper functions.

Avoid direct use of the db_record and db_context structs.

s3:net: convert net idmap to use only dbwrap wrapper functions

Avoid direct use of the db_record and db_context structs.

s3:idmap: convert idmap_tdb to use dbwrap wrapper functions.

Avoid direct use of the db_record and db_context structs.

s3:libnet: convert libnet_dssync_passdb to only use dbwrap wrapper functions.

Avoid direct use of the db_record and db_context structs.

s3:privileges: convert privileges.c to use dbwrap wrapper functions.

Avoid direct use of the db_record and db_context structs.

s3:g_lock: convert to use only dbwrap wrapper functions.

Avoid direct use of the db_record and db_context structs.

s3:registry: convert reg_backend_db to use dbwrap wrapper functions

Avoid direct use of the db_record and db_context structs

s3:talloc_dict: change to only use dbwrap wrapper functions

Avoid direct use of the db_record and db_context structs.

s3:sharesec: convert sharesec.c to use dbwrap wrapper functions

Avoid direct use of the db_record and db_context structs

s3:sessionid: convert to using dbwrap wrapper functions only

Avoid direct use of the db_record and db_context structs.

s3:sessionid: use dbwrap_travers_read() in sessionid_travers_read()

This also changes the return code of sessionid_traverse_read() to NTSTATUS.
It also uses traverse_read instead of traverse.

s3:sessionid: use dbwrap_travers() in sessionid_traverse()

This also changes sessionid_traverse to NTSTATUS return type.

s3:net: convert net serverid to only use dbwrap wrapper functions

Avoid direct use of the db_record and db_context structs.

s3:serverid: convert to using dbwrap wrapper functions only

Avoid direct use of the db_record and db_context structs.

s3:net: fix the exit code of net serverid wipe

s3:net: fix the exit code of net serverid list

s3:serverid: use dbwrap_traverse_read() in serverid_traverse_read()

This also fixes the return value: originally the int return value
of traverse_read which is a count of the records traversed or negative
upon failure, was simply casted into a bool return value.

s3:serverid: use dbwrap_traverse() in serverid_traverse()

This also fixes the return value: Originally, the int returned
by the traverse method, which is the count of the traverse was
casted into a bool return value.

s3:dbwrap_torture: convert to using wrapper functions only

s3:dbwrap_tool: convert to using wrapper functions only

s3:dbwrap: convert dbwrap_util.c to use dbwrap wrapper functions

s3:dbwrap: use dbwrap_fetch_bystring() in dbwrap_fetch_[u]int32()

s3:dbwrap: convert dbwrap_fetch(), dbwrap_fetch_bystring() and dbwrap_fetch_bystring_upper() to NTSTATUS

s3:dbwrap: add dbwrap_fetch_locked(), a wrapper for db->fetch_locked

s3:dbwrap: add dbwrap_record_delete(), a wrapper for record->delete_rec

s3:dbwrap: add dbwrap_record_store(), a wrapper for record->store

s3:dbwrap: add get-functions for db_record key, value and private_data

s3:dbwrap: add wrapper function dbwrap_get_seqnum()

s3:dbwrap: add wrappers for transactions start/cancel/commit

s3:dbwrap: add wrapper dbwrap_get_flags()

s3:dbwrap: fix db_rbt_traverse() to return the record count on success

this makes it consistent with documented behaviour of tdb_traverse()

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

s3:dbwrap: add specific dbwrap_wipe() implementation to dbwrap_rbt

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

s3:dbwrap: add specific dbwrap_wipe() implementation to dbwrap_tdb

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

s3:dbwrap: add a comment

s3:dbwrap: add function dbwrap_wipe()

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

s3:dbwrap: add specific dbwrap_parse_record() implementation to dbwrap_rbt

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

s3:dbwrap: implement dbwrap_fallback_exists() with dbwrap_parse_record()

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

s3:dbwrap: allow NULL parser for dbwrap_parse_record()

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

s3:dbwrap: dbwrap_parse_record() should always return -1 on "not found"

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

s3:dbwrap: add function dbwrap_parse_record()

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

s3:dbwrap: add specific dbwrap_exists() implementation to dbwrap_rbt

s3:dbwrap: factor out db_rpt_search_internal()

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

s3:dbwrap: add specific dbwrap_exists() implementation to dbwrap_tdb

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

s3:dbwrap: add function dbwrap_exists()

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

s3:dbwrap: add a wrapper dbwrap_traverse_read()

s3:dbwrap: change the dbwrap_traverse() wrapper to return the count in an additional parameter

s3:dbwrap: reorganize dbwrap.h some

s3:dbwrap: move dbwrap_traverse() to dbwrap.c, the core of the dbrwap subsystem

s3:dbwrap: move dbwrap_store() back to dbwrap.c, the core of the dbwrap subsystem

s3:dbwrap: move dbwrap_delete() back to dbwrap.c, the core of the dbwrap subsystem

s3:dbwrap: move dbwrap_fetch() back to dbwrap.c, the core of the dbwrap subsystem

s3:dbwrap: move the db_open_rbt() prototype to a new header dbwrap_rbt.h

s3:dbwrap: move the db_open_file() prototype to a new header dbwrap_file.h

s3:dbwrap: move the db_open_tdb() prototype to a new header dbwrap_tdb.h

s3:dbwrap: move the db_open_ctdb() prototype to a new header dbwrap_ctdb.h

s3:vfs_acl_common: also parse xattr.NTACL version 1

This is what the source4/ntvfs/posix code uses.
It's also used at provision time to setup the sysvol permissions.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue Oct 11 14:16:25 CEST 2011 on sn-devel-104

s3:wscript: add 'gssapi' dependecy to 'errors3'

metze

s3-passdb: split out passdb/pdb_ipa.h.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Tue Oct 11 12:15:36 CEST 2011 on sn-devel-104

s3-passdb: split out passdb/pdb_nds.h.

Guenther

s3-passdb: split out passdb/pdb_ldap.h.

Guenther

s3-smbldap: rearrange prototypes a bit.

Guenther

ldb:ldb_autotransaction_request - error string shouldn't be set here

A generic error string should be set independently of the access
mechanism to an LDB operation. Hence it should be handled in "ldb_wait"
and "ldb_request"

Signed-off-by: Andrew Tridgell <tridge@samba.org>
Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Tue Oct 11 10:15:18 CEST 2011 on sn-devel-104

ldb:pyldb.c - py_ldb_* modification calls - error string shouldn't be set here

As discussed with Jelmer, we shouldn't be setting a generic LDB error string
only for the Python bindings alone. This should be done in "ldb_request"
and "ldb_wait" - the common place for all possible LDB access mechanisms.

Signed-off-by: Andrew Tridgell <tridge@samba.org>

s4:ldap_backend.c/"map_ldb_error" - handle errors similar to "PyErr_SetLdbError"

If the call was done using an error string ("add_err_string"), then use that one
without an additional "ldb_strerror()" for the definitive LDAP output.
Otherwise generate one using "ldb_strerror()".
This omits redundancies in the error string generation (twice the same
information by "ldb_strerror()").

Signed-off-by: Andrew Tridgell <tridge@samba.org>

ldb:ldb.c/"ldb_wait" - make "ldb_wait" always return an error string

Signed-off-by: Andrew Tridgell <tridge@samba.org>

ldb:ldb.c/"ldb_wait" - change "ldb_wait" slightly in order to introduce error messages

In this occasion remove a redundant check for "LDB_ASYNC_DONE":

if (handle->state == LDB_ASYNC_DONE ||    if (handle->state == LDB_ASYNC_DONE)
    handle->status != LDB_SUCCESS) {              return handle->status;
        return handle->status;         == if (handle->status != LDB_SUCCESS)
}                                                 return handle->status;
...                                       ...
return LDB_SUCCESS;                       return LDB_SUCCESS;

Signed-off-by: Andrew Tridgell <tridge@samba.org>

ldb:ldb.c/"ldb_request" - make "ldb_request" always return an error string

Signed-off-by: Andrew Tridgell <tridge@samba.org>

ldb:ldb.c - "ldb_set_errstring" can be implemented by using "ldb_asprintf_errstring"

Signed-off-by: Andrew Tridgell <tridge@samba.org>

ldb:ldb_autotransaction_request - fix a typo in a comment

Signed-off-by: Andrew Tridgell <tridge@samba.org>

gensec: trim header includes back to what is actually required

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Tue Oct 11 06:13:08 CEST 2011 on sn-devel-104

auth/credentials Declare remaining functions are public interfaces and put into credentials.h

This is in preperation for this file being used by s3, and recognises that these are all
reasonable, public interfaces but were not declared as such in the past.

Andrew Bartlett