s3-winbind: BUG 8166 - Don't lockout users when offline.

Windows does not track bad password attempts when offline.  We were locking users out but not honoring the lockout duration.

Autobuild-User: Jim McDonough <jmcd@samba.org>
Autobuild-Date: Wed May 25 18:11:10 CEST 2011 on sn-devel-104
(cherry picked from commit b58534f1fca27e3e72f4f4107538ec05734bd42a)

s3: Safely mark our sconn as smb2 if we have that protocol

Otherwise smbd will crash at an unclean exit. Without this conn_close_all will
do a close_cnum() on all connection_struct's. In smb2, those are talloc
children of the smbd_smb2_tcon's. sconn is talloc_free'ed after the
conn_close_all, but the smbd_smb2_tcon destructor will still reference
tcon->compat_conn, referencing then free'ed (and null'ed out) memory.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Mon May 30 22:49:53 CEST 2011 on sn-devel-104
(cherry picked from commit c981d4fa1269569a1c2db4bf72a67a357aacd69f)

Fix bug #8199 (potential crash in smbd handling smb2).

s3:smbcacls: fix parsing of multiple flags

Signed-off-by: Michael Adam <obnox@samba.org>
(cherry picked from commit c967e8fae5babf6e337c0e2b85fc42eccfe9e4c4)

Fix bug #8192

Improve documentation for net rpc trust

Add man pages entries and fix usage output.

Signed-off-by: Günther Deschner <gd@samba.org>
Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Mon May 30 20:05:34 CEST 2011 on sn-devel-104
(cherry picked from commit e10f27d5759b2d21c82fdb20f5641e1f6feab158)

s3: Use the correct guest_login field in auth_server

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun May 29 13:57:21 CEST 2011 on sn-devel-104
(cherry picked from commit 0969c3398e73e66c9e004740127da7c29e951050)

The last 3 patches address bug #8185 (security=server does not obey guest login
field).

s3: Extract the guest_login field in sesssetup
(cherry picked from commit 01386ff3132ff5c786e83fc24328a80661de6bb7)

s3: Fix wct check in cli_sesssetup_blob_done
(cherry picked from commit 4ec00fd621e944ff979e9f0a20773202d8c66472)

WHATSNEW: Update changes since rc1.

Karolin

Fix bug #6911 - Kerberos authentication from vista to samba fails when security blob size is greater than 16 kB

We were not correctly checking the output of asn1_start_tag().
asn1_start_tag() returns -1 and sets data->has_error if the
remaining blob size is too short to contain the tag length.
We were checking data->has_error and returning NT_STATUS_OK
(to allow the second asn.1 parse to fail in that case). We
should not be checking data->has_error in this case, but
falling through to the code that already checks the length.

Thanks to Jim for reproducing this for me. We don't get bitten
by this as we announce a max buffer size of 16k, greater than
Windows's 4k, which means that most krb5 spnego packets already
fit.

Jeremy.

s3-docs: Fix some typos.

This patch is a manpage update addressing the typos and mistakes in the vfs_smb_traffic_analyzer manpage and the smbta-util manpage, which have been brought up in these threads on samba-technical: http://lists.samba.org/archive/samba-technical/2011-May/077801.html http://lists.samba.org/archive/samba-technical/2011-May/077800.html http://lists.samba.org/archive/samba-technical/2011-May/077753.html

I have seen Volker already fixed the typos in
127d417e8ebc967572df7a75b342897a6a8fb71e , but I haven't found
"Ultimatively" in any dictionary, so this patch just removes the
word completely.

Thanks to Samba-JP oota <ribbon@samba.gr.jp> for reporting.

The last 2 patches address bug #8178 (Collection of manpage updates for SMB
Traffic Analyzer).

s3: Fix a typo

Thanks to Samba-JP oota <ribbon@samba.gr.jp>
(cherry picked from commit 127d417e8ebc967572df7a75b342897a6a8fb71e)

Fix our asn.1 parser to handle negative numbers.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue May 24 22:57:16 CEST 2011 on sn-devel-104
(cherry picked from commit e719dfd4dc178f001a5f804fb1ac4e587574415f)

Fix bug #8163 (asn.1 library does not correctly read negative integers).

Actually make use of the SMBTA_SUBRELEASE define in smb_traffic_analyzer.h. This will allow to introduce new features or fixes into the protocol after the 3.6.0 release. The client software is designed to take care for the subrelease number.

Fix bug #8154 (Actually make use of SMBTA subversion numbers).

Fix bug #7054 - X account flag does not work when pwdlastset is 0.

