s3:smbd: correctly verify stream names to pass RAW-STREAMS

metze

s3:smbd: '|' isn't allowed in windows filenames

metze

Fix a debug message, append the correct \n
(cherry picked from commit 263c1ca96bd68dced4577a2504073e1140fa26a1)

Fix the offset checks in the trans routines

This fixes a potential crash bug, a client can make us read memory we
should not read. Luckily I got the disp checks right...

Volker
(cherry picked from commit 64a1d80851da5b05e70ec6c96f6e9bd473748369)

Fix circular dependency error with autoconf 2.6.3.

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

Fix bug #5914 - Build failure: redefinition of struct name_list
Jeremy.

s3-samr: fix _samr_LookupNames return code.

Guenther
(cherry picked from commit dc7c2d63b3ce142959aa863f000fa9140de99eb0)

zfsacl: return is not a function.

Michael

zfsacl: Prevent calling POSIX ACL vfs methods on zfs share.

This is a proposed fix for Bugs #5135 and #5446.

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

[PATCH] Allow SYSLOG_FACILITY to be modified with a new configure option called --with-syslog-facility

Fix bug #5873 - ACL inheritance cannot be broken. This regresses #4308, but that will have to
be fixed another way.
Jeremy.

Revert c76e4791d3ae4f180fba209c29bd3774c00858cf - "Fix the logic bug that caused us to run into kernel oplocks on an open for a stream inside a file with stream_xattr module. On opening the base_fsp we must break existing oplocks." as it broke make test.
Jeremy.

Fix the logic bug that caused us to run into kernel oplocks on an open for a stream inside a file with stream_xattr module. On opening the base_fsp we must break existing oplocks.
Jeremy.

[PATCH] FreeBSD configure check for backtrace_symbols

On FreeBSD backtrace_symbols is defined in libexecinfo.so.1.  Look for it
there as well.

Use fxattr calls whenever possible (trying to work around the strange Linux kernel oplock bug).
Jeremy.

winbindd: make all winbind rpc-methods static.

Now that the methods are no longer needed in winbindd_ads,
we can make them static again.

Michael

winbindd_ads: use the reconnect methods instead of the rpc methods directly

Some of the ads methods just point to the rpc methods.
This makes winbindd_ads use the reconnect methods instead of
calling the rpc methods directly in order to prevent
negative cache entries for e.g. name_to_sid, when the dc
has closed the connection without sending a reset.

Michael

winbindd_ads: prevent negative GM/ cache entries due to broken connections

The ads lookup_groupmem() function calls lda_lookupsids to resolve sids
to names. This is tried only once. So in case the connection was broken,
e.g. closed by the server (without a reset packet), there will be an empty
GM/ cache entry for the requested group which will prevent proper working
of access checks among other checks for the expiry period.

This patch works around this problem by retrying once if the lsa_lookupsids
call fails, re-establishing the dc-connection, as we already do in many other
places (e.g. the winbindd retry methods for the rpc layer).

Michael

Second part of the fix for bug #5903 - vfs_streams_xattr breaks contents of the file
Jeremy.

Ensure we always reply in reply_printopen().
Jeremy.

First part of fix for bug #5903 - vfs_streams_xattr breaks contents of the file.
Restructures parts of open code so that fsp must be allocated before calling
open_file_ntcreate(_internal). Also fix up file ref-counting inside files.c.
Jeremy.

Fix bug #5909 - MS-DFS does not work on Vista, if link name includes multibyte character.

Second part of fix for bug #5891 - smbd crashed when viewing the eventlog exported by "eventlog list". Don't leak memory on error paths.
Jeremy.

eventlog: don't crash in sync_eventlog_params().

When freeing the talloc ctx at the end of the routine,
it must be a talloc ctx created inside. talloc_tos() needs
to be valid after the function finishes, since callers
(may) have data attached to it.

Michael

Signed-off-by: Günther Deschner <gd@samba.org>
(cherry picked from commit 113c5d28edabdd17a2507e68aad612aa21c758f3)

This code mixes up int and uint32 when pulling out of a TDB. This is very bad. Fixing... May fix bug #5891, not sure.
Jeremy.

Fix error in commit for bugfix "Fix bug #5904 - libnss_wins causes SIGABRT while servicing getaddrinfo() request."
Jeremy.

