Fix bug disclosed by lock8 torture test

We have to drop the gpfs level share modes, regardless of whether we put
the file into the pending close queue.

Demonstrate a bug we have when dealing with real os-level share modes

Another one of those where you stare at logfiles for hours, and when you found
it, it's absolutely obvious what is happening...

s4: try to fix privileges implementation in order to pass the RPC-SAMR-USERS-PRIVILEGES test.

Guenther

Have ntvfs_connect() accept union smb_tcon *tcon instead of char* sharename

This change brings ntvfs_connect into compliance with other ntvfs functions
which take an ntvfs module, an ntvfs request and an smb io union.

It now becomes the responsibility of ntvfs modules to examine
tcon->generic.level themselves and derive the share name and any other
options
directly; e.g.

const char *sharename;

switch (tcon->generic.level) {
case RAW_TCON_TCON:
sharename = tcon->tcon.in.service;
break;
case RAW_TCON_TCONX:
sharename = tcon->tconx.in.path;
break;
case RAW_TCON_SMB2:
default:
return NT_STATUS_INVALID_LEVEL;
}

if (strncmp(sharename, "\\\\", 2) == 0) {
char *p = strchr(sharename+2, '\\');
if (p) {
sharename = p + 1;
}
}

service.c smbsrv_tcon_backend() is called before ntvfs_connect and fills in
some of the tcon->..out values.
For the case of RAW_TCON_TCONX, it filles out tcon->tconx.out.tid and
tcon->tconx.out.options

For the case of RAW_TCON_TCON it fills out tcon->tcon.out.tid and
tcon->tcon.out.max_xmit

Thus the ntvfs_connect function for vfs modules may override these values
if desired, but are not required to.

ntvfs_connect functions are required to fill in the tcon->tconx.out.*_type
fields, for RAW_TCON_TCONX, perhaps something like:

if (tcon->generic.level == RAW_TCON_TCONX) {
tcon->tconx.out.fs_type = ntvfs->ctx->fs_type;
tcon->tconx.out.dev_type = ntvfs->ctx->dev_type;
}

Signed-off-by: Sam Liddicott <sam@liddicott.com>
(I fixed the ntvfs_connect() in the smb_server/smb2/
 and the RAW_TCON_SMB2 switch case in the modules)

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

s3-selftest: add add and delete group scripts using nss_wrapper.

Guenther

nsswrapper: implement group_del() in nss_wrapper.pl.

Guenther

nsswrapper: implement group_add() in nss_wrapper.pl.

Guenther

s4:smb_server: initialy read the first 4 bytes only

Stop packet_recv getting greedy and reading the whole socket
and then dispatching te extra packets in a timer loop

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

s4:libcli/raw: initialy read the first 4 bytes only

metze

Do not do a merged build without shared libs

Jelmer, you might want to take a look at Andrew B's problem with

--enable-developer --disable-shared --disable-shared-libs

Only build the krb5 locator when building shared libs

Set errno=ENOMEM if tevent_add_fd fails

tevent_add_fd does not properly set that. At least in epoll and select this is
the only error condition.

Metze, please check!

Added mapping table for account object in lsa.
Jeremy.

Simplify the logic of tsocket_bsd_pending

Remove two indentation levels by returning early on error.

Metze, please check!

tevent/python: Makefile was still trying to build some non AC_SUBST python targets

Signed-Off-By: Jelmer Vernooij <jelmer@samba.org>

Make local-wbclient flexible in # of connections and ops

Fix wb_trans_done after conversion to unix calling conventions

Fix an uninitialized variable in wb_context_init

Another attempt at fixing the static build of vfs_notify_fam

Fix detection of "enum FAMCodes"

s3 onefs: Removing an incorrect TALLOC_FREE

Signed-off-by: Tim Prouty <tprouty@samba.org>

Use TDB_VOLATILE instead of tdb_set_max_dead()

s3/docs: Fix shutdown script example.

This fixes bug #5897. Thanks to TAKAHASHI Motonobu
<monyo [at] samba.gr.jp> for reporting and providing the example!

Karolin

s3: Always allocate memory in dptr_ReadDirName

This is a follow up to 69d61453df6019caef4e7960fa78c6a3c51f3d2a to
adjust the API to allow the lower layers allocate memory.  Now the
memory can explicitly be freed rather than relying on talloc_tos().

Signed-off-by: Tim Prouty <tprouty@samba.org>

s4 torture: Fix typo

s3 sendfile: Fix two bugs in sendfile

These were found interally via code inspection.

1) fake_sendfile was incorrectly writing zeros over real data on a
   short read.

