s3-printing: fix cups pcap reload with no printers

cups_async_callback() is called to receive new printcap data from a
child process which requests the information from cupsd.
Newly received printcap information is stored in a temporary printcap
cache (tmp_pcap_cache). Once the child process closes the printcap IPC
file descriptor, the system printcap cache is replaced with the newly
populated tmp_pcap_cache, however this only occurs if tmp_pcap_cache is
non null (has at least one printer).

If the printcap cache is empty, which is the case when cups is not
exporting any printers, the printcap cache is not replaced resulting in
stale data.

https://bugzilla.samba.org/show_bug.cgi?id=7915
Signed-off-by: Andreas Schneider <asn@samba.org>

s3-printing: remove unneeded local_pcap_copy global

The cups local_pcap_copy global served as a temporary buffer during
asynchronous cups printcap cache updates, as well as indicating when
the printcap cache had not yet been filled and printcap cache update
should block.

As smbd printcap reads are now triggered by the parent smbd following
printcap cache update, the variable and blocking mechanism are no longer
needed.

Signed-off-by: Andreas Schneider <asn@samba.org>

s3-printing: follow force user/group for driver IO

Configuring force user/group settings for the print$ share currently has
unexpected results, this is explained by how the driver upload/add
process takes place. Consider the following example:

[print$]
        path = /print-drv
        write list = $DRIVER_UPLOAD_USER
        force group = ntadmin

- the client connects to the [print$] share and uploads all driver
  files to the /print-drv/W32X86 directory.

- This is permitted, as /print-drv/W32X86 is owned by group ntadmin, and
  the "force group = ntadmin" takes effect for the [print$] session.

- Once all files are uploaded, the client connects to the [ipc$]
  share and issues an AddPrinterDriverEx spoolss request.

- In handling this request move_driver_to_download_area() is called,
  which attempts to create the directory /print-drv/W32X86/3

- The create directory fails, as it is done as the user connected to
  the [ipc$] share which does not have permission to write to the driver
  directory. The [print$] "force group = ntadmin" has no effect.

This is a regression from previous behaviour prior to the commit:
783ab04 Convert move_driver_to_download_area to use create_conn_struct.

https://bugzilla.samba.org/show_bug.cgi?id=7921
Signed-off-by: Andreas Schneider <asn@samba.org>

ldb:ldb tools - remove a superflous "return" (usage internally calls "exit")

Reviewed by: Tridge

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Fri Mar  4 09:39:22 CET 2011 on sn-devel-104

ldb:ldb tools - return LDB_ERR_INVALID_DN_SYNTAX on wrong DN parameters

Not all LDB databases have further DN checks.

Reviewed by: Tridge

ldb:ldb tools - ldbtest - convert other result values to LDB codes as well

I've forgotten this in my first patchset.

Reviewed by: Tridge

s4:LDAP server - remove validation checks of input DNs

We should rather try to let the LDB modules perform these checks
otherwise different backends behaviour differently.

Reviewed by: Tridge

s4:objectclass LDB module - if we cannot find DN's parent then the DN itself is invalid

ERR_INVALID_DN_SYNTAX fits better than ERR_OPERATION_ERROR in this case. This
one gets triggered if we perform "add" requests without the LDAP server.

Reviewed by: Tridge

debug: fixed a valgrind error

Thanks to Volker for spotting this one!

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Fri Mar  4 03:09:52 CET 2011 on sn-devel-104

s3-waf: add wildcard commands from main wscript.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Fri Mar  4 02:11:11 CET 2011 on sn-devel-104

s3-nterr: use strcasecmp in nt_status_string_to_code().

Guenther

s4-nterr: move auth_nt_status_squash to nt_status_squash and move to nterr.c

Guenther

s4-nterr: some minor cosmetic edits to further match s3 nterr.

Guenther

s3-nterr: within nt_errstr() compare codes using NT_STATUS_V().

This is to avoid future very special NT_STATUS_EQUAL semantics within s4.

Guenther

nterr: make sure both nt_err_desc structs are the same.

Guenther

nterr: make sure both nt_errs structs are the same.

Guenther

