s4-smbtorture: use torture_comment & torture_warning in RPC samr tests.

Guenther

s3:build: check for external liblua, link lua statically if using internal lib

So liblua is treated the same way as libtalloc and libtdb.

Michael

s4-smbtorture: correctly test comment behaviour in RPC-SAMR-USERS against s3.

Guenther

s3-pdb_tdb: give 'unknown_str' the proper name 'comment' and set comment in passdb.

pdb_{get,set}_comment were already existing in the API but were never used.

Guenther

s3:pdb_ads: we need to make the fd for tldap/tsocket non-blocking

metze

s3:fix build of old linux quota system and other unixes' quota implementation

By fixing the use of struct stat_ex.

Michael

s3:lib/sysquotas: fix usage of SMB_STRUCT_STAT (struct stat_ex).

This fixes the build with quotas / configure time detection
of sys_quota interface.

Michael

lib/util: fix building tevent_ntstatus without config.h

(when called from places with "#define NO_CONFIG_H" set, such as configure)

 Michael

tsocket/bsd: also ask for TEVENT_FD_READ when we want to write into a stream socket

Otherwise we would not notice a broken connection.

metze

tsocket/bsd: more correctly check if the cached tevent_fd is still valid

I some cases the pointer value of tevent_context is the same again,
if we do something like:

ev1 = tevent_context_init();
...
fde = tevent_add_fd(ev1, fd, TEVENT_FD_READ...);
...
talloc_free(ev1);
...
ev2 = tevent_context_init();

if (ev1 == ev2) {
/* this can happen! */
}

if (tevent_fd_get_flags(fde) == 0) {
/* this is always true */
}

But the "talloc_free(ev1)" will set fde->event_ctx to NULL
and tevent_fd_get_flags() will always return 0.

metze

s3-selftest: add RPC-SAMR-MACHINE-AUTH to list of test to run against s3.

Guenther

s4-smbtorture: add RPC-SAMR-MACHINE-AUTH test.