Fix bug #5904 - libnss_wins causes SIGABRT while servicing getaddrinfo() request.
Jeremy.

Fix extended DN parse error when AD object does not have a SID.

Some AD objects, like Exchange Public Folders, can be members of Security
Groups but do not have a SID attribute.  This patch adds more granular return
errors to ads_get_sid_from_extended_dn().  Callers can now determine if a parse
error occured because of bad input, or the DN was valid but contained no SID.

I updated all callers to ignore SIDless objects when appropriate.

Also did some cleanup to the out paths of lookup_usergroups_memberof()

Whitespace and >80 column cleanups.

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

Fix Bug #5898. ("net rpc shutdown" fails).

Guenther

Fix bug #5908 - Samba 3.0.32 - internal change notify on share directory fails"

libsmb: Fix up pointer passed to cli_send_trans in cli_link_internal

Seems like this one fell victim to the pstring removal. The other callers seem
to be fine, even though removing the casts would make the code less confusing.

Fix bug #5906 when running winbindd on a Samba PDC. Winbindd crash on 'getent group'.

When clearing out vuid cache remember to set entries = 0.
Jeremy.

Fix bug #5900 reported by monyo@samba.gr.jp - vfs_readonly.so does not work.
Jeremy.

s3 create-tarball.sh: Remove dashes in git commands.

Newer git versions (e.g. 1.6.0.2) do not provide the 'git-' commands
any longer.

Karolin
(cherry picked from commit 0cba859f12177aaf3ef2d96663f0a51f61c24d56)
(cherry picked from commit 3d64e67de2b50f7b781aa3f5ad13ec8e866d4333)
(cherry picked from commit aa41a80969bd413d5922c602c7309bd8c05f3181)

net: Fix documentation of net rap printq info

The man-page (see net(8)) and the usage-info (call "net help rap printq") of
'net rap printq' do contain an option "list" but in net_rap.c
the option is named "info".

Rename the option "list" in the documentation (man-pages + usage) to "info" to
match the code.

Signed-off-by: Kai Blin <kai@samba.org>

Fix bug 5901: Default for streams_depot location

fix build warning.

Guenther
(This used to be commit 507660706901a79544d436046127947e4baa7e52)
(cherry picked from commit 1939177582867fb54a416f3ea979ebd8d4335885)

Make memcache_add_talloc NULL out the source pointer

This is an orthogonality measure to make clear this pointer now belongs to the
cache.

Actually finish memcache_add_talloc

This fixes a memleak found by Martin Zielinski <mz@seh.de>. Thanks for
looking closely!

Volker

Use the correct cups-devel test for HAVE_IPRINT also
(cherry picked from commit a42d7e1146e7469062ead2c8f22f549a48154e03)

Correctly check for presence of cups-devel

On SuSE at least cups-config belongs to cups-libs which does not contain the
headers which are in cups-devel.
(cherry picked from commit 2408d2493f7c6e0beee58febd547516bbeb0929d)

make some more parts of security.idl public and re-run make idl.

Guenther

re-run make idl.

Guenther
(cherry picked from commit 19568f4e4e94266a7792e6dce8eaa5086f435fc3)

security.idl: sometimes ACEs have some padding at the end

metze
(cherry picked from commit 2dc120377baec2c8b7c9b6fe40d15218588c97fd)

Rename some variables in getpwnam_alloc() for clarity

sys_pwnam doesn't return talloced memory, so don't mix up the returned struct.

re-run make idl.

Guenther
(cherry picked from commit 86deff8d6e6b8e094b15a4e5392098d9b23e5ff5)

Fix PNP_GetHwProfInfo() (fixes Bug: #5888).

Guenther
(cherry picked from commit 111ec984729fe2c47e173dab82f438bcacf42d1c)

Pretty print out SD's on get/set.
Jeremy.

Fix memory leak in error path, spotted by Martin Zielinski <mz@seh.de>.
Jeremy.

Fix bug 5891] : smbd crashed when viewing the eventlog exported by "eventlog list"
Don't mix TALLOC and SAFE_FREE().
Jeremy.

Fix bug  5889. "delete veto files = no" seems to break.
Jeremy.