s4-nterr: add _N macro handling as in s3-nterr.

Guenther

s3-nterr: make nt_err_desc static const.

Guenther

nterr: remove space indents (use tabs instead), following coding standards.

Guenther

nterr: remove trailing whitespace.

Guenther

s3-waf: add check for SEEKDIR_RETURNS_VOID.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Fri Mar  4 01:13:54 CET 2011 on sn-devel-104

s3-waf: add check for HAVE_FUNCTION_ATTRIBUTE_DESTRUCTOR.

Guenther

libreplace: move "struct timespec" checks into libreplace (where timespec is already used).

Bjoern, Metze, please check.

Guenther

socket_wrapper: use swrap_sendmsg_before()/after() in swrap_writev()

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Thu Mar  3 23:43:39 CET 2011 on sn-devel-104

socket_wrapper: use swrap_sendmsg_before()/after() in swrap_sendmsg()

This also adds the same logic for broadcast as in swrap_sendto()
for SOCK_DGRAM.

metze

socket_wrapper: use swrap_sendmsg_before()/after() in swrap_send()

metze

socket_wrapper: use swrap_sendmsg_before()/after() in swrap_sendto()

metze

socket_wrapper: add swrap_sendmsg_before/after helper functions

Currently have almost the same logic in swrap_send(), swrap_sendto(),
swrap_writev() and swrap_sendmsg(), this helper functions
let combine all the logic in 2 places.

metze

socket_wrapper: replace recvmsg() correctly

metze

socket_wrapper: readv() should only work on connected sockets

metze

socket_wrapper: move swrap_ioctl() above the send*/recv* functions

metze

socket_wrapper: fix compiler warnings

metze

socket_wrapper: don't allow connect() to the broadcast address

This will simplify other code later.

metze

s3: Use dom_sid_string_buf in sid_to_fstring

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Thu Mar  3 22:56:57 CET 2011 on sn-devel-104

Add dom_sid_string_buf

This prints into a fixed buffer with the same overflow semantics as snprintf
has: Return required string length, regardless of whether it fit or not.

s3: Use dom_sid_string in _lsa_lookup_sids_internal

s3: Remove an obsolete comment

s3: Fix some nonempty blank lines

lib/util: LIBCRYPTO is in common already, so add it to samba-util-common.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Thu Mar  3 18:08:30 CET 2011 on sn-devel-104

s3-includes: remove global include of system/readline.h.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Mar  2 23:06:17 CET 2011 on sn-devel-104

s3-proto: fix some inline comments.

Guenther

s3-libds: use already existing ../libds/common/flag_mapping.h header.

Guenther

s3: Pass the aio_force operation through vfs_gpfs

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Mar  2 19:24:27 CET 2011 on sn-devel-104

s3: Ask GPFS for the IS_OFFLINE operation

s3: Announce HSM in the gpfs module

s3: Pass smb_filename through the set_offline vfs op

s3: Pass smb_filename through the is_offline vfs op

s3: Fix operation names in vfs_full_audit.c

s3: Print all flags in "smbclient allinfo"

s4:torture/rap/rap.c - remove unused variable "i"

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Wed Mar  2 18:38:58 CET 2011 on sn-devel-104

s3:idmap_cache: remove an leftover wip comment

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Wed Mar  2 14:55:20 CET 2011 on sn-devel-104

s3: adjust loglevel for idmap_cache

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

librpc/idl: remove unused string defines from idl_types.h

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Wed Mar  2 14:09:44 CET 2011 on sn-devel-104

librpc/rpc: let ndr_pull_spoolss_EnumPrinterDataEx() use a subcontext

This is not strictly needed, but it's good to have the logic
in common with the other Enum* calls.

This also allows us to play with the NDR_RELATIVE_REVERSE flag.

metze

s3-sessionid: avoid global include of sessionid.h

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Mar  2 12:58:12 CET 2011 on sn-devel-104

s3-lanman: talk to srvsvc dcerpc server in api_RNetSessionEnum().

Guenther

s4-smbtorture: add netsessionenum rap torture test.

Guenther

s4-smbtorture: add smbcli_rap_netsessionenum().

