Refactored the code that adds Domain Users to BUILTIN\Users to use the new helper functions.

- Modified create_builtin_users to take in the domain sid to reduce the number
of times it needs to be looked up.
- Changed create_builtin_users to call the new helper functions.
- Changed create_local_nt_token to call the new version of create_builtin_users
and handle the new error that can be returned.

Helper functions to enable domain groups to be added to builtin groups at domain join time

Added two new helper functions which wrap the raw pdb alias functions so they
can be more conveniently called while adding domain groups to builtin groups.

rpc_client: Bug 5616 - fix session keys also in rpccli_netr_LogonSamLogonEx wrapper.

Guenther

build: fix some no previous prototype warnings.

Guenther

winbindd: handle trusted domains without sid.

Guenther

libwbclient: let wbcStringToSid handle the global NULL sid.

Guenther

netapi: add NetGroupGetUsers example code.

Guenther

netapi: add NetGroupGetUsers to public header.

Guenther

netapi: add NetGroupGetUsers skeleton.

GUenther

re-run make idl.

Guenther

netapi: add NetGroupGetUsers to IDL.

Guenther

re-run make idl.

Guenther

netapi: add some more USER_INFO structs to IDL.

Guenther

netapi: fix libnetapi_samr_lookup_user_map_USER_INFO.

Guenther

netapi: use "buffer" in libnetapi.

Guenther

re-run make idl.

Guenther

netapi: cosmetics, use "buffer" everywhere.

Guenther

netapi: use init_samr_CryptPasswordEx and init_samr_CryptPassword.

Guenther

rpc_client: add init_samr_CryptPasswordEx and init_samr_CryptPassword.

Guenther

netapi: implement NetUserSetInfo_r() for at least level 1007.

Guenther

netapi: add convert_USER_INFO_X_to_samr_user_info21 fn and use it NetUserAdd.

Guenther

re-run make idl.

Guenther

netapi: add USER_INFO_X to IDL.

Guenther

netapi: add netapi testsuite.

Guenther

netapi: fix NetLocalGroupAdd.

Guenther

testparm: Display warning if invalid values are used.

This one came up while using "csc policy = disabled" instead of
"disable"... ;-)

pdb_interface: Fix typo in debug message.

Karolin

man pages: Fix typos in man smbclient.

Karolin

Attempt to fix bug 5616

We were calculating the session key but did not return it to the caller...
(cherry picked from commit 8ab79b1d009d53e414b90e4a0ab8fc7a4889b6df)

run "make idl" after idl change to unify Primary:Kerberos* structs (e8ae49f)

Michael

drsblobs.idl: unify the Primary:Kerberos and Primary:Kerberos-Newer-Keys structs

metze
(cherry picked from commit 4b79a7678571ac2f7d5f827913fdcb419f5d2e20)

run "make idl" after idl change "give some unknowns a meaning" (67de745)

Michael

drsblobs.idl: give some unknowns a meaning

metze
(cherry picked from commit 231e6f5ab2dc8a3e991a9872be252cffff6f14c6)

rpcclient: Set the pid filed of the outgoing DsBindInfo to 0.

This is for debugging and informational purposes only.
The assignment is implementation specific.
(WSPP docs, sec. 5.35).

Michael

Fix a debug message

Remove a pointless CONST_DISCARD

Fix srvsvc_Net[Get|Set]FileSecurity

There were two bugs in those routines: They did not send INTERNAL_OPEN_ONLY to
open_file_ntcreate() and they did not chdir, so the file could never be found.

While there I decided to remove the become_root() calls and call create_file()
instead of the lower-level routines.

Make create_conn_struct() public

Remove unused extern struct current_user from substitute.c

A bit hard to believe, but substitute.c does not use current_user anymore.

Save 128 byte (32-bit platform) or 256 byte (64-bit) per connection_struct

This was just too easy not to use :-)

make read/write to internal pipes available externally

Refactoring: Make close_internal_rpc_pipe_hnd a talloc destructor

