s3:libsmbconf: use talloc_free instead of TALLOC_FREE in reg backend

Michael

libsmbconf: add method is_writeable() and wrapper smbconf_is_writeable()

This allows for per-config-source checking of write support.

Michael

libsmbconf: add backend_requires_messaging() method to libsmbconf.

In a clustered environment, the registry backend needs messaging
to be set up since ctdb requires this.

Michael

libsmbconf: add "includes" to the forbidden_valnames[].

This removes the warning messages 'Unknown parameter encountered: "includes"'.

Michael
(cherry picked from commit b20019ceaaf4a8964792f6ba37f50f91b6847e7f)

libsmbconf: fastpaths first in smbconf_reg_valname_valid().

Before dropping into lp_parameter_is_valid().

Michael
(cherry picked from commit 59beb7acd777a4c224dce90fbbff7a137d2b89c1)

remove an unused variable

Michael

remove an unused variable

Michael

Fix a O(n^2) algorithm in regdb_fetch_keys()
(cherry picked from commit d98c43982c56ef788144a3fd67cdd65a9f95f2be)

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

Fix a buffer handling bug when adding lots of registry keys

This is *ancient*... From 2002, and nobody noticed until someone added lots of
shares using net conf... :-)
(cherry picked from commit 36ae846d15027df5e3a02ffabb08183dad9f6517)

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

Merge commit 'origin/v3-2-test' into v3-2-ctdb

s3:packaging: fix parsing of include.h in fill-templates

The content has changed slightly in master for the merged build,
and we must be more exact when parsing the lines.

Michael

s3:packaging: fix path to calling update-pkginfo after pushd in fill-templates

Michael

s3:packaging: paths cosmetics in fill-templates

Michael

enable IPv6 support for NetBSD, FreeBSD

most systems include netinet/in.h via netdb.h but *BSD don't.

Merge commit 'origin/v3-2-test' into v3-2-ctdb

s3:build: some implementations of "ln" are picky about the order or arguments

Michael

Fix resume handle for _samr_EnumDomainGroups

Merge commit 'origin/v3-2-test' into v3-2-ctdb

packaging: make update-pkginfo callable from any directory

Michael
(cherry picked from commit 7ea90835ebf6e2f2c6f325760800d8361d087ede)

packaging: use "find" instead of "du" to find files... :-)

Michael
(This used to be commit 768aa82361a2c4b0aab71433b7593723a2868f0a)
(cherry picked from commit fd7635a1161e25d88a8252a5cf9de70ca2fe4232)

packaging: remove commented out stuff from update-pkginfo

Michael
(cherry picked from commit d714e58ec1f74897cd13e7cd9dd9003600052b45)

packaging: don't clutter the output of fillup-templates  with popd output

Michael
(cherry picked from commit 0c711935d30b500a15ef06fd658d382b2c1e2c84)

packaging: fix fill-templates call to update-pkginfo to pushd appropriately first

Michael
(cherry picked from commit c28d4ae537d9fc0b76eae758d32356eaf1113c89)

packaging: add script fill-templates

This is a standalone-wrapper for update-pkginfo, which is
usually called from create-tarball. It basically repeats
some functionality of create-tarball.

Michael
(cherry picked from commit ca23469a906bab690162184e8d3949897f7b5a67)

s3-spoolss: fix memleak in get_remote_printer_publishing_data().

Guenther
(cherry picked from commit 97f1514ce95800f1296b1979f1b34dcc5d56a376)

Merge commit 'origin/v3-2-test' into v3-2-ctdb

s3 build: Fix "assignment discards qualifiers from pointer target type" warnings

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

s3:winbind_user: fix "getent passwd" to allocate new uids.

"getent passwd" used to fill the idmap cache with negative
cache entries for unmapped user sids.

Don't pass domain name unconditionally to idmap_sid_to_[ug]id().
idmap_sid_to_[ug]id() only creates new mappings (allocating
idmap backends tdb, tdb2, ldap...) when the domain name passed
in is "".

Note that it is _wrong_ to directly call the idmap_sid_to_[ug]id()
functions here, in the main winbindd. The correct fix would be
to send a sid_to_[ug]id request to winbindd itself, but this needs
more work to prepare the async mechanisms, and we nee a quick
fix for getent passwd now.

Michael

Fix count in _srvsvc_NetDiskEnum()

