Mark version as 3.0.28a

Updating release notes for 3.0.28a

Work around a handle leak in XP 64 bit

We have to tell XP64 that we store unicode, otherwise some library will to a
findfirst before each createfile, leaking a directory handle each time.

Emit debug message at 0 (already fixed for 3.2).
Jeremy.

Fix bug #5267 - nmbd shuts down when network interfaces go down.
Cause nmbd to wait for an interface, in a mode where SIGTERM
will kills us (same way we wait on startup for an interface).
Jeremy.

Respect FAMChanged -- attempt to fix bug 5307

Thanks to Ricardo Santos

Patch to fix the "Invalid read of size 4" errors. Bug #3617.
Jeremy.

 Fix part of bug #3617 from valgrind trace.
"Invalid read of size 1" errors.
Jeremy.

Rename the 'hidden' variable to 'administrative share'.

Fix possible uninitialized variable use.
Jeremy.

Fix from Guenter Kukkukk <linux@kukkukk.com> to fix listing against
OS/2 servers. OS/2 returns eclass == ERRDOS && ecode == ERRnofiles
for a zero entry directory listing.
Jeremy.

Closed memory leak on error path.

Add variable to define if a share should be hidden.

If you create a share on a Windows machine called foo$ then this share is
of the type STYPE_DISKTREE. So it is possible to administrate this kind of
share. Tested on Windows NT and 2003.

In samba we assume that if a share with a $ at the end must be of the type
STYPE_DISKTREE_HIDDEN. This is wrong, so we need a variable in the config
to define if the share should be hidden or not.

Correctly check return of rename(2)

Bug 5279 -- Thanks to Max Matveev

Fix inotify detection

Bug 5271 -- thanks to Tiziano Müller

Fix confusing error message -- bug 5252

Thanks to Walter Franzini

Fix typo.

Karolin

Change ldap search filter. This function is also used to search machine accounts which may be located in a different ou.

Karolin

Ensure we call nt_status_squash() on returns. Smnall whitespace cleanup.
Jeremy.

Ensure we call auth_ntlmssp_end in invalidate_vuid and invalidate_partical_vuid.
Jeremy.

Allow a *NULL to be passed to auth_ntlmssp_end().
Jeremy.

Revert "mount.cifs: fix several problems when mounting subdirectories of shares"

This reverts commit e8f569735e2c0523efa175ca44dd919f838ae49e.

We found that this patch does not play well with currently released cifs.ko
code, so after discussions with Jeff Layton and Steve french we decided it
is best to back it off and re-think a better approach. Jeff will send new
patches later, but for now it is better to just revert to the previous code

Fix obscure bug where if client sends us the krb5 part
of a SPNEGO packet we could drop into the NTLMSSP
part of the processing. This fix only for 3.0.28a,
I have a proper SPNEGO negotiate fix for 3.2.
Jeremy

Back-port from vl.

Author: Volker Lendecke <vl@samba.org>
Date:   Mon Feb 11 18:36:06 2008 +0100

    Add a missing return

    If I'm not completely blind, we should return here. Not doing it here seems not
    to be a major flaw, as far as I can see we're only missing the error code. This
    might account for some of the very unhelpful NT_STATUS_UNSUCCESSFUL error
    messages people see during joins.

    All with stake in Samba client, please check!

Fix bug #5247 "Wildcard expansion in mget is broken"
by making cur_dir an invarient ending in '\\' or '/'.
Will forward-port to 3.2 as the code is different here.
Jeremy.

Add configure check for LBER_LOG_PRINT_FN - to intercept ldap debug.

Use the resulting HAVE_LBER_LOG_PRINT_FN to determine whether we can
use it in init_ldap_debugging to intercept LDAP debug output and print
it out in the samba logs (controlled with "ldap debug level").

Michael

Add support for LDAP debug output in Samba log file.

Logging of the ldap libraries appears with a [LDAP] prefix
inside the samba logs. This is controlled by two new parameters:

* "ldap debug level" sets the debug level of the ldap libraries.
  It is the bit-field as understood by the openldap server.

