Do not pull samAccountName twice

s3:smbd: only check the next_status for related requests

metze

s3:smbd: response with the same SMB2 flags as given by the client

...but also add the response flag.

metze

s3:smbd: only pass the chained file handle for related compounded SMB2 requests

metze

s3:smbd: make sure we pad compounded SMB2 responses to 8 bytes

metze

s3:smbd: skip file handle checks for SMB2 compounded requests

metze

s3:smbd: more validation of the incoming SMB2 requests

metze

s3:smbd: for now ignore all non NBT Session Message requests for SMB2

metze

s3:smbd: don't include 1 byte padding for the dynamic buffer of SMB2 responses

It seems that Windows 2008 and Windows 7 doesn't do this anymore.

metze

SMB2-COMPOUND: add some tests for invalid requests

TODO: check why the INVALID1 tests fails with --signing=required.

metze

s4:smb2srv: don't allow the related flag on the first request in a compounded chain

metze

s4: smbcli_transport_send hit the socket right away if possible

[Metze; "make test" on git master outputs exactly the same test summary
with our without this patch (apart from the "using seed" lines)]

If the transport socket is writable, then push the queue along
rather than wait until the caller returns back to the tevent loop.

This strategy keeps the sockets piping hot, and is particularly good
for cases where reading requests from one socket causes lots of
writes on another socket, or where lots of writes are made in a batch.

It doesn't matter if the socket is not writeable yet, packet_queue_run
will return quite cheaply in such a case.

Signed-off-by: Sam Liddicott <sam@liddicott.com>
Signed-off-by: Stefan Metzmacher <metze@samba.org>

net: Only use the in memory ccache when not already using a kerberos ticket in net ads

s4:torture: add SMB2-COMPOUND test

metze

s4:smb2srv: correctly fail remaining compounded requests after a failure

metze

s4:smb2srv: remove old TODO comment, we already check the seqnum

metze

s4:smb2srv: fix handling of multiple compounded requests

metze

s4:smb2srv: remove the chained file handle on close

metze

s4:libcli/smb2: add smb2_transport_credits_ask_num()

metze

s4:libcli/smb2: only add the 1 byte dynamic buffer padding for non compounded requests

metze

s4:libcli/smb2: add support sending compounded requests

metze

s4:libcli/smb2: prepare SMB2 signing code for compounded requests

metze

s3-net: fix "net ads testjoin".

This always needs to use machine account credentials.

Kai, please check.

Guenther

s4-selftest: change rpc.secrets to rpc.lsa.secrets here as well.

Guenther

No need to free twice -- thanks gd :-)

Implement pdb_ads_[add|del]_groupmem

Make "net sam [add|del]mem" work for domain groups

lsa: use the lsa access masks in idl.

Guenther

Implement pdb_ads_delete_dom_group

Implement "net sam deletedomaingroup"

s4-smbtorture: rename SECRETS to LSA-SECRETS.

Guenther

It seems that IRIX doesn't have IOV_MAX

try to get PICFLAG right for HP-UX with gcc

work around conflict in pidfile() prototype for heimdal on NetBSD5

fixed socket wrapper to determine family from the right structure

In convert_in_un_remote() the socket family can be accessed either as
si->family or inaddr->sa_family. We were using the si->family to
determine how to cast the inaddr structure, but if si->family !=
inaddr->sa_family then we will incorrectly be casting a in6 structure
as in4 or vice-versa.

Replace the "ipv4" specific strings in libcli/cldap/cldap.c with "ip". CLDAP can
run over IPv4/IPv6, even though some of the netlogon messages are IPv4 specific.
Fix the new ads_cldap_netlogon() to be IPv6/IPv4 agnostic. This compiles but
I don't have a good test env. for this (although as the previous code was
*completely* broken over IPv6 this will expose previously hidden bugs if it's broken :-).
Jeremy.

Implement pdb_ads_enum_group_members

Make "net ads listmem" also work for domain groups

nsswitch: try to fix segfault in nss_winbind on NetBSD found by torture test.

Guenther

Fix some nonempty blank lines

s3-lsa: remove old code that we cannot even compile anymore.

Guenther

s4-smbtorture: when testing RPC-SAMR-LARGE-DC its fine to just close the objects.

Guenther

nss_wrapper: fix typo in testsuite.

Guenther

s4-smbtorture: add test_QueryDisplayInfo_level to RPC-SAMR-LARGE-DC.

Guenther

libsamba-util: Fix soversion.

python: Fix samba4.dcerpc.rpcecho.RpcEchoTests.test_surrounding test.

