Fix the build
(cherry picked from commit e160a19ffc0727c46a696335c9d140e236923cdb)

Make use of talloc_tos()
(cherry picked from commit d0f7261031f76965879036bd5c7d0a19eabf47cc)

Fix some nonempty blank lines
(cherry picked from commit 5f48a925a25a08d6c2077906eec439e49b51070a)

Fix a "nexted extern declaration" warning
(cherry picked from commit 5822bb7458331c5d52a5a4cb0906777e1f7e3b8d)

Initialize near creation of resource. Makes code clearer.
Jeremy.
(cherry picked from commit cb23dddc723084ca9f8d8eca6c78ee87d903e60c)

Ensure we do reinit_after_fork().
Jeremy.
(cherry picked from commit 906e5bf4d2112777ab94ee3761b6376a2150d31a)

Allow data flow to be debugged and only log on error. All seems ok now.
Jeremy.
(cherry picked from commit a2f989b783f417d6ed5c5d4fb737f61192d087ab)

Async is trickier than it looks :-). Don't use a stack variable for a private data ptr.
Jeremy.
(cherry picked from commit b443507a0cc97821af0011b56bc971fea6839a98)

If you have a large number of cups printers, then scanning for print info can cause a client to timeout
(it takes longer than 30 seconds to enumerate them). Make scanning for printers async with a callback
from the main loop. This fixes a bug that was irritating *me* :-).
Jeremy.
(cherry picked from commit d66f58ad1b5c7195ea09a55f733c6f6551f663c2)

Ensure we handle signals correctly during the async calls.
Jeremy.
(cherry picked from commit eb5323a728e9df0be56b6f73a127aed011b250d0)

Remove SEC_ACCESS. It's a uint32_t.
Jeremy.
(cherry picked from commit 5e77ccaa866d422e56c28af610692b8167d630e8)

Make use of ZERO_STRUCT (the first memset was actually wrong)
(cherry picked from commit 723fe06bf20e5f6e9c2075f66de01475ff8bfc68)