Guenther

rap: add rap_NetSessionEnum to IDL.

Guenther

s3-server_id: only include server_id where needed.

Guenther

s3: Attempt to fix the build on IRIX

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Mar  2 11:33:05 CET 2011 on sn-devel-104

s3: GPFS can't deal with mangled names

In getrealfilename, avoid calling the GPFS function

build: fixed the ELF name for private libraries

thanks to Simo and rpmbuild for spotting this!

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Wed Mar  2 02:49:34 CET 2011 on sn-devel-104

s4-dsdb: Ensure we permit multi-valued backlinks on single-valued attributes

This was already done in repl_meta_data, but it needs to be done here
as well to cope with Windows 2000 level links.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Wed Mar  2 02:03:58 CET 2011 on sn-devel-104

s3-printing: vfs_connect prior to driver/dfs IO

samba3.posix_s3.rpc.spoolss.driver fails with the xattr_tdb vfs module
loaded as a part of make test. The (now checked) create_directory() call
in move_driver_to_download_area() fails, uncovering another bug in the
printer driver upload code path.

move_driver_to_download_area() creates a new conn_struct for
manipulating files in [print$]. The VFS layer is plumbed through with
the call to create_conn_struct(), however SMB_VFS_CONNECT() is never
called. Many vfs modules expect state stored at connect time with
SMB_VFS_HANDLE_SET_DATA() to be available on any IO operation and fail
if this is not the case.

This fix adds a call to SMB_VFS_CONNECT() in create_conn_struct() prior
to IO.

https://bugzilla.samba.org/show_bug.cgi?id=7976

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed Mar  2 01:16:30 CET 2011 on sn-devel-104

s3-printing: clean up get_correct_cversion error paths

Remove an unneeded variable and simplify error paths.

s3-printing: fix move_driver_to_download_area() error paths

WERR_ACCESS_DENIED errors are mapped to WERR_UNKNOWN_PRINTER_DRIVER,
resulting in incorrect error messages on Windows clients.

move_driver_to_download_area() returns the same error status values
to the caller via the *perr argument as well as the return value.

The create_directory() call is not checked for error.

s3: remove superfluous debug messages from msg_idmap

Signed-off-by: Michael Adam <obnox@samba.org>
Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Wed Mar  2 00:30:41 CET 2011 on sn-devel-104

Revert "examples/LDAP: remove overoutdated samba-schema.IBMSecureWay"

This reverts commit 90b0f86d81d4c68bf326c3e7c2247d9dabd75058.

this isn't that outdated at all. Got confused by the splitted oc/at files

Autobuild-User: Björn Jacke <bj@sernet.de>
Autobuild-Date: Tue Mar  1 23:43:51 CET 2011 on sn-devel-104

examples/LDAP: remove overoutdated samba-schema.IBMSecureWay

examples/LDAP: delete unmaintained samba-schema-netscapeds4.x

libds/common/flag_mapping.c - fix another build dependency

Signed-off-by: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Tue Mar  1 18:53:40 CET 2011 on sn-devel-104

s4:drsuapi RPC server - this resembles more "samdb_is_gc"

Reviewed by: Jelmer

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Tue Mar  1 17:57:47 CET 2011 on sn-devel-104

librpc/ndr: handle NOALIGN flag for relative pointers and alignment DATA_BLOBs

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue Mar  1 17:11:03 CET 2011 on sn-devel-104

s4:netlogon RPC server - "LogonGetDomainInfo" - check for NULL attributes

This is needed to complete the transition from "samdb_msg_add_string" to
"ldb_msg_add_string".
And this patch yields better NTSTATUS error results than before
(INVALID_PARAMETER rather than OUT_OF_MEMORY).

Reviewed-by: Jelmer
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Tue Mar  1 14:42:15 CET 2011 on sn-devel-104

s4:drsuapi RPC server - check for the "SPN" attribute != NULL

The SPN attribute could derive from an untrusted source (client).

Reviewed-by: Jelmer

s4:dsdb - always handle the attribute "options" as 32bit unsigned integer

