Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Fri Jul 29 13:34:22 CEST 2011 on sn-devel-104

Also start new folder lib/dbwrap/ where dbwrap_open.c is stored and
make the fallbacke implementation functoins non-static and create a
dbwrap_private.h header file that contains their prototypes.

Reported by herb@samba.org. Remove the (premature) optimization
on file close.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat Jul 16 02:32:02 CEST 2011 on sn-devel-104

My previous patches fixed up all direct TDB callers, but there are a
few utility functions and the db_context functions which are still
using the old -1 / 0 return codes.

It's clearer to fix up all the callers of these too, so everywhere is
consistent: non-zero means an error.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

This is needed for OpenChange, which prints Samba struct server_id
values in debug messages.

Andrew Bartlett

Using the standard macro makes it easier to move code into common, as
TALLOC_MEMDUP isn't standard talloc.

Using the standard macro makes it easier to move code into common, as
TALLOC_ZERO_P isn't standard talloc.

Using the standard macro makes it easier to move code into common, as
TALLOC_P isn't standard talloc.

Using the standard macro makes it easier to move code into common, as
TALLOC_ARRAY isn't standard talloc.