mount.cifs: make return codes match the return codes for /bin/mount (try #3)

The manpage for /bin/mount specifies that the return code should be a
positive integer (actually, it's a bitfield). Clean up the return
codes from mount.cifs to make them match the expected return values
from /bin/mount. This necessary for proper integration with autofs.

This is the third attempt at this patch. The changes here are minor,
just changing some return's from main() into exit() calls for
consistency's sake.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit a6e250ad2f50ec1c083a86d0161c1c00c1c3129e)

mount.cifs: have uppercase_string return success on NULL pointer

We currently don't attempt to uppercase the device portion of the mount
string if there isn't a prefixpath. Fix that by making uppercase_string
return success without doing anything on a NULL pointer.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit 1c9f269280983119c7c906bbc2bf1ccaaba8c740)

Remove a pointless level of indirection
(cherry picked from commit f953fd62f7e4d4d18eb4d713989eacce630c4949)

Fix a typo
(cherry picked from commit 7111d24f585643e92e5cd62cb339e771e7fcc8c4)

For the vfs_acl_xattr.c module, make sure we map GENERIC file and directory bits
to specific bits every time a security descriptor is set. The S4 torture suite proves
that generic bits are not returned when querying an ACL set using them (ie. only
the specific bits are stored on disk).
Jeremy.
(cherry picked from commit 306b507c3ba1b864b6d66f9438f91f979b02030f)

Fixed "might be uninitialized" warning
(cherry picked from commit 1f95d82dd1b86e308d75e9f755f83c34e6ceb28d)

Deal with inheritance from parent directory when setting Windows
ACLs.
Jeremy.
(cherry picked from commit efc3de5e2490be615121f7231cd1611df2163e78)

Fix bug #5814 - Winbindd dumping core in a strange manner while doing "rescan_trusted_domain".
From analysis by hargagan <shargagan@novell.com> :
"The winbindd_child_died() is also getting called from process_loop() in case of
SIGCHLD signal. In this case it doesn't make the timeout_handler to NULL for
the first request. It then initiate a new request using
schedule_async_request() which installs a new timeout handler for the same
request. In such a case, for a badly unresponsive system both the timeout
handler can be called. For the first call the "private_data" will be cleared
and for another call the timeout handler will be detecting the double free. So,
for such a case as well, the winbindd_child_died() should make the
timeout_handler to NULL."
Jeremy.
(cherry picked from commit 6be98cbb23f4cdb65353a426ae5edbf0290774d8)

Remove unused CHECK_FSP macro
(cherry picked from commit 9bb057a7712393b1b699e25c5cab44401fd00005)

Fix const warning.
Jeremy.
(cherry picked from commit 2b8c7a53860b43435f1b68235abd14e2f850a8f5)

Update vfs version as I've added a const to the security_descriptor paramter in fset_nt_acl().
Need to watch the build farm to make sure I haven't broken the AIX or Solaris ACL modules.
Jeremy.
(cherry picked from commit 3869464895d51588ed156606051812efb26755a9)

Make map_errno_from_nt_status() a generic call, not just a cli specific one.
Remove some unused calls from vfs_acl_xattr. Test for SD's on existing files.
Jeremy.
(cherry picked from commit acaaf6e2effd34a159ceb6ff1dbfd2cfb650e613)

Fixed build warning "passing arg from incompatible pointer type"

The fix explicitly makes the conversion from timeval to time_t using the
existing time utility functions.

Compiling modules/vfs_smb_traffic_analyzer.c
modules/vfs_smb_traffic_analyzer.c: In function `smb_traffic_analyzer_send_data':
modules/vfs_smb_traffic_analyzer.c:173: warning: passing arg 1 of `localtime' from incompatible pointer type
(cherry picked from commit a1dd4fc647340238eac10d411f531e37dba901b9)

If name_to_fqdn fails, retry with the dns domain the DC gave us

This is a workaround for the cases where you want to join under a netbios name
that is different from your hostname, i.e. a name that can not be found in
/etc/hosts or dns. In these cases, name_to_fqdn fails or gives invalid results.
(cherry picked from commit 16e9a454404a5a70985caf15d1b987fe6332d195)

Log in the parent winbind log where a request is going
(cherry picked from commit 8d82a75ec9875137928c69370bbaf621bf4fc3fb)

idmap_adex: Add more debugging to the basic search function.

Log the dn of all located entries in order to verify search results.
(cherry picked from commit 46627792a69fd6d5a24f4ca69ad26c1f9e2213ff)

idmap_adex: Add log messages to dc_add_domain for easier debugging.

Part of continue work on BUG 5806.
(cherry picked from commit e8f3a35193bcbdf69c5eebdac532cfaa7b44edc2)

Fix some nonempty blank lines
(cherry picked from commit b3735cfc098e27a2af29a4671bc4e40b47314585)

Remove an unused variable
(cherry picked from commit a7ba58ebede769e944005adaecbeda237ba9b5e0)

Fix some missing error handlings
(cherry picked from commit 3a848570996c7df8c3230e0b3a9369b41658a650)

OS/X does not have rl_done in readline.h
(cherry picked from commit 02e303b3f08656e9bf1d59dc452c7f48f2cb0dea)

Fix an uninitialized variable warning

Not sure if we can ever not get domain and dns_domain, but gcc complained
(cherry picked from commit 50b86766442f27748d8d16223c8cd68fd162eabc)

Fix an unlikely memleak found by the IBM checker
(cherry picked from commit 09d76244a1e3eeeb1396a707e78325320e4e4f9e)

Fix an uninitialized variable found by the IBM Checker
(cherry picked from commit 2a4bf6cc1f7f20b9c95e4166b366990ed330604e)

Fix a potential NULL deref in line 258 found by the IBM checker
(cherry picked from commit d1c213b23649172ed684a22d3095f5ac95685dac)

"gwen/cc" does not like the double const :-)
(cherry picked from commit ab37affd6edec0d80109a7b635bc7fdb3b84eb6b)