Don't allow pass_last_set_time to be set to zero (which means
"user must change password on next logon") if user object doesn't
allow password change.

Don't automatically allow user object password change if
"user must change password on next logon" is set.

Jim please check.

Jeremy.

s3-testparm Warn about incorrect use of 'password server'

The last 5 patches address bug #8151 (deprecate security parameters for 3.6).

s3-param Depricate 'password server = foo:12389' syntax

This was originally intended to allow the LDAP port on a DC to be
varied, but makes little sense to change one port when in an
environment where krb5, ldap, smb and potentially DCE/RPC over TCP are
involved.

Andrew Bartlett

s3-param Deprecate a number of security parameters for 3.6

This follows up on the agreement on the samba-technical list in Jan
2011 to deprecate these options, and to possibly remove these in the
4.0 release after user feedback.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Fri May 13 19:51:41 CEST 2011 on sn-devel-104

docs: Clarify the 'security=server' fails for NTLMv2

docs: Rewrite 'password server' documentation

I think this new version is more clear.

Andrew Bartlett

Fix bug #8150 - Ban 'dos charset = utf8'

s3: Document "async smb echo handler"

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Thu May 26 12:50:55 CEST 2011 on sn-devel-104
(cherry picked from commit 875e29ba830b269faf8ca7ff7cd7fc95c0c18f28)

Fix bug #7571 (Documentation on new "async smb echo handler" parameter is
missing).

WHATSNEW: Start adding changes since rc1.

Karolin

Fix bug #8157 - std_pcap_cache_reload() fails to parse a cups printcap file correctly.

The parsing code made some strange assumptions about what is a printer
name, and what is a comment.

Patch for bug #8156 - net ads join fails to use the user's kerberos ticket.

If kerberos_get_realm_from_hostname() or kerberos_get_default_realm_from_ccache() fails due to
 a misconfigured krb5.conf, try the "realm =" from smb.conf as a fallcback before going back to
 NTLMSSP (which we'll do anyway).

s3: Remove two false references to cli->inbuf (cherry picked from commit b1a7bdb93c7fda54a29284f1691de1dc4f3bbf6b)

Fix bug #8159 (Memory corruption in fetching cli->server_domain from the
server.).

s3: Fix a leftover from fstring removal in cli_state

Jeremy, please check!
(cherry picked from commit 9514f96856ccf822b683b5362fd2eb4a4e9e418a)

Fix bug 8133 - strange behavior for the file (whose filename first character is period ) in SMB2 case.

When doing SMB2 renames, we need to match all filetypes (no attributes field in the SMB2 call).
By default a file starting with a period is returned as FILE_ATTRIBUTE_HIDDEN in Samba.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Fri May 20 19:26:04 CEST 2011 on sn-devel-104

Optimization. If the attributes passed to can_rename() include both FILE_ATTRIBUTE_HIDDEN and FILE_ATTRIBUTE_SYSTEM then there's no point in reading the source DOS attribute, as we're not going to deny the rename on attribute match.

Make protocol version 2 the default protocol, and only run on version 1 if V1 is explcitly given as a module option.

I haven't received a single line of feedback on protocol v1
for at least 1 1/2 years, whereas protocol v2 has an active
userbase and more people developing around it.

This patch includes a manpage update, describing the new
version handling, as well as documenting the recent changes
making the module transfer the IP address of the client machine
as submitted with
464c69609aa7e582f484c1d357b7c6d3eb2bcbe3.

Bug #8148 (Default to protocol version 2 for SMB Traffic Analyzer in Samba
3.6.0).

s3-epmapper: Fixed endpoint registration.

Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Fri May 20 12:03:18 CEST 2011 on sn-devel-104
(cherry picked from commit 7ab9e26b601e4e51736ce6eace46e6588fa1148f)

Fix bug #8155 (Registering only named pipes on EPM for a service doesn't work).

Fix bug 8153 found when building on an IPv6-only system by Kai Blin.

When building on IPv6-only, doing:

hints.ai_family = AF_INET;
getaddrinfo("0.0.0.0", NULL, &hints, &ppres)

fails as AF_INET is unavailable on an IPv6-only system. This
causes us to fallback to our replacement getaddrinfo code
which is IPv4-only.

As we're only trying to detect a specific AIX bug here,
broaden the tests to find that bug, and also test for
working getaddrinfo in an IPv6-only safe way.

Fix Bug 8152 - smbd crash in release_ip()

release_ip() needs the private_data, but it was never saved away
to feed it into release_ip() later

Autobuild-User: Christian Ambach <ambi@samba.org>
Autobuild-Date: Thu May 19 21:21:14 CEST 2011 on sn-devel-104
(cherry picked from commit 642c6ba2b9c581bacfcb9a6cb4c5c95d446263ce)