2) sendfile_short_send was doing 4 byte writes instead of 1024 byte
   writes due to an incorrect sizeof usage.

Jermey, Vl please check

s4-smbtorture: Fix build warning in RPC-SAMR tests.

Guenther

s3: re-run make samba3-idl.

Guenther

lsa: add access_masks for accounts, secrets and trusted domains to IDL.

Guenther

Change access_check_samr_object -> access_check_object.
Make map_max_allowed_access global. Change lsa_get_generic_sd
to add Everyone:LSA_POLICY_READ|LSA_POLICY_EXECUTE, not just
LSA_POLICY_EXECUTE.
Jeremy.

s3-lsa: let _lsa_OpenPolicy() just call _lsa_OpenPolicy2().

Guenther

s3-selftest: enable RPC-SAMR-USERS-PRIVILEGES.

Guenther

Fix SAMR server for winbindd access. Ensure we allow
MAX_ACCESS to be mapped to what we're giving Everyone.
Jeremy.

s3-lsa: let _lsa_GetSystemAccessAccount() call into _lsa_EnumPrivsAccount().

Inspired by lsa server from Samba 4.

Just removing a user in SAMR does not remove a user in LSA. If you use
usermanager from windows, the "User Rights" management gui gets unaccessable as
soon as you delete a user that had privileges granted. With this fix, that
no longer existing user would properly appear as an unknown account in the GUI
(as it does while using usermanager with windows domains).

This almost makes Samba3 pass the RPC-SAMR-USERS-PRIVILEGES test.

Guenther

s3-lsa: start a very basic implementation of _lsa_DeleteObject().

Certainly not the full story but this gets us closer to pass the
RPC-SAMR-USERS-PRIVILEGES test.

Guenther

s4-smbtorture: add RPC-SAMR-USERS-PRIVILEGES test.

This test demonstrates the independence of the lsa and samr accounts while
remove a samr users that still has privileges granted.

Guenther

Move down the become_root()/unbecome_root() calls into the VFS modules

The aio_fork module does not need this, as it does not communicate via signals
but with pipes. Watching a strace log with those become_root() calls in aio.c
is absolutely awful, and it does affect performance.

Fix bug 5681: Do not limit the number of network interfaces

Jeremy as far as I can see there is no real technical reason to limit the
number of interfaces. If you like this patch, can you please merge it to 3.4?
If you don't please tell me :-)

Thanks,

Volker

In aio_fork, we have to close all fd's, we might hold a gpfs share mode

Keeping such an fd open prohibits another open of that same file.

Add "file_walk_table" to do stuff with all open files

Fix a race condition in vfs_aio_fork with gpfs share modes

Fix broken pipe handling

Metze is right: If we have *any* error at the socket level, we just can
not continue.

Also, apply some defensive programming: With this async stuff someone else
might already have closed the socket.

Fix a type-punned warning

s3: tevent_req_poll() loops forever when pipe is broken

Signed-off-by: Bo Yang <boyang@samba.org>

s3:fix bug #6371, unsuccessful net conf setparm leaves empty share

Wrap creation of share and setting of parameter into a transaction.

Michael

s3-events: Not only timed, also immediate events generate a timeout return

Move smb_create_user() in samsync

It is not used anywhere else, so make it also static and remove
it from proto.h

tevent: Define TALLOC_FREE() if it's not defined yet, to allow building
with released versions of talloc.

Fix static linking of vfs_notify_fam

Fix a typo

s3-events: If immediate requests are pending, the timeout is 0

Fix the merged build -- thanks jelmer for the hint!

tevent: Remove python module.

This module didn't have any functionality that we actually used yet, and
it was quite small.

Tevent is quite low level and perhaps doesn't make much sense to expose
directly as a Python module. It was also causing build problems when used with a
system-tevent. We can always back later if necessary.

tevent: Install tevent_internal.h in the standalone build.

This is not ideal, but at least it fixes the build of samba-gtk for now.
I've also added a warning about API guarantees at the top of the header.

Fix linker flags on non-Linux GNU platforms.

(e.g. Debian GNU/kFreeBSD)

Signed-off-by: Jelmer Vernooij <jelmer@samba.org>

Update copies of config.guess and config.sub.

tevent: Don't install headers, since we don't install a shared lib
either (from Samba).

s3-privileges: add privilege_delete_account().

Guenther

s3-privileges: remove trailing whitespace from privileges codes.

Guenther

Oops. Forgot to delete now unused local from the previous patch.
Jeremy.

Fix bug #6359 - smbclient -L does not list workgroup for hosts with both IPv4 and IPv6 addresses