ntlm_auth: Put huge NTLMv2 blobs into extra_data on CRAP auth

This fixes bug #5865

mount.cifs: use lock/unlock_mtab scheme from util-linux-ng mount prog

The util-linux-ng sources have a good, but rather complex scheme for
locking the mtab before updating it. Mount helpers need to follow the
same scheme. Advisory locking only works if everyone is using the same
locking scheme.

Copy the routines we need from util-linux-ng into a separate source file
and then have mount.cifs and umount.cifs link in this object.

The long term goal is to have these routines in a separate helper
library (libmount). Mount helpers can then dynamically link in that lib.
Until that happens, this should serve as a suitable stopgap solution.

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

Fix bug #5825 - Account locking out doesnt work with an LDAP backend.Based on a problem found by Boyang. Only the pdb_nds backend implements login attempts so this was broken for tdbsam and ldap.
Jeremy.

Add reference to bug #4308 to remind me to add regression test to smbtorture.
Jeremy.

Fixed typo in source/utils/net_rap.c

Ignore 3.0 style invalid group mappings during upgrade to ldb

Fix bug triggered by the RAW-SAMBA3OPLOCKLOGOFF test

Fix bug 5860: safe_strcpy gives a nasty error message for overlong strings

Thanks to Robert Dahlem <Robert.Dahlem@gmx.net> for reporting this!

use glibc sys/inotify.h header

when we first added the inotify code glibc didn't have the inotify
functions yet. Now that it does we can use the official header and
avoid the asm/unistd.h syscall workaround

auth: Fix build warning.

Guenther
(cherry picked from commit 4661ef625a6522d6f859b83e3e3702f01d0b952f)
(cherry picked from commit 60649a74cdf2594bc89c301025f86d23caba91c2)

Fix CID: 456 - resource leak on function exit.
Jeremy.

Fix CID 606 - failed to return -1 after socket() fail.
Jeremy.

Fix CID 574 - tidies up the code expression.
Jeremy.

Coverity fix CID: 592 - null deref (can't happen but doesn't hurt to be sure).
Jeremy.

Coverity fix #CID: 607 - resource leak on error path.
Jeremy.

winbind: fix smbd hanging on Solaris when winbindd closes socket.

On some versions of Solaris, we observed a strange effect of close(2)
on a socket: After the server (here winbindd) called close, the client fd
was not marked as readable for select. And a write call to the fd did
not produce an error EPIPE but just returned as if successful.

So while winbindd had called remove_client(), the corresponding smbd
still thought that it was connected, but failed to retrieve answers
for its queries.

This patch works around the problem by forcing the client fd to
the readable state: Just write one byte into the socket before
closing.

Michael

build: fix bug #5765 - fix installlibs on solaris by using portable "test -r"

instead of test -e, which /bin/sh on solaris does not know.

Michael

build: fix bug #5677 - fix test_{shlibs,nss_modules,pam_modules} on Solaris

and other systems where sh does not support "export FOO=bar"
by separating setting and exporting the variable.

Thanks to Yasuma Takeda <yasuma@osstech.co.jp> for the patch.

Michael

Error return is boolean false, not -1

- There were a few places in SMBC_getatr() that returned -1 instead of a
  boolean.  -1 was intended to mean error, but that's what False/false is for,
  and the usages of this function assume that it returns a boolean false as
  the error condition.

- per Jelmer's request, use false vs. False in new code, even if not making
  changes globally.

Derrell

s3-samr-server: unify callback convention: _samr_UserSetInfo.

Guenther
(cherry picked from commit aa301e82d0c44c6a733e8be2546d661ea56512ef)

s3-samr-server: unify callback convention: _samr_QueryDomainInfo.

Guenther
(cherry picked from commit ac2c35bc379de83091644455dbeba0bea3e5ceb6)

s3-samr-server: fix return code in _samr_QueryDisplayInformation.

Guenther
(cherry picked from commit 30fa6c3ba19a8f816043405ba5d9eec84dd1c97b)

Revert the 4d83924470b267f2267abd62a3be2ac3ae373d7b change. The server_info struct *must* own the ptok pointer inside it.
Jeremy.

use correct argument