Correctly fix smbclient to terminate on eof from server.
Jeremy.
(cherry picked from commit d8e629864a53d603c1bb2b08ec853d178d2d9dd3)

Simply our main loop processing. A lot :-). Correctly use events for all the previous "special" cases.
A step on the way to adding signals to the events and being able to merge the S3 event system with
the S4 one.
Jeremy.
(cherry picked from commit 8674c683e37b13554b6f0f121198bd4746c78796)

Document "cups connection timeout".
Jeremy.
(cherry picked from commit 3e5662bfda7856af5d6607c23397da0b18c9827a)

net_dns: Make "lwinet ads dns register" honor the "interfaces" parameter.

This is helpful on multihomed hosts that only require a subset
of IP addresses be registered with DNS.
(cherry picked from commit 17c9f8810598d8117befc896d12a597a52b0ac3b)

libaddns: Use the same prerequisite for DDNS update as Windows XP.

   Hostname, TYPE: CNAME, CLASS: NONE

This has to have been broken for ages.  I cannot see
how it would have worked in any environment.
(cherry picked from commit 6b6402bce318a48b0890ed6fc23ed5b30440927b)

Don't reject a successful alloc :-(.
Jeremy.
(cherry picked from commit 89df6afb7a16feb1a0b186965ff37a1f8990bcca)

Fix bug 5805: don't close stdout

- When calling setup_logging multiple times, the code was closing the debug
file descriptor before opening or assigning the new one.  We don't, however,
want to close the debug file descriptor if it is stdout.

Derrell
(cherry picked from commit 475c9a7ff5a96f7d1392610066fdd1ae28b7e039)

Fix bug #5080. Access to cups-printers via samba broken with cups 1.3.4, Unsupported character set.
Cups 1.3.4 expects utf8 to be used in all messages to/from the server. We may be using a
different character set so we need to use talloc utf8 push/pull functions in all communication.
Needs more testing. Don't release until I've done a thorough test. I also have a version for 3.2.x.
Jeremy.
(cherry picked from commit 2a2bb6b0757b1f2a6199474f93ea8d598575d650)

WHATSNEW: Start WHATSNEW for 3.3.0rc1.

Karolin
(cherry picked from commit 21d8210724dea1ceb94081cfebef73639d6f3741)

VERSION: Raise version number.

Karolin

Whitespace cleanup.
Jeremy.
(cherry picked from commit e782150175ff324b6e0ef1802a9e17f9a5aea560)

