The big character set handling changeover!

This commit gets rid of all our old codepage handling and replaces it with
iconv. All internal strings in Samba are now in "unix" charset, which may
be multi-byte. See internals.doc and my posting to samba-technical for
a more complete explanation.

This brings HEAD into line with SAMBA_2_2, they now both use bindir.

Andrew Bartlett

removed some debug code

check for bad usernames early in session setup

Removed bogus server_n local variable in connect_one() function.

Ensure we correctly round down to 2 second boundary if dos filetime
set for a share.
Jeremy.

Small warning removal tidyup.
Jeremy.

Remove the exit_server() calls that stop nmbd linking.
Jeremy.

Fixed warnings and made const-safe so it has the correct prototype.
Jeremy.

Removed the default addition of /include to the directory given in
--with-sslinc. Should allow building on RH7.1 when set correctly.
Jeremy.

Updated to latest from gnu.org.
Jeremy.

Added #define of int32 to int for cray. This will almost certainly cause
the rpc code to fail on the cray....
Jeremy.

Fixed incorrect debug parameters for lock_pid.
Jeremy.

Split pam_winbind out of WINBIND progs so it can be build separately
only if --with-pam is specified.

See if we can stick to slightly valid C..

(With thanks to tpot)

Andrew Bartlett

Fixed incorrect comment for cli_NetServerEnum()

Running torture-RANDOMIPC on the build farm doesn't go down as a 'good idea' as its logsfiles are BIG!

fixed socketpair_tcp for OpenBSD

Reran autoconf.

Fixed quoting bug - shell programming sucks.

Adding Michael Sweet's LSB packaging.
Jeremy.

- sorry, forgot to check a pointer

Applied Michael Sweet's patch, with one minor signal change.
Jeremy.

- sorry, forgot to test a pointer

Made talloc_realloc() semantics match realloc(). JF was complaining :-).
realloc(NULL) == malloc. realloc(p,0) == free() - a no-op in talloc.
Jeremy.

Missing prs_align() was causing Win9x nexus tools to fail. security_info_sent
was being miss-parsed.
Jeremy.

Fixed auto-alloc of dispinfo code when unmarshalling.
Jeremy.

Fixed extra file_free calls - my fault :-(.
Jeremy.

Very minor cleanup.  I was looking for another problem and found some bits
of code that were unclear.

Chris -)-----

This fix from Eelco Vriezekolk <eelco@nexus.com.na> is for a SIG11 bug where we
would always file_free(fsp) twice, once in close_file and once afterwoulds.

The bug was reported in SAMBA_2_2, but a code inspection shows it to be in HEAD
as well.  (Unfortunetly I don't have the facilites to actualy check this, but
the change is quite simple, makes sence and compiles).

Andrew Bartlett

Fix the torture test, we had the wrong file-name in the tests

Andrew Bartlett

Re-ran configure.

Only build the pam_winbind module if --with-pam used.

Replaced extra shell variable with_pam_for_crypto with a check against
$with_pam

We need strict locking to pass LOCK4

Andrew Bartlett

Start of smbtorture based testing.  Not all of smbtorture's facilites
are used, but these ones should pass.

Andrew Bartlett

Not all OSs have setbuffer, so we better check for it.

Andrew Bartlett

Make smbtorture return status values for use in the build_farm tests.

This should not have changed behaviour, and it has been tested.

Andrew Bartlett

Fixed the nastiest locking bug to track down.... smb_pids are sent in the
lockingX calls - use that instead of smb_pid in the packet.
Jeremy.

Insure caught the fact that PTRDIFFs were being done between two unrelated
pointers.
Jeremy.

"netbios aliases" and "interfaces" options change from P_STRING to P_LIST

Fix for building --with-profile for new vfscalls.
Jeremy.

cli_read() was reading too many bytes.

- fix bug in reply_nt- fix bug in reply_nt1

Make sure we compile with gcc by default.

Add a static linking target to the examples Makefile so I can test out static linking ...