s3:rpc_server: create lp_ncalrpc_dir() with 0755 before lp_ncalrpc_dir()/np with 0700

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue May 17 13:01:14 CEST 2011 on sn-devel-104
(cherry picked from commit cb227d6d1492247d8aff03807cac0b7266202a38)

The last 2 patches address bug #8141 (wrong permissions on lp_ncalrpc_dir()).

ncalrpc: Force ncalrpc dir to be mode 755 in all users

This allows this directory to be shared between Samba3 and Samba4 in a
Franky-style setup easily.

Andrew Bartlett
(cherry picked from commit aae9353ecf56323b63da66aa84d8a0a4f219d94d)

talloc: splitout _talloc_free_children_internal()

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue May 17 10:49:13 CEST 2011 on sn-devel-104
(cherry picked from commit df2cb2f672569e5d113fe2e77fdc1ee16c8b646d)

The last 9 patches address bug #8140 (talloc: valgrind false positives and other
backports).

talloc: fixed a use after free error in talloc_free_children()

This is similar to commit 6f51a1f45bf4de062cce7a562477e8140630a53d.

metze
(cherry picked from commit 38633c9f0b7f86673f08903999583ad5b62c3548)

talloc: use _talloc_free_internal() in talloc_free_children()

metze
(cherry picked from commit f3b855d2ff9576715afe50d75678829c6bc0842d)

talloc: test talloc_steal out of a talloc_pool

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue May 17 09:43:01 CEST 2011 on sn-devel-104
(cherry picked from commit 37b2130ed9612a7334888ecd2fee26b0b45ac271)

talloc: add memset() calls to test_pool()

This way we the pool based valgrind code.

metze
(cherry picked from commit 16cc52cf70a9918843f9761baf483338c80bf1d0)

talloc: setup the new 'tc' before TC_UNDEFINE_GROW_CHUNK() _talloc_realloc()

metze
(cherry picked from commit c281f2fc1a359d0d3b91b94438f11bb7c88170b5)

talloc: make really sure only optimize realloc if there's only one pool chunk

*talloc_pool_objectcount(pool_tc) == 2 doesn't mean the one of the objects
is the pool itself! So we better check for == 1 and calculate the chunk count.

metze
(cherry picked from commit 7102105c8954627dc30a851327cf2642ac0783d5)

talloc: make use of _talloc_free_poolmem() in _talloc_realloc()

This should follow the same logic...

metze
(cherry picked from commit 14b662ee4f278764b9dfd620851e908d29f29fc4)

talloc: split the handling of FLAG_POOL/FLAG_POOLMEM in _talloc_free_internal

The optimization of the object_count == 1 case should only happen
for when we're not destroying the pool itself. And it should only
happen if the pool itself is still valid.

If the pool isn't valid (it has TALLOC_FLAG_FREE),
object_count == 1 does not mean that the pool is the last object,
which can happen if you use talloc_steal/move() on memory
from the pool and then free the pool itself.

Thanks to Volker for noticing this!

metze
(cherry picked from commit 2d514be1ed3b8245157a0a51186ec7f9db828202)

Fix typos in LDAP schema files

Reported by: John Danks <john.danks@gmail.com>

Signed-off-by: Günther Deschner <gd@samba.org>
Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Tue May 17 11:56:08 CEST 2011 on sn-devel-104

Fix bug #8142 (typo in several LDAP schema files).

Fix bug #8144 - touch /mnt/newfile fails to set timestamp with CIFS client.

The extra checks added for Windows correctness in our metadata changing paths
to ensure the file handle has been opened with the correct access mask to
allow FILE_WRITE_ATTRIBUTES etc. caused problems with the POSIX open code.

The old POSIX open code maped O_RDONLY into FILE_READ, O_WRONLY into FILE_WRITE,
and O_RDWR into FILE_READ|FILE_WRITE. This patch extends the mapping to add
FILE_WRITE_ATTRIBUTES, FILE_READ_ATTRIBUTES and FILE_WRITE_EA, FILE_READ_EA to
allow POSIX opens to set these values.

WHATSNEW: Start release notes.

Karolin

VERSION: Bump version up to 3.6.0rc2.

Karolin

WHATSNEW: Add further changes since pre3.

And remove double entry.

Karolin

Fix the SMB2 showstopper, found by an extended torture test from Volker.

