This commit was manufactured by cvs2svn to create tag
'release-3-0-3pre1'.

syncing more changes from 3.0

This commit was manufactured by cvs2svn to create branch
'SAMBA_3_0_RELEASE'.

Apply some more of Derrell Lipman's changes.

preparing for release of 3.0.3pre1

sync configure.in with 3.0

add --with-libdir and --with-mandir (patch from Guenther Deschner)

tightening down the release branch now -- last planned merge from 3.0

cleaning up a few compiler warngings

wrap cli_tdis() in check to make sure we have a valid cnum

syncing with 3.0

add privileges support to ldapsam too

the buffer must be greater then src string len otherwise bad things happen!

Fix gcc warnings. Fix mkproto with new type.
Jeremy.

Apply Craig Barratt's fixes to allow multiple exlusion files and patterns.

Much of this was applied by hand because of the changes in the code. However,
it builds and smbclient seems to work still. The Amanda folks are testing
and it seems to work there as well.

This commit was manufactured by cvs2svn to create branch
'SAMBA_3_0_RELEASE'.

merge from 3.0...LDAP password lockout support

BUG 932: ulrich@holeschak.de (Ulrich Holeschak); fix local password change using pam_smbpass

Correctness patch from fumiya@miraclelinux.com to count characters correctly,
doesn't affect what got put on the wire.
Jeremy.

Commit Derrell Lipman's changes and fixes to libsmbclient. The build but
I have not tested them beyond building. I did fix the two instances of
safe_strxxx that were introduced.

removing files not present in the 3.0 tree

starting to sync the cvs tree

Fix sambaUserWorkstations for network samlogons against us as DC. Strip
the \\ off the workstation.

Volker

Two little annoyances:

net rpc did not inform you if no smbd is running.

I never liked the error message (!) Success! when we established a trust.

Volker

Added a varient of Volker's patch to fix the "self-referrals as anonymous user"
problem.
Jeremy.

Remove excess logging when probing for the length of the next mb char.
Jeremy.

asu/syntax/pc_netlink doesn't fill in the pipe name in the rpc_bind response so dont check for it

BUG 758 -- patch from jpeach@sgi.com (James Peach); fix pdma build

Change check_path_syntax() to use the new next_mb_char_size() function
to make it generic. Remove the mb-codepage "blacklist". Alexander, please
check this fix as it reverts your blacklist changes, but I'm hoping it
fixes the problem in a more generic way for all charsets. I'm not trying
to trample on your (excellent!) work here, just make things more generic
without special cases.
Jeremy.

Add function next_mb_char_size() that returns a size_t of the number of
bytes in the mb character at a pointer. Will be useful in fixing check_path_syntax()
to not use a "blacklist". Also re-added my (C) to reply.c. I mean, really - I've
been adding code to the file for over 10 years and I recognise many of the
fuctions as mine ! :-).
Jeremy.

merges from 3.0

BUG 979 -- fix quota display on AIX; patch from Heinrich.Mislik@univie.ac.at (Heinrich Mislik)

Fix check_path_syntax() for multibyte encodings which have no '\' as second byte.
This is intermediate fix as discussed with Jeremy until we move check_path_syntax() to UCS2 internally where all
ambiguity is resolved. Please add other encodings into charcnv.c with such property.

merging print change notify fix from HP appliance.  Also might address some one the issues in BUG 1007

BUG 1165, 1126: Fix bug with secondary groups (security = ads) and winbind use default domain = yes

Make sure we don't clobber the stack when response consists of the empty
string.

Avoid mb conversion overflow when sending an announcement.
Jeremy.

Use "unix netbios name" type unstring - 64 bytes long to manipulate netbios
names in nmbd. Allows conversion from dos codepage mb strings (ie. SJIS) to
expand to utf8 size on read.
Jeremy.

removing extra comma in --with-fhs

more fixes from 3.0

sync small fixes from 3.0 and fix compiler warning in priv code

fix byte ordering problem when storing the updating pid for the lpq cache; was causing an abort in process_exists() on solaris

Ok, one of the latest cleanups did too much... :-)

Re-add adding the local aliases to winbindd_getgroups.

Volker

Clean up a bit :-)

Volker

Fix 'getent group aliasname', this did not look in pdb.

Add and delete aliases via srv_samr_nt. For that I added a RID allocation call
to winbindd. idmap_allocate_rid wants information about whether this will be a
user or a group, I did not export this to the winbind interface.

The reason for idmap to get that info is to keep consistent with the
algorithmic convention to alloc only even rids for users and odd rids for
groups. I'm not fully convinced that this really gains us anything. Any real
good arguments?