Ensure we return correct error on trying to unlock a region not locked.
Jeremy.

Fix another small problem with Makefile.in ... not doing the correct
check for whether or not we should build a shared library.

This should return the build to normallity.

When you update an #ifdef'ed function, also update its matching null function
or all hell breaks loose on the build farm!

Andrew Bartlett

Fixed the first locking error (test #8 found by locktest code from
Clarion locktest.
Jeremy.

Modified Makefile.in to work properly. Define BLDSHARED as a variable in the makefile now.

These changes cause the libsmbclient stuff to always build a non-shared library and to build a shared library only for the four OSes that we currently like.

I will probably add HP/UX and AIX support based on the CUPS stuff soon ...

Added LsaGetConnectedCredentials patch from Manoj Naik <manoj@almaden.ibm.com>.
Jeremy.

Ensured all the system calls in msdfs.c go through the vfs layer.
Added vfs calls to symlink() and readlink() with appropriate configure
checks.
Jeremy.

From JF....

hi jeremy,

can you commit the following patch against HEAD. I can't do it right now

Thanks Tim for me. He changed the SAM_DISPINFO_1 array without checking if
he didn't break the server code. And he did.

So on my way I cleaned info_1, 2, .. 5

it may break winbind. I leave to tim the pleasure to fix it ;-)

        jf.

I added some talloc changes and checks for alloc fails.

Jeremy.

Experimental support for joining a domain without creating an account in
server manager first.  Just use the -U parameter to smbpasswd when joining
the domain:

smbpasswd -r PDC -j DOMAIN -U administrator%password

Should also work with domain users with the 'add workstation to domain'
user right.

Removed another silly static array.

Win9x weirdness.  When setting the named pipe handle state on a Unicode
server, Win9x only sends \PIPE instead of \PIPE\.  Looks suspiciously like
an off-by one bug in the Win9x dce/rpc pipe code.

Replaced memcpy() with memmove() to make safe_strcpy() safe for overlapping
source and destination.

Use a logical cli_read(), removed the cli_read_one() hack.
Jeremy.

param/loadparm.c: Fix for bad volume label, found by TAKAHASHI Motonobu
Samba Users Group Japan. Ensure same as in 2.2.x codebase.
Jeremy.

Added info to some of the debug messages to get a better handle on a problem
people are reporting regarding multiple responses to queries on <1D> names.

There should only ever be one LMB but some users are seeing multiple replies
to queries for the LMB name.  This is probably due to nodes on the LAN that
have NetBIOS over NetBEUI and/or IPX enabled.  Previously, the debug message
did not include the IP address associated with the name.  It *did* include
the source address of the packet, but in the examples I've seen all of these
were the same, eg:

[2000/06/22 11:58:25, 0] nmbd/nmbd_namequery.c:query_name_response(93)
query_name_response: Multiple (2) responses received for a query on subnet
129.130.10.136 for name NT.CIS.KSU.EDU<1d>. This response was from IP
129.130.10.24
 [2000/06/22 11:58:25, 0] nmbd/nmbd_namequery.c:query_name_response(93)
query_name_response: Multiple (3) responses received for a query on subnet
129.130.10.136 for name NT.CIS.KSU.EDU<1d>. This response was from IP
129.130.10.24
 [2000/06/22 11:58:25, 0] nmbd/nmbd_namequery.c:query_name_response(93)
query_name_response: Multiple (4) responses received for a query on subnet
129.130.10.136 for name NT.CIS.KSU.EDU<1d>. This response was from IP
129.130.10.24
[2000/06/22 11:58:25, 0] nmbd/nmbd_namequery.c:query_name_response(93)
query_name_response: Multiple (5) responses received for a query on subnet
129.130.10.136 for name NT.CIS.KSU.EDU<1d>. This response was from IP
129.130.10.24

Note that all of the above are reported as having come from 129.130.10.24.
This should never happen.  If 129.130.10.24 is a WINS server it should
send a Negative Name Query Response for a <1D> name query (wierd but true).