Without this, remotely adding a share via MMC fails

s3:winbind_group: fix "getent group" to allocate new gids.

"getent group" used to fill the idmap cache with negative
cache entries for unmapped group sids.

Don't pass domain name unconditionally to idmap_sid_to_gid().
idmap_sid_to_gid() only creates new mappings (allocating
idmap backends tdb, tdb2, ldap...) when the domain name passed
in is "".

Note that it is _wrong_ to directly call the idmap_sid_to_gid()
functions here, in the main winbindd. The correct fix would be
to send a sid_to_gid request to winbindd itself, but this needs
more work to prepare the async mechanisms, and we nee a quick
fix for getent passwd now.

Michael
(cherry picked from commit 8c64302915bde8a5400b575389b12e0eaf2cf140)

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

Memory leaks and other fixes found by Coverity

(This is the winbindd_group.c portion of the patch - obnox)

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

Memory leaks and other fixes found by Coverity

(This is the winbindd_user.c portion of the patch - obnox)

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

s3:configure: autodetect ctdb/cluster support

We need to make sure ctdb has transaction support
and we autodetect ipv6 support.

metze
(cherry picked from commit 11163720e6f30e83f523c5009e1e60b5cf28cda6)
(cherry picked from commit f87f4842cc0ffc62d76ee5350b94f7bda735aa71)

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

s3:ctdbd_conn: canonicalize ips before sending them to ctdbd

This makes samba work with older ctdb versions.

metze
(cherry picked from commit 7540d13956a8a69d25f6fbbc4f9a3417e91db1d2)
(cherry picked from commit a7fcaff295c3a36e2b783398ac212500eef33ecb)

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

Merge commit 'origin/v3-2-test' into v3-2-ctdb

s3: reinit_after_fork() should reinit the event context before the
messaging context

Because messaging_reinit() may add events to the event context, which will
removed by event_context_reinit().

metze
(cherry picked from commit 56af6112d67aa9dd3757297fbe29ce1a3eabf00d)

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

Merge commit 'origin/v3-2-test' into v3-2-ctdb

mount.cifs: initialize rc to 0 in main

The value of rc in main() isn't initialized in the declaration. This
wasn't a problem before, but Shirish's fakemount patch can make it so
that we return the uninitialized variable if the -n flag is used.

Fix this by initializing rc to 0.

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

mount.cifs: add fakemount (-f) and nomtab (-n) flags to mount.cifs

...so that these options work correctly when passed in by mount(8).

s3:net conf: remove check for sharename being a usernam in "net conf addshare"

This is useless and can be overriden by "net conf setparm" anyways.

Michael

docs: fix two typos in the mount.cifs manpage

Michael
(cherry picked from commit 145fe37766cf1ecffb16a03b58b44d08f7ed7558)

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

Don't try and delete a default ACL from a file.

Have nmbd check all available interfaces for WINS before failing

When nmbd is acting as WINS, it picks the first interface's IP as WINS
server's IP. If the first interface's IP is zero, we will just quit
(even though we might have other interfaces with valid IPs).

This patch makes nmbd look at all interfaces and pick the first interface
with a valid IP as the WINS server's IP.

umount.cifs: clean-up entries in /etc/mtab after unmount

This patch removes the remaining entry in /etc/mtab after a filesystem
is unmounted by canonicalizing the mountpoint supplied on the command
line.

Please refer to bug 4370 in samba bugzilla.

s3-net: remove unused ENUM_HND.

Guenther
(cherry picked from commit a713060a6e36706cfd5b997aa4cbfabf4e180c84)

Fix bug #6085 - In vfs_default.c change utime( ) call.

Probably fixes a crash during name resolution when log level >= 10
and libc segfaults if printf is passed NULL for a "%s" arg
(eg. Solaris).

Merge commit 'origin/v3-2-test' into v3-2-ctdb

s3: Fix 'net rpc join' for users with the SeMachineAccountPrivilege.

This used to be commit fda8abac in master.

packaging(RHEL-CTDB): makerpms.sh: build winbind-32bit libs in the 64bit build

Michael

packaging(RHEL-CTDB): Build winbind-32bit package in the 64bit build

Michael

WHATSNEW: Update changes since 3.2.7.

Karolin

packaging(RHEL-CTDB): extend makespec.sh to extract VENDOR_PATCH from version.h

Michael