Refactor make_internal_rpc_pipe_p: connection_struct is not needed

Create doxygen comments for async_req.[ch]

Make api_rpcTNP static to srv_pipe.c

Fix an incompatible pointer warning
(cherry picked from commit 3282f7289b7b33beeaa1ca649651cca6537a69af)

Fix bug creating files using DOS clients with mixed
case files. Reported by Daniel Johnson <Progman2000@usa.net>.
The smb_set_file_time() call to set the filetimes is failing
because it's using the unmodified name passed in by the
client, not the modified name (matching case on the
disk) that comes out from create_file().
Jeremy.

Fix a race condition in winbind leading to a crash

When SIGCHLD handling is delayed for some reason, sending a request to a child
can fail early because the child has died already. In this case
async_main_request_sent() directly called the continuation function without
properly removing the malfunctioning child process and the requests in the
queue. The next request would then crash in the DLIST_ADD_END() in
async_request() because the request pending for the child had been
talloc_free()'ed and yet still was referenced in the list.

This one is *old*...

Volker
(cherry picked from commit 8691709626b0d461de91b8fc9d10c730d1f183dd)

talloc_string_sub2: Don't return NULL if src is empty.

This fixes BUG #5635.

Finished print jobs were not removed from the $PRINTER.tdb file if
"printing=cups".

In print_queue_update, talloc_string_sub2 is used to assemble the
"lprm command". In the case of using "printing=cups", the default
"lprm command" is an empty string. talloc_string_sub2 is called with
this empty string and returns NULL which leads to exiting
print_queue_update without doing the actual print queue update.

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

If we're not allowing streams on this conn ptr,
then don't allow create_file() to call down to
create_file_unixpath() with a stream name.
Jeremy.

Remove trailing withespace from wbinfo -m which breaks gdm auth.

Signed-off-by: Andreas Schneider <anschneider@suse.de>

Add casts to uint32_t to match prototype for sid_append_rid().
Jeremy.

Use sys_getgrnam not getgrnam. Pointed out by Herb.
Jeremy.

Merge branch 'v3-3-test' of git+ssh://sfrench@git.samba.org/data/git/samba into v33

cifs.upcall was not recognizing the newer name "dns_resolver" key type
(as a synonym for the older "cifs.resolver" name) when resolving host
names to ip addresses for the kernel.

Acked-by: Jeff Layton

Remove unused request_finished_cont()

Make use of TALLOC_FREE when freeing the per-winbindrequest memory context

pdb_ldap: Raise level for debug message to avoid log file flooding.

re-run "make idl" after adding idl for idl for Primary:Kerberos-Newer-Keys

Michael

drsblobs.idl: add idl for Primary:Kerberos-Newer-Keys blob in supplementalCredentials

metze
(cherry picked from commit 97b7901afbccc9647ad2958d4cf12300de2655d1)

re-run "make idl" after fixing unknowns in package_PrimaryKerberos (drsblobs.idl)

Michael

drsblobs.idl: fix unknowns in package_PrimaryKerberos idl

metze
(cherry picked from commit da9ceb2bf17f964334d9317829d40483e2c04b10)

re-run "make idl" after fixing idl for supplementalCredentialsSubBlob.

Michael

drsblobs.idl: fix idl for supplementalCredentialsSubBlob

metze
(cherry picked from commit 24c5b10136f6e640832193aaf9e6d7e865c288bc)

re-run "make idl" after changing unknown1->reserved in drsblobs.idl

Michael

drsblobs.idl: rename unknown1 -> reserved

metze

cifs.upcall: fix compile warning

Steve French noticed these warnings when building cifs.upcall:

   Compiling client/cifs.upcall.c
   client/cifs.upcall.c: In function 'usage':
   client/cifs.upcall.c:204: warning: declaration of 'prog' shadows a global declaration
   client/cifs.upcall.c:33: warning: shadowed declaration is here