s3:build: fix linking with --disable-shared-libs

Michael

Ensure users with SeAddUser privs get full access to
groups/aliases when opening.
Jeremy.

Add extra abilities for a user with SeAddUsers, so they
can manipulate groups and aliases.
Jeremy.

DeleteUser doesn't need the priv checks, this is done at OpenUser time.
Jeremy.

s3-samr: Fix samr access checks in _samr_RemoveMemberFromForeignDomain().

Guenther

Fix bug 6361: Make --rcfile work in smbget

Thanks to j scott <gl@arlut.utexas.edu> for reporting!

mount.cifs: directly include sys/stat.h in mtab.c

This file is mysteriously getting included when built via the makefile,
but when you try to build mtab.o by hand it fails to build. Directly
include it to remove any ambiguity.

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

s3-netdomjoin-gui: allow to switch between workgroups/domains with the same name.

Guenther

s3-netdomjoin-gui: cosmetic fix for empty hostnames.

Guenther

s3-netdomjoin-gui: only gray out labels when not root and not connecting to
remote machines.

Guenther

s3:swat: hide "config backend" from swat

Michael

s3:param: prevent includes from being dumped in dump_*() functions.

This fixes bug #4271: testparm should not print includes.

Michael

s3:build: add install-dbwrap_tool target

Michael

s3:build dbwrap_tool with make everything

Michael

errormap: Remove wbc -> NTSTATUS error map for now. It is not used.

This was breaking the Samba 4 installation since it added dependencies
on header files we weren't installing.

Confirmed with Kai.

s3-samr: Fix samr access checks in _samr_SetUserInfo().

Guenther

s3-samr: Fix samr access checks in _samr_QueryUserInfo().

Guenther

s3-samr: in _samr_QueryUserInfo() make sure to not return any info in error case.

Guenther

s3-samr: Fix samr access checks in _samr_SetDomainInfo().

Guenther

s3-samr: Fix samr access checks in _samr_QueryDomainInfo().

Guenther

s3-samr: use normal integer in r->in.level switch statements.

Guenther

s3:build: rename OBJ collection for dbwrap_tool consistently

Michael

s3:first cut at dbwrap_tool - a tdb tool that is CTDB-aware.

This tool (in contrast to tdbtool) reads Samba's configuration
and if clustering = yes, it talks to CTDB instead of accessing
the TDB data bases directly. This is done by simply using
the dbwrap mechanim, just like the Samba daemons.

This first version can read and write int32 and uint32 values
and delete records from a (c)tdb database.
More operations will follow.

This tool can already be useful in CTDB environments, e.g. when
"net idmap restore" fails to set the USER and GROUP HWM keys,
because the methods are deliberately not implemented in
idmap_tdb2.c. You can manually set the high water marks
with
"dbwrap_tool store idmap_tdb2.tdb 'USER HWM' int32 12345"
and
"dbwrap_tool store idmap_tdb2.tdb 'GROUP HWM' int32 67890"

Michael

Turn on SHORTNAME-TEST in torture.
Jeremy.

Make us pass SHORTNAME-TEST.
Jeremy

Make error message clearer on fail.
Jeremy.

Add a test showing what ascii values cause an NTFS volume to
create a mangled name. We don't pass this yet, but it's not
run by default.
Jeremy.

Remove one use of mangle_is_8_3(), not needed.
Jeremy.

Fix the core of the SAMR access functions. This passes make test, but
usrmgr fails against it. The core of this patch is to move all the
access mask setup into the _samr_OpenXXX functions, and then have
each specific function check the attached access_mask against the
required bits. We can then go through the MS-SAMR doc and match
things up. Signed off by Guenther, and writespace cleanup removal
by Volker.
Jeremy.

Maximum password length check too short in mount.cifs

    Windows allows up to 127 byte passwords, and we using a 64 byte limit
    in most places and a 16 byte limit when using the "pass=" mount option

Acked-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>

Add extensive tests for str_list function behaviour

Remove two unused variables

s3-netapi: implement NetShutdownInit_r and NetShutdownAbort_r.

Guenther

s3-netapi: add example code for NetShutdownInit and NetShutdownAbort.

Guenther

s3-netapi: add NetShutdownInit and NetShutdownAbort to public API.

Guenther

s3-netapi: add NetShutdownInit and NetShutdownAbort skeletons.

Guenther

s3-build: re-run make samba3-idl.

Guenther

s3-netapi: add NetShutdownInit and NetShutdownAbort.

I know, they don't exist as such calls on windows but having them in
libnetapi.so would be very handy.

Guenther

Fix a type-punned warning