This test talks to a DC as a joined workstation member - in the same way
winbindd does, in particular the calls used in this test's query pattern
will all request for SEC_FLAG_MAXIMUM_ALLOWED access_mask
(which pretty much all of samba's client code does as well).

In fact this test verifies that winbind can correctly talk to a samba dc using
samr dcerpc calls.

Guenther

s4-smbtorture: add torture_suite_add_machine_workstation_rpc_iface_tcase.

Unlike torture_suite_add_machine_bdc_rpc_iface_tcase() which joins as a BDC
(ACB_SRVTRUST) this joins as a member workstation (ACB_WSTRUST).

Guenther

mount.cifs: don't pass text ro/rw options to kernel

/bin/mount strips off the ro/rw options after setting the MS_RDONLY
flag appropriately. Make mount.cifs do the same thing.

Signed-off-by: Jeff Layton <jlayton@samba.org>

Fix ndrdump to use a common setup_logging() API

By adding a new common setup_logging_stdout() API, we no longer need to abuse the ABI compatability between the different setup_logging() calls in Samba3 and Samba4's DEBUG() subsystems.

The revert of 49a6d757b4d944cd22c91b2838beb83f04fbe1e9 works with this
to fix bug 6211.

Andrew Bartlett

Revert "s4:debug: make setup_logging() a bit more compatible with samba3"

The problem is that the enum was previously a 'rachet', that is, it
would only reset to a level higher than it was previouly set to.
Changing the order broke file-based logging for our production sites.

This reverts commit 49a6d757b4d944cd22c91b2838beb83f04fbe1e9.

Adds the impersonation level in ntcreatex requests to SAMBA 3 misc torture test

Specifies the impersonation level according to the reporter commit in bug #6283

Fix bug 6514: net gives unhelpful "lp_load failed" when it's missing smb.conf

Correct some typos in the LDB partition module

SAMDB: Don't check for "sAMAccountName" twice

Enhancement of "simple ldap map" with "systemFlags" attribute

Enhance the simple ldap map to support also the "systemFlags" attribute in the
correct way.

ldb: Properly handle NULL when copying attr lists.

When copying an attribute list, ensure the list itself is not NULL before
attempting to access elements of the list.

Correct the headers of some SAMBA 4 setup python scripts

Two patches which fix issues on cross compiling/building

Fixes for SAMBA3RPC torture test

On calls where both NT_STATUS and WERROR results are returned and consulted
we have to make sure to form function results considering both.
This errors have been found through a run against SAMBA 4.

Small patch for SPOOLSS pipe

Prevents a crash of the SAMBA 4 daemon on the torture SPOOLSS test due to not
initialised structures.

_lsa_QueryInfoPolicy: Use symbolic info level names

Turn the pdb_rid_algorithm into a capabilities call that returns flags

Make pdb_ads survive a restart of Samba4

The search function retries once, the modifying call that hits a dead smbd
returns an error. The next try will reconnect. This was simple to implement and
provides a good compromise against Samba4 idling our connection. Most of the
modifying calls are quickly after a search (like OpenUser) anyway.

If the connection is down, don't try another write.

Add tldap_search_va

tldap_msg_received: Properly free the asn1_struct in case of an error

Move read_ldap_done after read_ldap_send

Convert tldap to tstream

Add tstream_read_packet

Properly free a downlevel readv request. Metze, please check

tldap: Don't fire off more than one read_ldap request during searches

Stop listening for inotify data when there's none

There have been some reports that we're flooding syslog with "No data on
inotify fd?!". I haven't been able to reproduce this yet. Until then, it is
better to just list this message once.

Fix some nonempty blank lines

s3/lanman: Workaround for KB932762.

This addresses bug #6498.

The solaris linker seems to require an extra mention of dependent libs

Don't use ads realm name for non-ads case.  #6481

Also check that the connection to ads worked.

Fix some dead code warnings from SUN Studio

Fix a typo in a comment

Upgrade ntvfs_map_*info to ntvfs_map_async_setup/ntvfs_map_async_finish

ntvfs_map_fsinfo, ntvfs_map_qpathinfo, ntvfs_map_qfileinfo used an
old synchronous mapping technique, acceptable on the grounds that
they were only used by the simple vfs which was synchronous.

Other vfs may/do use these functions, and by upgrading them to use the
ntvfs_map_async_setup/ntvfs_map_async_finish framework, they can now be
used asynchronously.

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

s3:passdb: fix bug #6509: use gid (not uid) cache in fetch_gid_from_cache().

With the previous code, the cache can never have been hit at all.

Michael

s3-vlp: drastically shrink size and dependencies of the virtual line printer.

This gets vlp from 13M down to 214K on my box.

Guenther

s3: pass a valid stat into file_ntimes()

file_ntimes() calls can_write_to_file() which expects a valid stat struct

s3 file_access: Convert some more functions over to use smb_filneame

s3: simplify some redundant logic in is_ntfs_default_stream_smb_fname()

s3: Remove get_full_smb_filename() from open_directory()

This is possible because open_directory() returns an error if
the fname is a stream, so the base_name can be used.

s3 onefs: Plumb smb_filename through onefs createfile path

s3: Change set_ea() and its callers to use smb_filename

Fix bug #6506 - SMBD server doesn't set EAs when a file is overwritten in NT_TRANSACT_CREATE.
Reported and verified by Long Li <longli@microsoft.com>
Jeremy.

s4 auth_winbind: Internally, info3 has utf8 buffers, not utf16 buffers.

Thanks to gd for the catch.

s4 auth_winbind: Don't allocate the rids for the info3 structure within the loop

s4: Add libwbclient backend to auth_winbind

s3-netlogon: remove remaining netlogon init functions.

Guenther

Add a \n to a debug message in smbacl4_nfs42win

mount.cifs: add support for sending IPv6 scope ID to kernel

When getaddrinfo returns an IPv6 address with a non-zero scope_id, send
that to the kernel appended to the address with a '%' delimiter. This
allows people to mount servers via their link-local IPv6 addresses
(given a kernel that understands this address format, of course).

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

s3-netlogon: remove init_netr_SamInfo functions.

Guenther

s3-netlogon: fix validation level 2 support in netr_SamLogon and friends.

Guenther

s3-examples: make get_next_oid exectuable.

Guenther

s3-samr: refactor _samr_SetDomainInfo().

Guenther

s3-samr: refactor _samr_QueryDomainInfo().

Guenther

s3: Plumb smb_filename through SMB_VFS_STAT and SMB_VFS_LSTAT

This patch introduces two new temporary helper functions
vfs_stat_smb_fname and vfs_lstat_smb_fname.  They basically allowed me
to call the new smb_filename version of stat, while avoiding plumbing
it through callers that are still too inconvenient.  As the conversion
moves along, I will be able to remove callers of this, with the goal
being to remove all callers.

There was also a bug in create_synthetic_smb_fname_split (also a
temporary utility function) that caused it to incorrectly handle
filenames with ':'s in them when in posix mode.  This is now fixed.

s3: Convert is_visible_file to use talloc_asprintf instead of malloc

s3:smbd: send SMB2 interim responses for async calls

metze

s3:smbd: add support for async interim SMB2 responses and prepare SMB2 cancel

metze

s3:smbd: keep a list of outstanding SMB2 requests

metze

s3:smbd: add smbd_smb2_send_oplock_break()

metze

s3:smbd: the SMB2-COMPOUND test shows that the related vs. unrelated flags isn't checked first

metze

Reenable the LDAPI socket for the merged build

It seems that the samba4 part of the merged build does not pick up the
DEVELOPER flag from the s3 configure.

Jelmer, can you fix that properly?

Thanks,

Volker

Only set the password if there is one

s4-smbtorture: more paranoid checks while testing group membership in RPC-SAMR.

Guenther

s3-printing: eliminate another non sec_initial_uid using security check.

Guenther

s3-spoolss: restructure _spoolss_EndDocPrinter().

Guenther

s3-lsa: Fix error path in _lsa_EnumAccountRights.

This needs to return NT_STATUS_OBJECT_NAME_NOT_FOUND
again as described in MS-LSAD 3.1.4.5.10 and tested with the
RPC-SAMR-USER-PRIVILEGES test.

Guenther

s3-eventlog: implement _eventlog_ReportEventW().

Guenther

s3: re-run make idl.

Guenther

samr: add samr_UserInfoLevel and samr_DomainInfoClass enums based on MS-SAMR.

Guenther

s3: forward MSG_DEBUG from smbd parent to all children

Before 3.3, an smbcontrol debug message sent to the target "smbd" would
actually be sent to all running processes including nmbd and winbindd.
This behavior was changed in 3.3 so that the "smbd" target would only
send a message to the process found in smbd.pid, while the "all" target
would send a message to all processes.

The ability to set the debug level of all processes within a single
daemon, without specifying each pid is quite useful.  This was implemented
in winbindd in 065760ed.  This patch does the same thing for smbd.

Upon receiving a MSG_DEBUG the parent smbd will rebroadcast it to all of
its children.

The printing process has been added to the list of smbd child processes,
and we now always track the number of smbd children regardless of the
"max smbd processes" setting.

s3-libnet: fix libnet_unjoin_remove_machine_acct() when called without ads struct.

Guenther

Add tldap paged searches, together with two helper routines

Reorganize retrieving errors and server-sent controls

This attaches the data to the tldap_message instead of the tevent_req.

It adds tldap_ctx_lastmsg() to retrieve the last message for the users of
the sync wrappers.

Move asn1_load_nocopy() to lib/util/asn1.c

Move asn1_blob() to lib/util/asn1.c

Add tldap_supports_control

Add tldap_entry_has_attrvalue

tldap control support

Prepare control support

We will have arrays of controls passed to tldap.c. Follow a mantra from the
classic book "Thinking Forth" by Leo Brodie: Favor counts over terminators :-)