So, are all of the above coming from different systems, all of which
think are the LMB?  Are they all coming from one system that is, for some
strange reason, replying five times to the same query?

Anyway, I needed more info so I've changed the debug messages.

Chris -)-----

Properly instrument print_job_start()/print_job_end() so every fail case
logs a debug. We need this to track fail cases.
Jeremy

fixed guest account for build farm boxes

temporarily made smbclient a non-error target so that builds will succeed when it fails. This will give richard a chance to fix problems without breaking the tree

Syncup between 2.2 and HEAD.
Jeremy.

Jigger around with the tests a bit more

We now test security=share, security=user, security=server for both the
positive and negitive case (good/bad pw) and check that guest shares work for
share level security.

The server level security stuff seems to test positive without actualy
contating a server (another LIBSMB_PROG based smbd) - I will need to look into that...

Andrew Bartlett

on sco2 socketpair_tcp needs a bind

handle EISCONN in socketpair_tcp

this should get the auto-tests working on IRIX

Add the generic funcions file for the basicsmb tests

Andrew Bartlett

Added some missing entries to usage().

Added -d option to set debug level.

Added back the automatic build of libsmbclient.so in head.

This is so I can find out what platforms it fails on. I will pull it again tomorrow if there are too many problems, like > 2 platforms that it fails to build on, but will pop it back in again as I resolve platforms.

Added doco for the -d parameter.

Fix a stupid typo ...

Changed some of the tests around, made failures for parts actualy count

Andrew Bartlett

SGI compiler fixes.
Jeremy

Add check for working AF_LOCAL sockets, which are borken
on RH7.0 with _LARGEFILE64_SOURCE, and probably not
present on non-Unix systems like VMS.

Added the basic tests of smb functionality for HEAD

This works with the new build farm system

Andrew Bartlett

Ensure we always have a valid pointer on unmarshalling an SD with zero
ace entries.
Jeremy.

Put an 0x in front of a hex number.

Fixed bug introduced by changeover of security descriptor code from
malloc() to talloc().  Previously, creating an ACL containing zero ACEs
would return a non-NULL pointer to zero bytes of memory.  The talloc() code
would return a NULL pointer making the ACL a NULL ACL instead of an empty
one.  The difference is a NULL ACL allows all access and an empty ACL
denies all access.

We solve this by calling talloc(ctx, sizeof(SEC_ACE) * num_aces + 1).
Heh.

Always use DOMAIN\user first that this is the more specific case.
Jeremy.

fixed build

don't rush commits :)

fall back to "unsigned" for uint32 on systems that don't have one

Remove warning about trapdoor systems for non-root mode.

Andrew Bartlett

Ensure numeric group or user names don't get misinterpreted.
Jeremy.

Added Andrew's pam password change stuff. Needs some testing but looks good !
Jeremy.

Fixed stupid typo that would stop trusted domains working.
Jeremy.

Cosmetic fixups found while playing with the server manager.  Added support
for NET_SRV_SET_INFO rpc call which is made when double-clicking on a
computer in the server manager and changing the description.  We always
return NT_STATUS_NOPROBLEMO as NT doesn't seem to decode any error messages
passed back.

Maybe the changed comment string could be stored in a tdb and regurgitated
instead of the "server string" smb.conf parameter?

got rid of the date headers off all text docs

fixed usage of socklen_t and also tidied up SIG_ATOMIC_T, using a typedef instead of a define

make sure we have BOOL in autoconf usage of util_sec.c

added a -L option to smbpasswd to force it to run locally so we can test smbpasswd as non-root

- make the regresison test mode code build in by default. This should
  allow us to have test targets without special configure options
- fixed make proto so that it actually does something

fixed error
using wrong pointer to test and free s/str/s/

Log debug before and after netbios names copied so we know if they've been
changed.
Jeremy.

uhm some pointer went wrong

handle P_LIST parameters

fix compiler warnings

fix compiler warnings

Added other_safe_chars to alpha_strcpy(). Needs testing but is a better
fix for the problem.
Jeremy.