It is defined as LDAP syntax 2.5.5.9 so no need at all to treat it as
64-bit integer.

Reviewed by: Kamenim and Metze

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Tue Mar  1 12:46:15 CET 2011 on sn-devel-104

libcli/security Add unix_token and unix_info to auth_session_info too

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Tue Mar  1 07:13:43 CET 2011 on sn-devel-104

s3-auth struct security_unix_token replaces UNIX_USER_TOKEN

librpc/idl Add helper structures for use by samba3 in auth_session_info

The unix info and in particular unix token needs to be preserved into
the struct auth_session_info.

Andrew Bartlett

pidl Add support for uid_t and gid_t types

These are mapped to uint64_t, which should be big enough.  This is
proposed to be used for internal Samba representations, where it would
be more painful to convert all the callers to an uint64_t calling
convention.

Andrew Bartlett

s3-rpc_server Handle session key as a constant buffer

This way, we don't have to check for memory allocation failures.

Andrew Bartlett

s4:dsdb/common/util.c - remove "samdb_find_or_add_value"

This isn't used anymore.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

s4:dsdb/common/util.c - fully remove "samdb_msg_add_string"

This isn't needed anymore and will be substituted by
"ldb_msg_add_string".

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

s4:libnet/libnet_samsync_ldb.c - add a "samdb_msg_add_string" wrapper

Add this in order to allow the "ADD_OR_DEL" macros to work.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

s4:remove many invocations of "samdb_msg_add_string"

This call can be substituted by "ldb_msg_add_string". We only need to be
careful on local objects or talloc'ed ones which live shorter than the message.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

autobuild: enabled samba3-waf build

this is just a build check for now. See the mailing list discussion.

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Tue Mar  1 06:22:07 CET 2011 on sn-devel-104

lib/util: new merged debug system

This is the s3 debug system, with a number of changes to tidy it up
for common use.  The debug class system is simplified by the removal of the
ISSET table, the system no longer attempts to cope with assignment of
DEBUGLEVEL, and the full class table is always available (rather than
just DEBUGLEVEL_CLASS[DBCG_ALL]) from startup.  It is also no longer
confusingly described as a hack, but as the initial table.

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Tue Mar  1 04:32:12 CET 2011 on sn-devel-104

lib/util move debug.[ch] out of the way

This will allow a modified version of the s3 debug system to be the
new common debug system.

Andrew Bartlett

s4-libnet_vampire: Ignore some attributes when building working schema cache

Working schema cache will be used to convert replicated Schema objects
again later, i.e. used as reference, so we don't need to resolve all
attribute OIDs for working Schema cache to be usable.

Autobuild-User: Kamen Mazdrashki <kamenim@samba.org>
Autobuild-Date: Tue Mar  1 03:45:16 CET 2011 on sn-devel-104

s4-repl/working_schema: Ignore some attributes when bulding working schema cache

We don't need all object attributes resolved and converted for a working
schema to be functional.

s4-replicated_objects: Implement a mechanism to relax some attributes conversion

during replicated object convert stage.
The problem is that we may have loops in schema graph and we can't
resolve those loops in just one pass. Ignoring some attributes
conversion will allow us to have a functional schema cache that we
can use later to resolve all attribute OIDs on another pass

s3-debug Always use C99 true/false rather than True and False

This will help with the merge into the common code.

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Tue Mar  1 02:58:55 CET 2011 on sn-devel-104

s3-build __FUNCTION__ is always available, always use it

This avoids duplication between FUNCTION_NAME and __FUNCTION__

lib/util/time: Merge time functions from source3/lib/time.c

spoolss.idl: align spoolss_DriverFileInfo relative pointer to 4 byte

metze

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue Mar  1 02:12:37 CET 2011 on sn-devel-104

spoolss.idl: align spoolss_PrinterEnumValues 'data' based on the type

metze

TODO test/review librpc/ndr: remove align2 hack for relative pointers

metze

TODO test/review librpc: align nstring and nstring_array to 2 byte

metze

librpc/ndr: ndr align relative pointers based on the given flags

We used to do this only for the reverse relative pointers
and now we always do it.

metze