Change the usage function to not take and arg and have it just use the global
"prog" variable. Fix a typo in the log message generated when an unknown
option is specified. Also getopt() always returns '?' when it sees an unknown
option so there's no point in printing it out.

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

rpc_parse: Unify spoolss debug messages.

Karolin

Change occurrences of the u1 member of DsBindInfo* to pid after idl change.

Michael

re-run "make idl" after changing u1 member to pid in DsBindInfo*.

Michael

drsuapi.idl: change the u1 field in DsBindInfo* to "pid".

According to the WSPP docs, section 5.35,
this is the "process identifyer" of the client.
It is meant for informational and debugging purposes
only and its assignment is implementation specific.

Michael

Check for f_frsize when using statvfs

Add a configure test for the availability of f_frsize in struct statvfs
(for broken platforms that define statvfs but still have
f_bsize/f_iosize). Also add sys/types.h to the other struct statvfs test
(again, some platforms need it).

Make DSO_EXPORTS_CMD regexp more POSIX compliant

The FreeBSD sed command doesn't understand \? without passing -E to turn
on extended regexps. This patch changes the DSO_EXPORTS_CMD regexp to a
POSIX compliant RE by switching the \+ to a \{1,\} bound and the \? to a
\{0,1\} bound.

Fix warnings on FreeBSD-based platforms

Fix two shadowed declaration warnings on FreeBSD-based platform:
'reboot' is a 4.0BSD syscall in unistd.h and 'tcp_seq' is a typedef in
netinet/tcp.h.

Fix various build warnings

This fixes various build warnings on our platform. I'm sure I haven't
caught them all, but it's a start.

Add --enable-picky-developer

This adds an --enable-picky-developer option that will halt compilation
on warnings. Yes, this could be handled by a direct Makefile change, but
people should be encourage to do it!

rpcclient: make DsGetNCChanges work against w2k8 (add knowledge of DsBindInfo48)

Michael

re-run make idl after adding drsuapi_DsBindInfo48 to drsuapi.idl

Michael

drsuapi.idl: add drsuapi_SupportedExtensionsExt bitfield.

This knowledge is obtained from the wspp-docs (section 5.35).

Michael
(cherry picked from commit f5afb695045b1a2f3b8c00a4d82d40e8e50726c9)

drsuapi.idl: the last 16 bytes in DsBindInfo48 ar the GUID of the config dn.

This bit seems not to be documented in the WSPP docs.

Michael
(cherry picked from commit 705f79bd0a5e93daa0cb11b5dcca36e75c75df93)

drsuapi.idl: add drsuapi_DsBindInfo48.

This is necessary to make DsGetNcChanges work with win2008.

Michael
(cherry picked from commit dd278b069b8683a0e3721ebb7d0de06d2bc1c86f)

re-run make idl after Jelmer's "poperly cast array lengths" pidl change.

(f321240fa91fa19c1131f119c42f64897d220682)

Michael

man pages: "printcap name" is a global parameter, not a share specific one.

Karolin

Update Roadmap
(cherry picked from commit d1c3b57085b493b47398daf71ce18f16ece57da0)

YESS -- remove the PI_* pipe constants

Make idl: cli_do_rpc_ndr does not use pipe_idx anymore

Remove unused function rpccli_is_pipe_idx

Refactoring: CLI_DO_RPC_* uses interface instead of pipe_idx

Remove a reference to pipe_idx

Remove some unused code

Remove one reference to PI_NETLOGON

Remove one reference to PI_LSARPC

Remove some fully outdated sample code

Remove now unused cli_get_pipe_name

Remove now unused cli_get_pipe_idx

Remove now unused cli_get_iface

Refactoring: connect_dst_pipe uses ndr_syntax_id instead of pipe_idx

Refactoring: net_sh_run uses ndr_syntax_id instead of pipe_idx

Refactoring: run_rpc_command uses ndr_syntax_id instead of pipe_idx

Refactoring: rpcclient uses ndr_syntax_id instead of pipe_idx