In the oplock refactoring, the algorithm underwent an unnoticed change.
In 3.5.x stat_opens were silently (i.e. no explicit code had comments
explaining this) ignored when looking for oplock breaks and share mode
violations. After the refactoring, the function find_oplock_types()
no longer ignored stat_open entries in the share mode table when looking
for batch and exclusive oplocks. This patch adds two changes to find_oplock_types()
to ignore the case where the incoming open request is a stat open being
tested against existing opens, and also when the incoming open request
is a non-stat open being tested against existing stat opens. Neither
of these cause an oplock break or share mode violation. Thanks a *lot*
to Volker, who persevered in reproducing this problem.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Mon May 16 22:38:20 CEST 2011 on sn-devel-104
(cherry picked from commit 8cf14c21b3bc55454728bf48b23f696e15c92aea)

Ensure we always write the correct incoming mid into the share mode
table entries.
(cherry picked from commit fe21bdc43c0aeacfc8592998e6a90f6f83c939e0)

s3-printing: remove cups_pull_comment_location from header file

Signed-off-by: Günther Deschner <gd@samba.org>
Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Mon May 16 16:03:57 CEST 2011 on sn-devel-104
(cherry picked from commit 6bfe488f29a9df26f3d6dbc4166200ea286536c1)

s3-printing: Remove obsolete and unused cups_pull_comment_location().

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

s3-spoolss: Get the printer location from cups.

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

s3-printing: Get the location info from cups.

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

s3-includes: no need to globally include libads/ads_status.h.

Guenther
(cherry picked from commit 1e208a705718adb105d89093041c0ba5d51f2429)

s3-includes: finally only include client.h when libsmb is used.

Guenther
(cherry picked from commit 675573d54b717ffc24cf3b49301ff649aecc259a)

s3-build Remove distinct LOCALEDIR subsystem

This is no different to the rest of dynconfig, and so should be dealt
with there.

Andrew Bartlett
(cherry picked from commit d309499b9e268bff5e366a9986528f0accbc02b3)

s3-libsmb: move protos to libsmb/proto.h

Guenther

s3: only include tdb headers where needed.

Guenther
(cherry picked from commit d8cfca3a9bd2b6b6c562fd202377d95a98eb5472)

lib/util Move source3 tdb_wrap_open() into the common code.

This #if _SAMBA_BUILD == 3 is very unfortunate, as it means that in
the top level build, these options are not available for these
databases.  However, having two different tdb_wrap lists is a worse
fate, so this will do for now.

Andrew Bartlett
(cherry picked from commit 14750139639b3531e57a3ca3f9e481d6e458dc06)

s3-includes: remove ipv6 related prototype which is already defined in ../lib/util/util_net.h.

Guenther
(cherry picked from commit a3609457912123037ca9fc51349d89d7921bc4e7)

lib/util Move more network utility functions from source3 into lib/util

This will help with the merge of the interfaces layer.

Andrew Bartlett
(cherry picked from commit fbea52f74a80268446f45936a0cf40400aba8565)

s3-rpc_server: run minimal_includes.pl.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Thu May  5 03:04:38 CEST 2011 on sn-devel-104
(cherry picked from commit 8563d5c384f672276d8ea2ab20a1810ff485e37d)

s3-printing: run minimal_includes.pl.

Guenther
(cherry picked from commit c9f3a2c83e208fcf05576eb2137c893fe4f3fe6d)

s3-rpc_server: (re)move last globally included rpc_server prototypes.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Mon May  2 16:05:31 CEST 2011 on sn-devel-104
(cherry picked from commit 56cd3b3bbbb595cb0c98fa2dfa397c915fbf37d2)

s3: include ntdomain.h before including generated srv_ headers.

Guenther
(cherry picked from commit 0e76eddcc8a4e7e98167b8f92387fae015fae095)

s3-rpc_server: add pipes_struct forward declaration.

Guenther
(cherry picked from commit 911cdc56c146014058063b872372ff8096c06065)

s3-printing: avoid using pipes_struct when only session_info is needed.

Guenther
(cherry picked from commit cd5b2b242b39437081592d41d297bce0668b40ab)

s3-proto: move remaining srvsvc protos to own header file.

Guenther
(cherry picked from commit daa02f34b181a3ec7224e920f787cc677bc3cdf2)

s3-proto: move remaining spoolss protos to own header file.

Guenther
(cherry picked from commit c233c21425a183dd1124329fdbca13ab92cc6d6a)

s3-proto: move more rpc_server prototypes out of proto.h

Guenther
(cherry picked from commit 047d8c073b57bc12648a251deaceedb65f4f59cf)

s3-proto: move rpc_srv_register protos and structs to town headerfile.

Guenther
(cherry picked from commit 49d5f62b42d821bb8a11b595efb53eb88ec3c570)