This makes the parameter lists to tldap pretty long, but everyone will have
wrapper routines anyway, see for example tldap_search_fmt. And the OpenLDAP
manpages call the non-_ext routines deprecated, probably for a reason.

Fix setting passwords in pdb_ads

The samba4 password_hash module does not allow changing the password fields via
the "delete oldval" -> "add newval" set of modify operations, it requires a
single "replace with newval" operation.

Andrew, Samba3 by default uses that delete/add pair to detect if between
fetching the old value and storing the new one the old value has changed. This
is lost by using the "replace" operation.

Would it make sense to add this to the password_hash module?

Volker

Fix empty lines

talloc_tos() aborts if it can not get a stackframe

Fix coverity #729. Resource leak in error path.
Jeremy.

Fix coverity #740. Resource leak in error paths. We should
always return queue here as the caller will free.
Jeremy.

Fix coverity #900. Resource leak.
Jeremy.

Fix coverity #920. Possible NULL deref.
Jeremy.

Fix coverity #676. Forward NULL.
Jeremy.

s3: fix make test with external libtalloc or libtdb.

This skips the talloctort and tdbtorture tests when the
corresponding binaries are not present.
There might be more clever ways of detecting wether samba
has been linked with internal or external libraries, but
as a first approximation, this seems valid.

Michael