packaging(RHEL-CTDB): The former release number has basically moved into VERSION

Michael

VERSION: add vendor suffix ("ctdb") and vendor patch level

Michael

WHATSNEW: Update changes since 3.2.7.

Karolin

Adjust regex to match variable names including underscores

This is required to get the CIFSUPCALL_PROGS setting extracted from
config.log.

Conditional install of the cifs.upcall man page

Only install the cifs.upcall man page if CIFSUPCALL_PROGS was set while
configure.

docs: Describe "service" in man mount.cifs.

This fixes bug #5346.
Thanks to the Debian Samba package maintainers for reporting and providing a
patch!

Karolin
(cherry picked from commit 73f4fc1f802f31459b70dba4777d142d00fcdd92)
(cherry picked from commit ab4768452811e67f6606253b5a79101184f777d0)

WHATSNEW: Update changes since 3.2.7.

Karolin

fix build with external dns_sd libraries

This used to be commit 16f07280 in master.
(cherry picked from commit 89919a131b234485564ff2f2f20e3d9f9eb83522)

s3-docs: fix typo in eventlogadm manpage.

Guenther
(cherry picked from commit fb7b92abc2bea4b3d91cdd896f02db88065a4b8f)
(cherry picked from commit 0cc588ac53316299d2e1ff7d61947fd864d412f9)

add Tru64 sub-second resolution timestamp support

add configure check for Tru64 sub-second timestamp resolution

add missing semicolons

the fixed configure check led to a missing semicolon in the now activated BSD
code. Then this error was even copypasted into the new AIX code. grrr

Following Björn JACKE's patch, unify the detection of the timespec code in configure.in, and the application of it in time.c
Jeremy.

add configure check for AIX style sub-second resolution support

rather cosmetic fix for failed birthtime configure checks

give configure check "sub-second timestamps without struct timespec" a chance to succeed

WHATSNEW: Update changes since 3.2.7.

Karolin

Use talloc_tos() instead of the talloc NULL context.

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

Avoid valgrind errors

In event handlers, we might destroy other events that are pending in the lists.
We can only run one event safely per select call.

Yes, I've seen these valgrind errors :-)

Jeremy, with ccdd921e61 you had checked in the change to run multiple events.
Do you remember why it was necessary and could not be solved in a different
way?

Volker

Fix a valgrind error when the socket dies

Don't reference anything that might have been deleted in the async_req_error
call.

vfs_gpfs_prefetch: correctly return -1 on error condition in smbd_gpfs_fcntl()

Michael

VERSION: Raise version number up to 3.2.8.

Karolin

WHATSNEW: Prepare WHATSNEW for 3.2.8.

Karolin

Fix detection of dns_sd libraries.
Jeremy.

libreplace: fix detection of netinet/ip.h on solaris 8

(The test needs to additionally include <netinet/in_systm.h>.)

Michael
(cherry picked from commit a223b096700bc54b446a0e152c842b088566720b)

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

libreplace: fix bug #6066 - netinet/ip.h present but cannot be compiled

under solaris

Michael
(cherry picked from commit 1b7beaed69823b3c401bc4c0b1c502b671169168)

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

Document default of the printing config variable.

Signed-off-by: Andreas Schneider <anschneider@suse.de>
(cherry picked from commit d8f15e4efc00b9d509ff5761e9ca8ff5c6f443f7)
(cherry picked from commit 541704e253f06f8b33006b6c27f51b934c2eb51f)
(cherry picked from commit 69880dac28ac1681bb0c3d84280494de3d788cc6)

docs: Add manpage for vfs_shadow_copy2.

Karolin
(cherry picked from commit 512cf0ad50d30fcb35a28b0ef1b859d754f42a99)
(cherry picked from commit 7ca4b8ec65d5f8794c58f27f3f17033143732897)
(cherry picked from commit addd0f41eeaf255217f1ab156e970c41a929aa43)

s3-eventlog: fix various invalid memcpy in read_package_entry().

Guenther
(cherry picked from commit 75fff603625e0259f187e0fa47efd85887092fae)

s3-eventlog: fix buffer allocation in eventlog read call.

This broke with f6fa3080fee1b20df9f1968500840a88cf0ee592 back in April 2007...

Guenther
(cherry picked from commit 547c70e896b6890dd5f7b911e4e0ca107c8a3fa3)