* "ldap debug threshold" is the samba debug level at which ldap
  logging appears inside the samba logs.

This probably needs some configure tests since it makes
use of the LBER_OPT_LOG_PRINT_FN option to redirect the
debug output of the ldap libraries.

Michael

Move libads/util.o into the standard ADS list
from the server list.
Jeremy

Back-ported fix from 3.2 (mainly from Bo Yang <boyang@novell.com>)
to fix bad private_data pointer in winbindd_lookupname_async().
Jeremy.

Fix two memleaks

Thanks to Andreas Schneider <anschneider@suse.de> for nagging :-)

Ensure that convert_string_allocate() allocates 2 extra
bytes and null terminates them to ensure NDR wire-reads
of string types are always null terminated. Bug found by
Volker after great pain :-).
Jeremy.

use correct path for ignored file

Fix winbindd running on a Samba DC,
This patch make sure we do not try to contact smbd in the main dameon
to avoid deadlocks.
All the operations that require connecting to smbd are performed in
the domain child anyway.

Enable v3-0-test to successfully join a windows 2008 domain controller.

This is hand-merged from a couple of commits from 3-2-test, cherry-picking was
hardly possible without importing all the ldap sign/seal work from metze.

Guenther

Restrict the enctypes in the generated krb5.conf files to Win2003 types.

This fixes the failure observed on FC8 when joining a Windows 2008 RC1
domain.  We currently do not handle user session keys correctly
when the KDC uses AES in the ticket replies.

r24771: Use infolevel 25 to set the machine account's password (just like winxp).
This correctly updates pwdLastSet field on win2k3 server.

rafal

Fix build warning.

Guenther

idmap: Fix an incompatible pointer type warning.

Thanks to Simo for pointing me at the easier solution

Port SMB_FS_OBJECTID_INFORMATION from 3.2

Patch by Corinna Vinschen -- Thanks

Fix the same bug with user -> user_obj.
Jeremy.

Back port : Correctly set flags in ACE's inherited from parent.
Jeremy.

Fix a really subtle old, old bug :-). When canonicalizing the
NT ACL into a POSIX one, if the group being set is the primary group
of the file, map it into a SMB_ACL_GROUP_OBJ, not a SMB_ACL_GROUP.
Otherwise we get an extra bogus group entry in the POSIX ACL.
Jeremy.

Make explicit in debug we're ignoring flags from the parent SD.
Jeremy

Add debug messages to trace this if needed.
Jeremy.

The checks for OI and CI were just wrong.... Fix them. Thanks to
Jim for testing this.
Jeremy.

First part of fix for bug #4929 - worked out by jmcd.
Cope with protected ACL set correctly.
Jeremy.

Missed one flags change.
Jeremy.

Added :

Author: Jeremy Allison <jra@samba.org>
Date:   Wed Jan 23 15:23:16 2008 -0800

    Don't leak memory in error path.
    Jeremy.

Author: Jeremy Allison <jra@samba.org>
Date:   Wed Jan 23 15:00:40 2008 -0800

    Use strchr_m in seaching for '.' in the hostname to make sure we're mb safe.
    Jeremy.

Author: Andreas Schneider <anschneider@suse.de>
Date:   Thu Jan 17 11:35:40 2008 +0100

    Fix Windows 2008 (Longhorn) join.

    During 'net ads join' the cli->desthost is a hostname (e.g.
    rupert.galaxy.site). Check if we have a hostname and use only the
    first part, the machine name, of the string.

Author: Andreas Schneider <anschneider@suse.de>
Date:   Thu Jan 17 10:11:11 2008 +0100

    Windows 2008 (Longhorn) auth2 flag fixes.

    Interop fixes for AD specific flags. Original patch from Todd Stetcher.

Version of Matt Geddes <musicalcarrion@gmail.com>
patch for adding acct_flags to rpccli_samr_create_dom_user().
Jerry please test.
Jeremy.

Fix get_trust_creds() to return always an upper-cased krb5 principal (this
fixes winbind krb5 session at least with heimdal).

Guenther

Copy the 3.2 version of string_replace to 3.0