Make open_udp_socket() IPv6 clean. Trying to fix bug #6437 - Unable to join IPv6-only ads domain.
Avaiting feedback from submitter before backport to 3.4 and earlier.
Jeremy.

s4:heimdal: fix build on FreeBSD

Patch from Timur I. Bakeyev sent to samba-technical:

Heimdal requires openpty() presence. FreeBSD has in in standard libc, so
autodetection works, but compilation fails, as declaration of this function is
missing.

This patch adds proper header detection and inclusion for openpty().

s3-spoolss: add server-support for queries for the "all" architecture in printdriver enum calls.

Guenther

s4-smbtorture: also test for "all" architecture in enum driver tests in RPC-SPOOLSS.

Guenther

Fix two 64-bit warnings

Implement pdb_[add|del]aliasmem

Add tlda_add_mod_str

Do not use a variable format string

Implement pdb_ads_enum_aliasmem

Pass a talloc_ctx to pdb_enum_aliasmem

Implement pdb_ads_delete_alias

Implement pdb_ads_create_dom_group()

s4-smbtorture: fix test_ReportEventLog in RPC-EVENTLOG.

Guenther

s3-examples: fix usage for eventlog example script.

Guenther

s3-samr: fix enum_acb_mask type (uint32 instead of uint16).

Guenther

s3-rpcclient: allow to set query size for samr enum calls.

Guenther

s3:Makefile.in: use LIBDL as make variable instead of a shell variable

metze

Fix make test_pam_modules

In Samba3 there is a very handy Makefile target 'test_pam_modules'. It let
quickly veryfy, that obtained PAM module actually is loadable and doesn't
miss any dependency libs. the only problem that on FreeBSD it doesn't work
OOTB, as it unconditionally adds -ldl to the list of libraries when FreeBSD
doesn't have it and doesn't need it.

This small patch fixes the problem for FreeBSD and, I hope, still valid
for othe systems, where -ldl is required. Has to be tested there though.

With regards,
Timur Bakeyev.

Signed-off-by: Stefan Metzmacher <metze@samba.org>

Small fix to SMB_LIBRARY macro

It seems, that SMB_LIBRARY macro has small bug in the logic, when showing
if shall the SHARED version of the library be build.

If the default value is given as a parameter, it reports "yes" when library
is going to be build(?).

This small patch makes report consistent.

With regards,
Timur Bakeyev.

Signed-off-by: Stefan Metzmacher <metze@samba.org>

SIGRTMIN additions

In addition to [FreeBSD 14] there is another place, where we (re)define
SIGRTMIN - in SMB_IF_RTSIGNAL_BUG macro in /source3/m4/aclocal.m4.

Here is another small patch.

With regards,
Timur Bakeyev.

Signed-off-by: Stefan Metzmacher <metze@samba.org>

Set SIGRTMIN to NSIG

In the includes we define SIGRTMIN to 32 if it's not defined already. This
value could be fairly low and it's better to use NSIG(number of defined
signals) as the lower mark for the available signals.

We have similar defenition in the source3/smbd/aio.c, which can be safely
removed, as it comes from includes.h then.

With regards,
Timur Bakeyev.

Signed-off-by: Stefan Metzmacher <metze@samba.org>

Check for dmalloc at the end of the configure

Enabling dmalloc in Samba3 build leads to the wrong detection of the
strndup() function - there isn't one in FreeBSD prior to 7.2, but
dmalloc defines it, so, farther tests with -ldmalloc added wrongly
find it. the cheapest fix is to move dmalloc detection and inclusion
to the bottom of configure, so it can't affect detection of the system
capabilities.

Here is the patch.

With regards,
Timur Bakeyev.

Signed-off-by: Stefan Metzmacher <metze@samba.org>

Add langinfo.h to the system/locale.h

This patch for Samba4. It's not really inspired by any real need, just
for the consistency. We define replace/system/locale.h as a generic
header, that includes all locale related headers and we are using
nl_langinfo at least in Samba3. So, it would be consistent to add yet
another locale-related header to the set of locale.h headers.

Here is the patch.

With regards,
Timur Bakeyev.

Signed-off-by: Stefan Metzmacher <metze@samba.org>

Make tunable for modulesdir

This patch is for Samba4. It adds configure tunable for modulesdir -
location, where modules should be installed. In the case, when no
FHS compliance is used and libdir is redefined, modulesdir still
points to $PREFIX/modules. In some installations it may be not desired.
I'd rather set it myself :)

So, here is the patch.

With regards,
Timur Bakeyev.