docs: Fix formatting issue in man libsmbclient.

Karolin
(cherry picked from commit aa2fb0efb7ce2bc732569b4baf20c57375fdce59)
(cherry picked from commit e1e3a0554fcaabacc5bec51c213622271a83ad04)

lib/replace: add defines to let the callers find out if pwrite and pread are thread/fork safe

metze
(cherry picked from commit 7fc7ee9331d0539359ad88c527f59d5fdf212209)
(cherry picked from commit d74b63091781e442e42b1fa7881e903a0bcc928c)

Apply same logic fix for #4308 Excel save operation corrupts file ACLs
to NFSv4 ACL code as this uses the same flawed logic as posix_acls.c.
Jeremy.

Fix logic error in try_chown - we shouldn't arbitrarily chown
to ourselves unless that was passed in.
Jeremy.

Second part of the attemt to fix #4308 - Excel save operation corrupts file ACLs.
If the chown succeeds then the ACL set should also. Ensure this is the case
(refactor some of this code to make it simpler to read also).
Jeremy.

Another attempt to fix bug #4308 - Excel save operation corrupts file ACLs.
Simo is completely correct. We should be doing the chown *first*, and fail the
ACL set if this fails. The long standing assumption I made when writing the
initial POSIX ACL code was that Windows didn't control who could chown a file
in the same was as POSIX. In POSIX only root can do this whereas I wasn't sure
who could do this in Windows at the time (I didn't understand the privilege
model). So the assumption was that setting the ACL was more important (early
tests showed many failed ACL set's due to inability to chown). But now we have
privileges in smbd, and we must always fail an ACL set when we can't chown
first. The key that Simo noticed is that the CREATOR_OWNER bits in the ACL
incoming are relative to the *new* owner, not the old one. This is why the old
user owner disappears on ACL set - their access was set via the USER_OBJ in the
creator POSIX ACL and when the ownership changes they lose their access.

Patch is simple - just ensure we do the chown first before evaluating the
incoming ACL re-read the owners. We already have code to do this it just wasn't
rigorously being applied.
Jeremy.

build-docs: cleanup exit of the script

exit in the directory where it was called using pushd/popd.

Michael
(cherry picked from commit b319549f129b1c79afc9bfd4a84f2730b96d69a3)

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

s3:docs: clean build/catalog.xml in "make clean"

Michael
(cherry picked from commit 5e21fc3506f2ba7b1135b1acad2697dfb86b5df0)

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

s3:docs: clean generated .png images in "make clean"

Michael
(cherry picked from commit 9b32e839bec8611c30745607a3a6b124d5b34c01)

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

s3:docs: fix ommission in fix of (real)distclean targets

Michael
(cherry picked from commit 37412017c5dd2f05a7f4bbe0410a6e00ce4805e5)

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

s3:docs: fix distclean target and add realdistclean target

- remove stuff created by configure in distclean
- remove stuff created by autoconf in realdistclean

Michael
(cherry picked from commit 65c92fea3d18c3520ff2a1e53a0c5c8825c9788f)

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

Depend on latexfigures files directly as using a rule in between causes problems.
(This used to be commit bf3c2773f94c1db29a8a3e5935ff587f16f9a905)
(cherry picked from commit 5f6d0078d41e790ab4fa7dac15294821cdb4d4f0)

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

Use double colon targets.
(This used to be commit 825cf91fa8b30a67d04cbda8885ff62a4c26458f)
(cherry picked from commit 323be4a6907e4915bb76aa103bf5b868f0b459b1)

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

Add test target in Makefile.
(This used to be commit ac0d768676b1b8105a9141169b1afa248df6ce66)
(cherry picked from commit 0f3ba4b766ed83d2a38e6cae8db8690374d88989)

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

Make the make output a bit less chatty.
(This used to be commit 1fa4ef553c437bae07389c0f6a5410ba22ee4905)
(cherry picked from commit 70f52c06ba1d0d9638f1dfb8c42cd0d70364e9df)

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

build_docs: Use 'make distclean' instead of 'make clean'.

This fixes bug #6058.
Thanks to Christian Perrier for reporting!

Karolin
(cherry picked from commit 162e4b66601b41dd5adb24e192f3b5a91dfb41cc)

Merge branch 'v3-2-test' of ssh://jra@git.samba.org/data/git/samba into v3-2-test