There are several callers in 3.0 that don't give a pstring to string_replace,
thus it will end up in segfaults like the one reported by Sergio Pires
<suporte@grupovdl.com.br> on samba@samba.org. The 3.2 version of string_replace
does not have the pstring assumption anymore.

Jeremy, Jerry, please check!

Thanks,

Volker

Fix build warning.

Guenther

Actually test vl's new code and make it work to fix the build farm :-).
Jeremy.

Back port vl's fix for nlink counts.
Jeremy.

Fix two uninitialized variables in vfs_hpuxacl.c

Thanks to David Leonard <David.Leonard@quest.com>, this fixes bug 5208.

Volker

libsmb: Do not upper-case target name on NTLMv2 hash generation

This makes our NTLMv2 hash generation compatible to the Davenport example
and fixes a bug when ntlm_auth is called with a non-upper-case --domain
parameter and client ntlmv2 auth = yes

Jerry, please consider for 3.0.28a

Fix bug #5171 (perl syntax error) found by Jason Filley <jason@snakelegs.org>.

Michael

Enable building the notify_fam module.

Found by Timur I. Bakeyev <timur@com.bat.ru>.

Michael

Correct comment. Default debug level of smbclient is 1, not 0.

ensure uni_name.buffer is initialised

merge from http://samba.org/~tridge/3_0-ctdb

Fix coverity resource leak on error.
Jeremy.

Fix resource leak on error found by coverity.
Jeremy.

Merge branch 'v3-0-test' of ssh://jra@git.samba.org/data/git/samba into v3-0-test

Add missing quote. Fix bug 5172. Thanks to Jason Filley <jason@snakelegs.org> for reporting!

Karolin

Using a bool with a logical operation. IBM checker caught.
Jeremy.

Merge branch 'v3-0-test' of ssh://jra@git.samba.org/data/git/samba into v3-0-test

Add missing patch to allow smbcacls to do krb5 auth, bug #5175
from Tom Maher <tmaher@watson.org>.
Jeremy.

Revert "no-atime-on-mtime"

This reverts commit 54adb86890eb22b6bcf0bc1163662c3f9c075777.

Sorry I didn't want to push that...

metze

Happy new year !

Guenther
(cherry picked from commit 9a58cd57953d6aead14789daa47a3badef19496d)

Merge remote branch 'origin/v3-0-test' of /home/People/metze/devel/samba/samba-bare into v3-0-test

Fix for bug #5163 from Laurent Pinchart <pinchart@skynet.be>
Failure to change password in ldap is mapped to NT_STATUS_UNSUCCESSFUL unconditionally.
Jeremy.

Attempt to fix bug #3617. Mix of patches from Volker and
myself. Use standard dlinklist macros.
Jeremy.

Make send_getdc_request match 3.2.x codebase.
Jeremy.

Port 2135dfe91bf1ae114a18c15286b535662200677d from 3.2.
From Volker :

    Fix setting the initial permission bits

    This fixes a make test failure on Solaris. When creating a new file,
    file_set_dosmode() called from open_file_ntcreate calculates a new permission
    mask, very likely different from what had been calculated in
    open_file_ntcreate. Further down we overwrote the newly calculated value with
    SMB_FCHMOD_ACL, ignoring what file_set_dosmode had calculated.

    Why did Linux not see this? fchmod_acl on a newly created file without acls
    would not retrieve an acl at all, whereas under Solaris acl(2) returns
    something even for files with just posix permissions returns something.

    Jeremy, given that we have very similar code in 3.0.28 this might also explain
    some of the bug reports that people have concerning ACLs on new files.

    Volker

    P.S: This one took a while to find...

fix dump printout when byte >= 0x80

Only retrieve password policies in pam_auth when WBFLAG_PAM_GET_PWD is set.

This essentially re-establishes r14496 (2155bb0535656f294bd054d6a0a7d16a9a71c31b)
which was undone in r17723 (43bd8c00abb38eb23a1497a255d194fb1bbffffb) for
reasons that are unclear to me. Maybe I am being too naive.