Volker

When trying to remove a user from a group, winbind should be informed
about the user and group...

Volker

Merge from 3.0

Make alias membership maintainable via net rpc group add/delmem. Alias
creation and deletion still has to be done as well as a 'net alias' command.

Volker

Rest of my winbindd_passdb alias support

Add alias support to winbindd_passdb. Sorry, Simo, this does not leave too
much of your winbindd_passdb, users are currently not provided by that, only
aliases. Currently the code to maintain that stuff is not yet in, this will be
next, see my next posting to samba-technical.

Volker

Remove the prototype for alias support in winbind again. This will be
replaced by a winbindd_passdb.c checkin soon.

Volker

Add find_alias, create_alias, delete_alias, get_aliasinfo, set_aliasinfo and
enum_aliases to passdb. create and delete are not yet filled, you have to do
this with net groupmap and net idmap restore.

Volker

Ensure we don't truncate strcmps to nstring anymore...
Jeremy.

Modified fix for bugid #784. Based on a patch from moriyama@miraclelinux.com (MORIYAMA Masayuki).
Don't use nstrings to hold workgroup and netbios names. The problem with them is that MB netbios
and workgroup names in unix charset (particularly utf8) may be up to 3x bigger than the name
when represented in dos charset (ie. cp932). So go back to using fstrings for these but
translate into nstrings (ie. 16 byte length values) for transport on the wire.
Jeremy.

First part of patch from moriyama@miraclelinux.com (MORIYAMA Masayuki) to
fix up netbios names with mb strings. Includes reformat of libsmb/nmblib.c
so it's readable.
Jeremy.

Bug report 1079 and fix both from Bert Driehuis.

This is an odd corner case having to do with <1C> group names (DC names).
<1C> group names are called (by MS) "Internet Special Groups", which means
that Microsoft's WINS servers will attempt to handle these names in
something approximating an RFC1001/1002 compliant manner.

The problem being reported here is this:  If an initial registration sent
by one of the group members is lost (network error, whatever) then
subsequent refreshes from that particular machine will also fail.  This
only happens if the name is already active (because of other group
members).

In most cases, we (and MS) handle refreshes as registrations if the name
is not in the database.  In this situation, we missed the corner case.

Bert's fix adds an 'else if' that catches the situation in which a refresh
for an unlisted IP of an active <1C> group name is received.  The refresh
is simply handled as a registration when this happens.

[Note: Committing in HEAD.  I am writing some tools to do registrations
       and refreshes so that I can test this.  I don't have an NT Domain
       for testing so I'm going to have to fudge.  This fix is small (I
       cleaned up some Debug messages and comments in addition to the fix)
       and if you want to test it with 3.0.x you can just copy the HEAD
       version into your 3.0.x tree and recompile.]

Fix from Robert.Dahlem@gmx.net (Robert Dahlem) for bugid #1048
when mangled names are turned off.
Jeremy.

Fix problems with very long filenames in both smbd and smbclient.
It appears that we pass filename through resolve_wildcards() as pstring and use fstring temporary buffer there.
As result, a long filename in unix charset (UTF-8 by default) can easily expand over 255 bytes while
Windows is able to send to us such names (e.g. Japanese name of ~190 mb chars) which we unable to process through
this small fstring buffer. Tested with W2K and smbclient, Japanese and Cyrillics.

Fix build after Jeremy -- yet another place where convert_string() wasn't updated

Fix string overflow due to wrong size calculation

Allow msdfs symlink syntax to be more forgiving (it took me ages to remember
what it was :-). Allow msdfs links to now look like UNC paths : eg.
sym_link -> msdfs://server/share/path/in/share
or :
sym_link -> msdfs:\\server\share\path\in\share
Jeremy.

Fix bug introduced by Jim (touche :-) :-) - use new smbldap_get_single_pstring
call :-).
Jeremy.

Restore the contract on all convert_stringXX() interfaces. Add a "allow_bad_conv"
boolean parameter that allows broken iconv conversions to work. Gets rid of the
nasty errno checks in mangle_hash2 and check_path_syntax and allows correct
return code checking.
Jeremy.

Merge from 3.0.  Fix munged dial in pdb_ldap

add missing #ifdef HAVE_BICONV stuff

BUG 1088: patch from SATOH Fumiyasu <fumiya@miraclinux.com> -- use strchr_m() for query_host (smbclient -L)

Ensure we use check_path_syntax() to pull out DFS paths.
Jeremy