s3-proto: move more headers to rpc_server/rpc_ncacn_np.h where they origin from.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Fri Apr 29 22:05:07 CEST 2011 on sn-devel-104
(cherry picked from commit bbacaec4e9365b128eb11467c95412de7473c1ec)

s3-proto: remove duplicate prototypes.

Guenther
(cherry picked from commit bc781bf7d98baca57c8043bf7dc0a95f8ffd1345)

s3-rpc_server: remove some unneeded headers.

Guenther
(cherry picked from commit 487cb0c96ef8ca05fcef35d0d7c49bc067f6cff7)

s3-proto: move rpc_server/rpc_handles.c protos to ntdomain.h

Guenther

s3-proto: move more librpc prototypes to librpc/rpc/dcerpc.h

Guenther
(cherry picked from commit faf11751255202274505d72848c54ee6e5dce7b6)

s3-proto: move more rpc client prototypes to cli_pipe.h.

Guenther

s4-smbtorture: add test_netremotetod.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Mon May 16 12:45:52 CEST 2011 on sn-devel-104
(cherry picked from commit 74d82faa78e64bdaed68efb9331a7af1c3016c34)

s4-libcli/rap: add smbcli_rap_netremotetod().

Guenther
(cherry picked from commit b1948659ad987ce50d048dabc34f03031a243ee5)

s3-rap: add rap_NetRemoteTOD to IDL.

Guenther
(cherry picked from commit b82caab5c9d8821cd125654e3233da40eb4c61d1)

s3-rpc_client: run minimal_includes.pl.

Guenther
(cherry picked from commit bd92826aa837f56119153b93cfce7f843e567896)

s3-rpcclient: run minimal_includes.pl.

Guenther
(cherry picked from commit c11d52c7f6b560fe66a2f197e30602d57370fd5c)

s3-param: run minimal_includes.pl.

Guenther
(cherry picked from commit 62314da8059ae888c0d1b299c486156664611e06)

s3-libsmb: run minimal_includes.pl.

Guenther
(cherry picked from commit ea8960448d0a40067382a8d14ae19351d20acbdd)

s3-libnet: run minimal_includes.pl.

Guenther
(cherry picked from commit a05a59e8e2a421f22978e6eedbb1a7501a603aff)

s3-libads: run minimal_includes.pl.

Guenther
(cherry picked from commit 653b84d2d6ed7387f93e5a126b5f2114b831044e)

s3-lib: run minimal_includes.pl.

Guenther
(cherry picked from commit 63ebc918e4514f0006b94856d9b089255fca9718)

s3-auth: run minimal_includes.pl

Guenther
(cherry picked from commit 58db720e27d4b0c580fb4d3e036d823bc5d3a141)

libcli/named_pipe_auth: add forward declaration in npa_tstream.h.

Guenther
(cherry picked from commit 89ceb489784358f9476bb6be0ad3ab9ea4d1b9c2)

s3-waf: check for pthread support.

Guenther
(cherry picked from commit cad8fafa0391e9f13c2958c45ca5ec57f93f1e7b)

s3-proto: move remaining krb5 protos to krb5_protos.h

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Tue May  3 15:38:40 CEST 2011 on sn-devel-104
(cherry picked from commit 9c48345e32c9aebd93c67103ea6984d74f2061b4)

s3: remove various references to server side dcerpc structs (which are not needed).

Guenther
(cherry picked from commit 0bb4701a747599042242b0612bc392a6e6d777af)

s3: move pipe_auth_data to dcerpc.h

Guenther
(cherry picked from commit 80fa624861e6f45c4b6837969253d5fbac2f3b1e)

s3-smbd: avoid using pipes_struct when only session_info is needed.

Guenther
(cherry picked from commit 61cd1067efacd91e22b4acbdf0d1d92dc2efc162)

s3-printing: include ../librpc/ndr/libndr.h  in ads printing.

Guenther
(cherry picked from commit f7d6a0a1121c355a87dd113d2cb2a4985cdc06ce)

s3-winbindd: remove unused headers.

Guenther
(cherry picked from commit 233779cce4f47799d5912f362faadcd564bbf096)

s3-system: move LOCK_ defines to lib/system.c

Guenther
(cherry picked from commit 4fa2bac6e7ebe11447b535d9c8ec4348a2bbd39e)

s3-spoolss: remove another unused header.

Guenther
(cherry picked from commit 1a561dedb9995f52411d2fed2c6e0cc1e37a85d1)

s3-util: move valid_share_pathname() to lib/util.c

Guenther
(cherry picked from commit c1f3ff734043082a9488c787324e76a37702f94d)