Signed-off-by: Stefan Metzmacher <metze@samba.org>

rl_event_hook detection

This patch is relevant for Samba4 source mostly. The way, how readline
compiled under FreeBSD makes it require stdio.h to get all the necessary
declarations. Without this addition rl_event_hook is not properly detected.

With regards,
Timur Bakeyev.

Signed-off-by: Stefan Metzmacher <metze@samba.org>

s3-test: finally enable RPC-SAMR against samba3.

We now pass it :-)

Guenther

s3-winbindd: workaround alias enumeration in expand_groups().

alias enumeration in NSS is not done properly done atm and needs to be fixed.

Guenther

s3-selftest: forward NSS_WRAPPER_WINBIND_SO_PATH env to tests.

Guenther

test: add NSS_WRAPPER_WINBIND_SO_PATH to make test.

Guenther

s3-samr: support _samr_SetGroupInfo level 2.

Guenther

s4-smbtorture: exclude oem_information checking for s3 in RPC-SAMR.

Guenther

s3-charcnv: always talloc_free in convert_string_talloc() error path.

Guenther

s3-charcnv: remove remaining malloc references in convert_string_talloc().

Guenther

nss_wrapper: add my copyright.

Guenther

Further fix for 6449

Thanks to TAKAHASHI Motonobu <monyo@samba.gr.jp> for reporting!

Fix a valgrind error in "net rap user add"

Fix bug 6449

Thanks to TAKAHASHI Motonobu <monyo@samba.gr.jp> for reporting!

replace: add fallback define for IOV_MAX

This hopefully fixes the build on IRIX.

metze

talloc: try to fix the source4 build on AIX

metze

async_sock: try fix the source4 build on FreeBSD, Solaris, SLES8

metze

Update WHATSNEW for an alpha8 release this week.

Please update this file with things you have worked on, if you want
them to be mentioned in the release.

Andrew Bartlett

Add "net sam createdomaingroup"

More pdb_ads stuff

Add tldap_pull_uint32

mount.cifs: update the mount.cifs manpage

Add a new section entitled FILE AND DIRECTORY OWNERSHIP AND PERMISSIONS
that attempts to cover information about this topic. Change the uid=
and gid= options to refer to that section. Add new varlistentries for
forceuid, forcegid and dynperm.

Also update the information about how the program behaves when installed
as a setuid binary.

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

Make "net sam list" work for groups, aliases and builtins

Return full info in pdb_ads_search_users()

Fix syntax of sending a delete request

req_del and req_abandon are ASN1_APPLICATION_SIMPLE

Ok, ASN1_APPLICATION everywhere was too easy :-)

Fix after making tldap independent of ldap.h

s3-groupdb: fix enum_aliasmem in ldb branch.

It is totally valid to have an alias with no members.

Tridge, please check.

Found by RPC-SAMR torture test.

Guenther

s3-samr: fix return code of _samr_LookupRids when run with pdb_ldap.

when _samr_LookupRids is called with no rids, it needs to return
NT_STATUS_NONE_MAPPED (not NT_STATUS_NO_MEMORY).

Found by RPC-SAMR torture test.

Guenther

s3-samr: SetGroupInfo level 1 should not return NT_STATUS_INVALID_INFO_CLASS.

Found by RPC-SAMR torture test.

Guenther

mount.cifs: properly check for mount being in fstab when running setuid root (try#3)

This is the third attempt to clean up the checks when a setuid
mount.cifs is run by an unprivileged user. The main difference in this
patch from the last one is that it fixes a bug where the mount might
have failed if unnecessarily if CIFS_LEGACY_SETUID_CHECK was set.

When mount.cifs is installed setuid root and run as an unprivileged
user, it does some checks to limit how the mount is used. It checks that
the mountpoint is owned by the user doing the mount.

These checks however do not match those that /bin/mount does when it is
called by an unprivileged user. When /bin/mount is called by an
unprivileged user to do a mount, it checks that the mount in question is
in /etc/fstab, that it has the "user" option set, etc.

This means that it's currently not possible to set up user mounts the
standard way (by the admin, in /etc/fstab) and simultaneously protect
from an unprivileged user calling mount.cifs directly to mount a share
on any directory that that user owns.

Fix this by making the checks in mount.cifs match those of /bin/mount
itself. This is a necessary step to make mount.cifs safe to be installed
as a setuid binary, but not sufficient. For that, we'd need to give
mount.cifs a proper security audit.

Since some users may be depending on the legacy behavior, this patch
also adds the ability to build mount.cifs with the older behavior.

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