Now we do again only retrieve the password policy when called from
the pam_winbind module. This fixes logons delegated to AD trusted
domain controllers: We need to connect to the sam to retrieve the
password policy. But auhtenticated session setup is not possible
when contacting the trusted domain dc and afterwards, SamrConnect
also fails with whatever credentials and method used.

Michael

Fix a debug message: add missing space.

Michael

Fix logic error in cm_connect_sam().

Don't fall back to schannel when trust creds could be obtained.
This is still not complete, but I am getting closer.

Michael

Fix another segfault.

Michael

Fix a segv in winbindd caused by trying to free an fstring. Make a copy of the machine_password and machine_account strings in all conditional paths so that SAFE_FREE() will always be valid.

Make cm_connect_sam() try harder to connect autheticated.

Even if the session setup was anonymous, try and collect
trust creds with get_trust_creds() and use these before
falling back to schannel.

This is the first attempt to fix interdomain trusts.
(get password policy and stuff)

Michael

Refactor out assembling of trust creds (pw, account name, principal).

Michael

Streamline and fix logic of cm_prepare_connection().

Do not attempt to do a session setup when in a trusted domain
situation (this gives STATUS_NOLOGON_TRUSTED_DOMAIN_ACCOUNT).

Use get_trust_pw_clear to get machine trust account.
Only call this when the results is really used.
Use the proper domain and account name for session setup.

Michael

Refactor out get_schannel_session_key logic.

Refactor the actual retrieval of the session key through the
established netlogon pipe out of get_schannel_session_key()
and get_schannel_session_key_auth_ntlmssp() into a new
function get_schannel_session_key_common().
(To avoid code duplication.)

Michael

Pass NULL instead of unneeded &sid: secrets_fetch_trusted_domain_password() checks.

Michael

Rename get_trust_pw() to get_trust_pw_hash().

Michael

Export logic of get_trust_pw() to new function get_trust_pw_clear().

get_trust_pw() just now computes the md4 hash of the result of
get_trust_pw_clear() if that was successful. As a last resort,
in the non-trusted-domain-situation, get_trust_pw() now tries to
directly obtain the hashed version of the password out of secrets.tdb.

Michael

Refactor the lagacy part of secrets_fetch_trust_account_password() out

into a new function secrets_fetch_trust_account_password_legacy() that
does only try to obtain the hashed version of the machine password directly
from secrets.tdb.

Michael

Let get_trust_pw() determine the machine_account_name to use.

Up to now each caller used its own logic.

This eliminates code paths where there was a special treatment
of the following situation: the domain given is not our workgroup
(i.e. our own domain) and we are not a DC (i.e. it is not a typical
trusted domain situation). In situation the given domain name was
previously used as the machine account name, resulting in an account
name of DOMAIN\\DOMAIN$, which does not seem very reasonable to me.
get_trust_pw would not have obtained a password in this situation
anyways.

I hope I have not missed an important point here!

Michael

Remove silly amounts of trailing white spaces.

Michael

Streamline logic in cm_connect_netlogon()

by retrieving trust password only, when it will be used.

Michael

r25086: Fix interdomain trusts (this povides the fix expected in r22709):

Fix winbindd on a Samba DC talking to a trusted domain DC by
making it use the trusted domain password...

Michael

I hope this does not brake any other setup.

r22709: we can only use tschannel when commectcing to our primary (might need some fixing here for a Samba DC)

In cm_prepare_connection(), only get auth user creds if we need to.

Michael

Remove two unneeded functions.

secrets_store_trust_account_password() and trust_password_delete()
are the write access functions to the SECRETS/$MACHINE.ACC/domain keys
in secrets.tdb, the md4 hashed machine passwords. These are not used
any more: Current code always writes the clear text password.

Michael

packet_struct is used in several places as raw memory

-> Fix more uninitialized variable warnings

Some paranoia checks

Zero the tdb key, there might be padding

This leads to uninitialized variable warnings if nmbd is run under valgrind.

Ensure we can't pass -1 to smb_fn_name(). Fixes bug #4612.
This is not used in 3.2 code.
Jeremy.

Fix socket_wrapper build for 3.0.x.
Jeremy.