Fix use of DLIST_REMOVE as spotted by Constantine Vetoshev <gepardcv@gmail.com>.
This API is unusual in that if used to remove a non-list head it nulls out
the next and prev pointers. This is what you want for debugging (don't want
an entry removed from the list to be still virtually linked into it) but
means there is no consistent idiom for use as the next and prev pointers
get trashed on removal from the list, meaning you must save them yourself.
You can use it one way when deleting everything via the head pointer, as
this preserves the next pointer, but you *must* use it another way when not
deleting everything via the head pointer. Fix all known uses of this (the main
one is in conn_free_internal() and would not free all the private data entries
for vfs modules. The other changes in web/statuspage.c and winbindd_util.c
are not strictly neccessary, as the head pointer is being used, but I've done
them for consistency. Long term we must revisit this as this API is too hard
to use correctly.
Jeremy.
(cherry picked from commit 737825183115732de1f1d6d2bd89ce6402a65b20)

Turn the socket connections into a refcounted list - in the common case there'll now only be one socket per smbd.
Changed the format of the wire data to (a) include a version number (V1) as the first element. (b) removed the
";)" at the end an replaced it with a "\n". Receiver can change back if needed, and now receiver can just log
"as-is" to a text file (making testing easier). Added my (C). Sorry Holger, but I've changed quite a bit now.
Jeremy.
(cherry picked from commit 9a90ceac52f8623dde2bf50e630b10aebbc0e28d)

WHATSNEW: Update changes since 3.3.0pre1 and release date.

Karolin
(cherry picked from commit eef33fe6383afb5870aef0eef132cd0ba88bac83)

Convert to allocated strings. Use write_data(), not send as this doesn't correctly
deal with EINTR. Jim and Holger please check this still works.
Jeremy.
(cherry picked from commit bec55237c9c5687d947562524124b7aa76209b8f)

Remove current_user_info - not needed.
Jeremy.
(cherry picked from commit 093fc8c6bf1420c0f1e171598c4894b4df24862c)

Restructure the module so it connects to the remote data sink
on connect, and closes the socket on client disconnect. This should
make it much more efficient. Store the remote fd in a private data
pointer off the handle. Finally we need to remove the fstrings and
convert to allocated buffer storage.
Jeremy.
(cherry picked from commit e3e89077a5559eda39ea27f0ce6f9f0ebeba8f1e)

Fix the make test problem Karolin reported. Now rename_open_files actually works correctly we must emit the change notify before we change the name, not before.
Jeremy
(cherry picked from commit 48c0dff0c5e36313619fb39d2bfb57d49ed2141e)

re-added "winbind:ignore domains" patch

This option really is essential, as we discover again and again at
customer sites. Due to bugs in winbind some domains are toxic. When
you are installing at a site and a particular domain in a complex
setup causes winbind to segfault or hang then you need a way to
disable that domain and continue.

In an ideal world winbind could handle arbitrarily complex ADS
domains, but we are nowhere near that yet. If we ever get to that
stage then we won't need this option.
(cherry picked from commit f0f2fbd4800fb4a1df26f03f37b9972101657fdd)

fixed an (unlikely) memory leak
(cherry picked from commit 8b908862ec7de000865cd8dfa691f5643cb3d099)

fixed a segfault on the ctdb destructor code
(cherry picked from commit f495968c88f07b74a943da9e034084b72f5a1c8b)

removed unused variables
(cherry picked from commit c48fd3cee0a91617123d79bccff7ec1f6008fb26)

fixed segv on startup with trusted domains

With some setups, idmap_tdb2_allocate_id can be called before the
allocate backend is initialised, leading to a segv. This change
ensures that the db is opened in all paths that use it
(cherry picked from commit 8ec11590f471d2852e5ffd3d1d65da54bd2182ca)

Ensure we null out fsp after a close in all paths.
Jeremy.
(cherry picked from commit 35a074c24b9b320bb23e82d3f44e17c5fadbcf62)

Avoid freeing fsp twice when opening new_file fails. (Debian #431696)
If opening new_file fails, fsp would still be set to the files_struct of
old_file.
(cherry picked from commit bc0a340c61198879dc10ccbcc3c3ceb7f358b270)

docs: Adapt version strings in man vfs_smb_traffic_analyzer.

Karolin
(cherry picked from commit 662eda8458a304b224b6bc502f8030a07030b7c2)

Add an initial version of a manpage for the vfs_smb_traffic_analyzer module.
(cherry picked from commit 7857b783049b19d32a078927a46cb1070e863b45)

Second part of the fix for bug #5790 - samba returns STATUS_OBJECT_NAME_NOT_FOUND on set file disposition call.
This was my fault. I use a singleton cache (positive and negative) to speed up pathname based qfileinfo/setfileinfo lookups for alternate fsp's open on the same path. I only invalidated the negative cache on adding a new file fsp, as I incorrectly imagined the new fsp was put at the *end* of the open files list. DLIST_ADD puts it at the start, meaning any subsequent open wasn't seen once the cache was set. Doh !
Jeremy.
(cherry picked from commit 857433ec526caf73905dc9cb4fa25fb6355e5004)

Fix bug #5790 samba returns STATUS_OBJECT_NAME_NOT_FOUND on set file disposition.
We were checking that fd != -1 in file_find_di_XXX calls which is no longer
needed due to a change in internal semantics.
Jeremy.
(cherry picked from commit 276c8395f15bc18f10d455848f7038e3465f0d52)

WHATSNEW: Add Holger's VFS module to the list of changes.

Karolin
(cherry picked from commit c96dd7196606f53f60aff34b9358881fdd9d9f20)

libwbclient: fix doxygen warnings.

Guenther
(cherry picked from commit 461dfd9e4e225d4bf458baf1cc384be253c73e40)

wbinfo: add change-user-password command.

Guenther
(This used to be commit e572ede9995a66ae452ab25018b8df16101a2c2a)
(cherry picked from commit 1f0cd6a7443b08044d9fbc328fc729e3e2658e46)
(cherry picked from commit 410d69fde157a3bd7b3a92592e1e2741286b7cc5)

libwbclient: add wbcChangeUserPassword and wbcChangeUserPasswordEx.

Guenther
(cherry picked from commit 62e7b4aa32051bce34c890cb41270e5fe31111ca)
(cherry picked from commit 8627f684c64d0185245dea9b6b73aa2e2633a8f6)

libgpo: add gpext_security module build to configure.

Guenther
(cherry picked from commit 1861a8b8e1958c3118008f536817e93cca410fa4)

libgpo: fix copyright.

Guenther
(cherry picked from commit ff503694ff3db57e1b7e98f752e3c989a79f46bd)

libgpo: add security CSE skeleton.

Guenther
(cherry picked from commit 49fb18eaab6fb852cd5ba5971cbaa5e15d542654)

Fix bug #5797 - Moving readonly files fails. Reported by infomail@lordb.de.
We don't need to deny a DELETE open on a readonly file (I'm also adding a s4
torture test for this), the set_file_disposition call will return
NT_STATUS_CANNOT_DELETE if the delete-on-close bit is set
on a readonly file (and we already do this).
Jeremy.
(cherry picked from commit 14fec47f7171591e5799b81c90c3f5a856002ed0)

libgpo: fix invalid cast in scripts CSE.

Guenther
(cherry picked from commit af1878009e0bb7ef43bf445f5c167f3bbcc43fd2)

net: Make share type lookup a function.
(cherry picked from commit 5db15bc0f67b478ef95ec3e327f7a40d90c535bd)

s3-srvsvc: fix _srvsvc_NetShareAdd segfault.

parm_err is not a ref pointer.

Guenther
(cherry picked from commit 29942b7043c1a31ad4fb76d01ab19fd3dbf26f0a)

Use IPv4/v6 independent calls. Change safe_strcpy/cat to strlcpy/cat (this
needs changing to talloc_sprintf) and fix file descriptor resource leaks
in error paths. Jim and Holger please check !
Jeremy.
(cherry picked from commit eb249830ae4037244de6616b4ec8771568da34c6)

idmap: Fix typo is gid2sid() that was caching using idmap_cache_set_sid2uid()
(cherry picked from commit 89fa178011c44dc5c4bcfafc6643234e01b2d536)

WHATSNEW: Add summary of the idmap_hash plugin and Winbind's "name aliasing"
(cherry picked from commit bddc2956634194d7daedbf5cd18964811ef6fe5e)

Remove these no longer used include files.
Jeremy.
(cherry picked from commit 18cf398870e4d7cb336c9cf50f863611862af874)

idmap_hash: Fix the nss_info link during "make install"
(cherry picked from commit 701b9f34c986283fdead9ddd42147dd49d5e481e)

idmap_adex: Fix the nss_info install link.
(cherry picked from commit ba2c14e8474a20a9aabb07486a3c4e322980f96a)

Document the new hash and adex idmap/nss_info plugins.
(cherry picked from commit 77bc0be0536bcd6a3ce8283a708828bebbbffee3)

Document how to enable the name aliasing support in Winbind.
(cherry picked from commit 6dc81eb85b11851794619a1f2769d68cd9a54fb6)

s3:lib/ldb: Don't return already freed pointer on error.

metze

Signed-off-by: Karolin Seeger <kseeger@samba.org>
(cherry picked from commit 508cd214ef9abf0d69fbc7f5408d004162092b39)

Fix the new vfs_smb_traffic_analyzer build for static links
(cherry picked from commit 41887550fbacae887d9ad28559cc8bdcc6ac84ac)

SMB traffic analyzer vfs module from Holger Hetterich <hhetter@novell.com>

Used to gather data to feed to a database for live and historical
analysis of usage per user, per share, etc.

Helper apps to read the data still to come.  This one still needs to be
made ipv6 enabled (connection is made to the helper app).
(cherry picked from commit bd9499baa078ce3ea3640ce441a0e078b59a423f)

WHATSNEW: Update changes since 3.3.0pre1.

Karolin
(cherry picked from commit c8d5a9088f6a120e2da8c404b269d47a235fb933)

printing: Rename new parameter "cups timeout" to "cups connection timeout".

Karolin
(cherry picked from commit 417071214370e1d87417556af9e1410ef06403c7)

pam_winbind: fix bug #5784, build issue on solaris.

Guenther
(cherry picked from commit 8b5745d5caa2f41ab4b1717872f2ab0c23fbba2c)

WHATSNEW: Update changes since 3.3.0pre1.

Karolin
(cherry picked from commit 58e15a9d0fbf096b6b1587a34772c68f8c99393a)

printing: Add new parameter "cups timeout".

The default timeout for connections to CUPS servers is set
to 5 minutes in the CUPS libraries. The smbd hangs on startup
until the timeout is reached if the CUPS server is unreachable.
This parameter makes the timeout configurable. The default value
is set to 30 seconds.

Karolin
(cherry picked from commit 263cacf1f2dd4003862184ea3a2716ace0dbb086)

Fix winbindd crash in an unusual failure mode. Bug #5737. Based on original patch from shargagan@novell.com
Jeremy.
(cherry picked from commit d5c490d79024ee41544512f7968999a6b3a313dc)

Fix bug #5783 FindFirst fails where search pattern == mangled filename.
That was an old and subtle bug.
Jeremy.
(cherry picked from commit 86a0c271ef467810810d1adda982bbb0dc8b928e)

idmap_adex: Add new idmap plugin for support RFC2307 enabled AD forests.

The adex idmap/nss_info plugin is an adapation of the Likewise
Enterprise plugin with support for OU based cells removed
(since the Windows pieces to manage the cells are not available).

This plugin supports

  * The RFC2307 schema for users and groups.
  * Connections to trusted domains
  * Global catalog searches
  * Cross forest trusts
  * User and group aliases

Prerequiste: Add the following attributes to the Partial Attribute
Set in global catalog:

  * uidNumber
  * uid
  * gidNumber

A basic config using the current trunk code would look like

 [global]
      idmap backend = adex
      idmap uid = 10000 - 29999
      idmap gid = 10000 - 29999
      winbind nss info = adex

       winbind normalize names = yes
       winbind refresh tickets = yes
       template homedir = /home/%D/%U
       template shell = /bin/bash
(cherry picked from commit 62785b8e2aa862b7eb85e20a11f7a29e1e8b2825)

Attempt to fix bug 5778

Jeff, Steve, please check!
(cherry picked from commit 110756cc7bcaed5a9d6aa58f3b3fe4481f8d1f31)

WHATSNEW: Add changes since 3.3.0pre1.

Karolin
(cherry picked from commit 9c464946c1f77a6f95da4a36fd590ec75734432d)

Add a comment for the bump of version
(cherry picked from commit 91935b08aa38f3022b500424546d05a747d7ceb3)