Fix processing of pathnames with embedded '\\' characters (0x5c) in CP932
character set. Because of the allowing of "broken conversions" for people
who have broken iconv libraries we can't rely on the return from convert_string()
to be valid - we must check errno instead. This is ripe for revisiting at
some stage. I prefer adding a bool parameter to all convert_string_XX varients
to specify if we will allow broken conversions or not. With "allow_broken_conversions"
set to false we could then rely on the return from convert_string rather than
checking errno.
Jeremy.

lsa_enumaccounts lists all SIDs that have been granted privileges. Simo,
you should look at this.

Change behaviour from listing all groups to listing only BUILTIN\Administrator

Volker

Remove expanding sids from smbd, this will be done in winbindd.

Volker

Apply some const

Don't pass DOM_SIDs around when pointers are enough.

Volker

another type

Moving towards getting msdfs working again (this looks like it was broken
in 3.0.2a actually). We now send a correct referral back, but the client
refuses to follow it... Not sure why. Will do more tests.. Maybe unicode
character count is wrong (it looks it).
Jeremy.

Formatting reformat before I start to change anything here.
Jeremy.

fix type for tag in proto file

Fix dfs referrals - don't use check_path_syntax. Also use mb char calls in
dfs code.
Jeremy.

Yet another '\n' termination of debug message.

rafal

Forgot to put some more comments.

rafal

1) Two new functions to trust passwords interface in passdb:
   settrustpwent, gettrustpwnam

2) Implementation of another couple of these functions in tdbsam:
   settrustpwent, gettrustpwnam, gettrustpwsid

3) Testing (mostly for now) usage of the interface in pdbedit
   which is soon to be offline tool back again.

This is quite a new code, so many changes will be put in soon.

rafal

Fix to debug message lacking termination with '\n'.

rafal

fix build with gcc 2.96 and --with-developer

Commit to HEAD the updates to smb signing code that I was propsing for 3.0.

This code implements 'opportunistic signing' in our client (when the
server supports it, we will use it), and correct downgrading on both
the client and server for the 'enabled' (rather than required) signing
level.

This means that we can actually set 'server signing = yes' and not
have the world fall apart.  We had a number of bugs in that code, and
it certainly looks like most of the testing was with the 'requried'
setting.

While the changes are reasonable, I'm putting this into HEAD rather
than 3.0 for the timebeing.  SMB signing, like NTLMSSP, tends to have
gotchas in it :-)

(I also need to give it a workout with more than smbclient before I
move it across).

Andrew Bartlett

This commit was manufactured by cvs2svn to create branch
'SAMBA_3_0_RELEASE'.

Merge fixes and tests for jra's broken strstr_m() function from 3.0

STF rules!  (Pity we don't use it for more stuff)

Andrew Bartlett

Added strstr_m() function. Use in all places where we might run into mb
(should fix the mb service name problem, can't remember the bugid).
Jeremy.

Fix assumption about following directory sep in check_path_syntax(). We
need to try and convert 1 byte, then 2 bytes if that fails. Fixes bug
reported by Simo.
Jeremy.

Do some checks about data passed to this function

2 fixes to enhance readability of source code and debug messages
1 fix for a memleak

Get us a little closer to Windows LSA semantics.

A windows DC does not reply to DCNAME\\Administrator, only to
DOMAIN\\Administrator. Fix that.

Without winbind we are wrong as domain members, we should forward the request
DOMAIN\\Username to the DC on behalf of the asking client. Winbind fixes that
nicely.

Volker

This might not be used a lot and might not survive for long, but at least it
should work as expected :-)

Fix wb_delgrpmember.

Volker

Fix typo.

Enclose use of fchown() with guards.  Bugzilla #1155.

tdbdump doesn't need to #include <sys/mman.h>

Bugzilla #1154.

Several mb tidyups - getting ready to address the XXX_sub function.
Jeremy.

BUG 488: fix the 'show client in col 1' button and corrctely enumerate active connections

fix compiler warning

Add -O (write downloaded files to stdout), based on patch from
Bas van Sisseren <bas@dnd.utwente.nl>

all_string_sub() is broken as it doesn't handle mb chars correctly (and
so breaks when substitution '/' and '\'). It's used by unix_clean_name(),
which is used by reduce_name, which is used by check_name() (phew!).
Now that we know all filenames passed to check_name() are in a "good"
format (no double slashes, all '\\' chars translated to '/' etc.) due
to the new check_path_syntax() we can avoid calling reduce_name unless
widelinks are denied. After this check-in I can fix all_string_sub() to
handle mb chars correctly as it won't be in the direct path in the
main path handling code.
Jeremy.