Fix bug #5929 - vfs_prealloc does not build with option --with-cluster-support and GPFS
(cherry picked from commit a1e4637922a565278cceac7b2fe95f82ad049f4a)

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

winbindd: fix more horrible brace mis-indentations in winbindd_group

Sorry, but I can't bear this.

And while my vim is smart enough to cope with this,
it confuses _me_, and I have seen an emacs confused as well.
(Which comes as no surprise, almost human as it is...)

Michael
(cherry picked from commit addbddf90a4935f7fda8ed4acbf5a4574d7fc92b)

winbindd: fix more misindentations of top level braces in winbindd_group

Michael
(cherry picked from commit 21dedb62d866bb0a2ae808c8d5a411c7f0fb8604)

winbindd: fix horrible mis-indentation of toplvl braces in getgrsid_sid2gid_r

to make code more readable

Michael
(cherry picked from commit d28a0f970e06c67729d71d505c35ae1d49ad6ccc)

winbindd: don't open a new dc connection in winbindd_getpwnam (main winbindd)

This is just to find the corresponding domain struct.
Actual connection is handled by the domain child.

Michael

winbindd: don't open a new dc connection in winbindd_getgrnam (main winbindd)

This is just to find the corresponding domain struct.
Actual connection is handled by the domain child.

Michael

packaging(RHEL-CTDB): don't show failure messages in check for ccache

by fixing the order of "> /dev/null 2>&1"

Michael

packaging(RHEL-CTDB): fix the build when ccache is not found.

Michael

vfs_tsmsm: fix potential segfault (freeing uninitialized buffer)

Michael

packaging(RHEL-CTDB): Bump release number to ctdb.52 for the next RPM.

Michael

winbindd/nss_info: make nss_domain_list_add_domain() static.

Michael

packaging(RHEL-CTDB): explicitly build the gpfs module

to prevent potetential problems on PTF8 systems where there
is no valid autodetecting configure check yet.

Michael

winbindd/nss_info: remove unused variable from nss_init()

Michael

packaging(RHEL-CTDB): use ccache if available

Michael

winbindd/nss_info: fix default backend handling for ad backends.

This fixes "winbind nss info = rfc2307" (or sfu or sfu20).
Originally, only explicitly configured domains (like "rfc2307:domain")
worked with the ad module, since the domain name was not passed
backe to the module. This is fixed by recording the first backend
listed without domain in the "winbind nss info" parameter as the
default backend, and creating new nss_domain entries (using this default
backend) on the fly as requests for domains which are not explicitly
configured are encountered.

Michael

winbindd/nss_info: whitespace cleanup.

Remove trailing spaces and fix tab / space mixup.

Michael

winbindd_cache: add debugging to get_nss_info_cached()

Michael

winbindd/nss_info: add entry debug message to nss_get_info()

Michael

winbindd/nss_info: add debugging to nss_init()

Michael

winbindd/idmap_ad: add entry debug message to nss_ad_get_info()

Michael

winbindd/idmap_ad: add support for trusted domains to idmap_ad (bug #3661)

This initial fix does at least work for explicitly configured domains.

The patch has a few disadvantages:

1. It does work only for explicitly configured domains, not with
   the default backend (idmap backend = ad), since it relies on the
   domain name being passed in via the idmap_domain. One workaround
   for this would be to create clones of the default idmap_domain
   for domains not explicitly configured.

2. It calls find_domain_from_name_noinit() from idmap_ad_cached_connection.
   The problem here is that only the NetBIOS domain name (workgroup
   name) is passed in via the idmap_domain struct, and the module
   has to establish a connection to the domain based on that information.
   find_domain_from_name_noinit() has the disadvantage that it uses the state
   of the domain list at fork time (unless used from the main winbindd).
   But this should be ok as long as the primary domain was reachable at
   start time.

For nss_info, the situation is similar - This will only work for domains
explicitly configured in smb.conf as follows:
"winbind nss info = rfc2307:dom1 sfu:dom2 rfc2307:dom3 template:dom4"
Setting the default nss info to one of the ad backends (rfc2307, sfu, sfu20)
will fail since the domain name is not passed in with the nss_domain_entry.

Michael

winbindd/idmap_ad: refactor core of nss_{sfu|sfu20|rfc2307}_init to common function.

Michael