Got to the bottom of another weird one...

Piping the output of smbmount back to autofs/automount was causing the
automount process to hang.  Reason was that automount was depending
on the pipe to close to continue on, rather than detecting the child
signal.  This occured with debug enabled and the daemon process was not
closing the stdout process.  Disabling debuging avoids the problem.
Debugging is turned off in the cvs repository and a warning placed over
the debugging option.

rpcclient shutdown command

bitmap to strings

Always null-terminate strings.
Also some string length and sizeof(pointer) corrections.

Always null-terminate strings.

Added caching of user password if it was entered via getpass.  We were already
caching the password if it came from the command line or from the environment.
This completes the set and deals with the bloody nusance when reconnecting
a connection after we have gone daemon...  Grrr...

Added code to clean up the mount point following a catastrophic failure
during reconnect.  The smbmount daemon was exiting but leaving the mount
point in an unusable state.  If smbmount must exit following a reconnect
failure, we "unmount" the mount point and clean up mnttab.  Currently,
the unmount works, fixing some really ugly I/O errors and failure when
trying to remount.  The cleanup on mnttab still has problems.  This
is better than what it was and doesn't break anything that wasn't broken
before, so I'm committing this in even with the mnttab problem.  Will
commit the fix to that when I figure out what is busted there...

-mhw-

Folding into Samba-2.1.X tree documents added to 2.0.3.

Fixed a glibc glitch in smbumount.c and reenabled some debugging
in smbmount.c

const cast issues.  [p.s - tidy work, matt!]

const issues with byte mod / byte read macros.

Cut and paste error.

UNICODE cleanup (see lib/util_unistr.c).

No more ugly static library buffers and all functions take a destination
string length (especially unistrcpy was rather dangerous; we were only
saved by the fact that datagrams are limited in size).

UNICODE issues.

the UNICODE issue...

enum dom users buffer was 0x80 not 0x8000.  must fix "enumeration" code
as you need to make multiple enum dom users calls.

refinement of random ipc$ SMBtrans torture test.  send requests, and
don't wait for responses :-)

uninitialised pointer being freed if lookupnames failed.  oops.

use jeremy's versions of the UNICODE routines.

Having Well-known Domain Groups ("Domain Admins/Guests/Users") returned
under SID_NAME_ENUM 0x5 instead of 0x2 (Well-known group instead of
Domain Group) was making it impossible to view these groups from USRMGR.EXE.

groups and aliases being "manually" added which do not necessarily have
representation in the underlying database: removed this code.

for example, with the nt->unix mapping system (don't know about the ldap
one) if you want "Domain Admins" to appear you _must_ put it in the
domaingroup.map file.

the previous code was adding builtin aliases and well-known groups even
if they weren't in the domain maps.  bad idea.

when multiple independent large rpc calls come in on the same pipe,
prev_pdu_file_offset was not being re-initialised to zero.

removal of the use of unistrn2 function: replace it with unistr2_to_str
which is more appropriate.

pwdb_smb_to_sam was not returning NULL for nt name so that
pwdb_sam_map_names() was using a "blank" static string instead of
a NULL pointer for nt names.  NULL means over-ride, so the nt name
got left as "blank".

this causes nt clients to terminate with extreme prejudice.

Fix for NT BSOD problem. There's no reason to have two "NT usernames" running
around anyway.

The real problem is, once again, the brokenness of pwdb_sam_map_names et al.
This time it is deciding to return blank NT usernames, which NT's redirector
objects to.

I'm currently working on improving the pwdb/mapping code, should be ready in a
couple of weeks.

UNICODE byte ordering issue: typecast to uint16* replaced with SSVAL()

reload_services needs to be called prior to init_files but after
get_myname.

iteration of sam passwd entries was an order n-cubed algorithm due
to resolution of unix name to nt name being unnecessarily _inside_
another loop.

initialise my name (used in %h) prior to loading smb.conf files.

%s not $s

corrections to get data stream for 2nd and subsequent pdus copied from
right place (forgot to subtract 0x18 header bytes)

cache unix groups so that two-level getgrent calls don't occur.

multiple dce/rpc PDUs failed to work after ntlmssp update was added.

server type announcements modified to include the "role" of the server:
domain member, pdc, bdc.

Minor change to bring smbmount in the main branch in line with some bzero
to memset changes...

comma after DOM_MAP_USER removed

Must set password length to 24 after we encrypt a password.

Initialise NTTIME properly in make_reg_q_enum_key instead of using
unix_to_nt_time hack. Seems to me it's ignored anyway (dummy return
buffer?).

Fixed a domain functionality problem where NT clients would start
endlessly repeating a network SAMLOGON (hoping it to change, hmmm...).

( Guess what I found in pwdb_init_sam...

  unix_to_nt_time(&user->logon_time, (time_t)-1);
  unix_to_nt_time(&user->logoff_time, (time_t)-1);
  unix_to_nt_time(&user->kickoff_time, (time_t)-1);
  ... )

Added init_nt_time function which initialises an NTTIME to -1.

Ok...  Yet another round of fixes for smbmount and autofs.

1)  The earlier fix for the smbmount race conditions broke the PID
registration with smbfs.  That fix has been backed out and
replaced by a signalling convention from the child smbmount
process back to the parent telling the parent when it is safe
to exit.

2)  Fixing all of this uncovered a NASTY deadly embrace between smbmount,
smbmnt, and autofs.  This was caused by the setsid call in the
daemon code.  The smbmnt process no longer was registered as
"magic" because it was no longer in the autofs process group.
Many many kudos and thanks to H Peter Anvin for giving me the
clue to solving this agravating puzzle.  The setsid was moved
down the where the child signals the parent and a warn left in
its place in the daemonize code.

3)  Fixed (actually worked around with a BUTT UGLY HACK) a problem with
SMB_GET_MOUNTPID in smbumount.c.  The smb_fs.h header file has
the parameter to this ioctl defined as a uid_t.  Unfortunately
that's a 32 bit quantity under glibc and it's currently a 16 bit
quantity in kernel space.  Undefined the macro and redefined
it with a parameter of __kernel_uid_t.  That should keep us
out of trouble till I can have someone fix smb_fs.h in the
kernel sources...

fix for enumerate domain users (bug spotted by sean matthews).

also needed to use start index properly and generate next index.

both client and server code need to recognise error code 0x105
when there's not enough room to store all the users in one call.

sort this out another time.

removed encrypt-password code pre-cli_session_setup(), session setup
fn decides whether to encrypt password or not.

uni_svc_name not uni_srv_name

rpcclient "Service Control Manager" svcenum [-i] command.

error code cleanup

returned cli_session_setup to previous behaviour.  added a couple of
validation checks and also added capability to send plaintext passwords.
send "ntpasslen" of zero to do this.  sending same plaintext password
for pass and ntpass arguments will result in previous behaviour of
encrypting password if server supports it.

Service Control Manager - service enumeration.

set_port warning / unused / global/local variable overlap issues

cmd_svcctl.c tests got put into cvs by mistake.

Fuss fuss fuss.

Added missing default: statements to two switch blocks.

Chris -)-----

- got client code cleartext passwords working again in cli_session_setup.
  needed this for some tests.

- removed code that said "if lm password is not encrypted then encrypt both
  lm and nt passwords".  actually it said "if lm password length is not 24
  bytes and we're in security=user mode..."

  it didn't bother to check whether the nt password was NULL or not, and
  doing the encryption inside cli_session_setup is the wrong place.

- checked all instances where cli_session_setup is called with cleartext
  passwords that are expected to then be encrypted (see above) with the
  test "if pwlen != 24...".  there was only one: all the others either
  provide encrypted passwords, do null sessions or use
  cli_establish_connection.

* recommendation: use cli_establish_connection() in smbwrapper/smbw.c

compilation errors due to addition of smb file handle parameter.

Putting back the -p flag in smbclient.

However, it seems that the -s flag
in smbclient is also ignored :-(

More changes to get SSL working with 2.x.x

Now, we have most of the configure support done, next
have to fix Makefile.in and other things ...

Fixing up configure to properly support ssl

Adding first of the fixes for SSL.

A whole bunch of string variables in loadparm.c were
not being initialized properly.  Programs crashed as a result.

This set of code not tested, but same code tested elsewhere,
and all this guarded by #ifdef HAVE_SSL

In security=user mode we must allow cli_connect_serverlist to connect to our
own smbd process, rather than complaining about a password server loop.

Added load_interfaces in smbpasswd to allow name resolution by broadcast and
multiple interfaces. (Jeremy already committed this in SAMBA_2_0).

A small change to clitar.c (really, I promise :-)

If we are writing the tar file to stdout, set dbf to stderr
so that we do not screw up tar output with log info etc.

Compiles clean and tested with 38MB backup. Honest :-)

eclass != ERRDOS && num != ERRmoredata
is not the same as
!(eclass == ERRDOS && num == ERRmoredata)

This was causing smbclient to segfault on receiving certain errors.

Fixed typo in srv_samr.c where samr_add_groupmem and samr_del_groupmem were
the wrong way around.

Finally committing my LDAP changes.

* Added new APIs for modifying groups.
* RIDs are allocated similarly to NT, starting from 1000 and incrementing by 1
  for each new user/group.
* RIDs are now consistently in hex

* Fixed bugs reported by Allan Bjorklund <allan@umich.edu>:
   - ldap_close_connection is exported by OpenLDAP - changed to ldap_disconnect
   - Missing ldap_connect() in getusergroups functions
   - ldap_next_entry was being called too early while retrieving a sam_struct
   - LDAP globals should be extern in sampassldap.c

* Fixed bugs reported by Martin Hofbauer <mh@bacher.at>
   - Newly added workstation trust accounts had attributes DU rather than W.
   - User dn's were forced to start with "uid=XX" rather than using the existing
     dn.

Fixed the "You password will expire in 0 days.  Would you like to
change it now?" message when you login to a Samba Controlled domain.

The fix is a hard coded 42 days from right now until you need to
change you pasword again time (see passdb/sampassdb.c:pwdb_smb_to_sam())

Also fixed getsmbfilepwent() so that it will read in the last password
change time correctly.

* Related to this lib/util_pwdb.c:StrnCaseCmp() returns 0 if the
strings match.  Chouldn't this be the pther way?  Oh well.  I
didn't change the return code as it was used in several
other cases (see lib/util_pwdb.c:pwdb_get_last_set_time())

fixed an uninitialized variable in lookupsmbgrpgid() and
lookupsmbpwuid that was causing a SEGFAULT in smbd.

Updated the docs to match the changes I made to debug2html.
Chris -)-----

While writing the man page, I realized that it was a bit silly not to accept
command-line parameters.  I've added that capability.  Of course, the man
page is now wrong.  Sigh.

Chris -)-----

First go at documentation for debug2html.

Chris -)-----

Sorry that this is going so slowly.

I've added debug2html to Makefile.in so that it compiles as part of the
normal build.  Fixed a typo in debug2html.c as well.

One problem:  I found it necessary to link with both $(PARAM_OBJ) and
              $(LIB_OBJ).  The result is an executable that is much larger
              than it really needs to be.

Modified packaging-prep script to remove use of "rm -f *".

Changing to PHT standard SPEC settings.

fix for potential lsass.exe crashing due to negative response from
LsaLookupNames being incorrect.  this is a bit wierd: why would the
lsass.exe on the nt _client_ crash due to an LsaLookupNames response
from a samba _server_?

adding start of remote lookup for domain member role.

lib_sec_ctxt.o not .c

Updated makefile-path info contributed on the samba-technical mailing list.

Fixed problems in debug code because I did not compile
first :-(

Added some debugging to clitar ...

Fixed tar recurse bug.
Jeremy.

Fix bug with nmbd running wild due to recursion in retransmit_or_expire_response_records().
Jeremy.

and in head branch:

make sure we include sys/mman.h if available

A while back, Andrew and I talked about making the debug parsing code a
better "fit" with other Samba code.  This is a small first step toward
doing what (I think) we agreed to do.

I've moved the key function from ubiqx/debugparse.c into lib/debug.c.  I
have also moved the enum from ubiqx/debugparse.h into the debug section in
smb.h.

The next thing to do is to get debug2html added into the Makefile.in so
that it is always produced when compiling the suite.

Chris -)-----

on Linux force fcntl/mmap based shared memory and on other systems
force IPC/shmem based shared memory.

server_cryptkey() now calling cli_connectserverlist().  stupid microsoft
idiotic *SMBSERVER connectionism added to cli_connect_serverlist().
also added check for protocol < LANMAN2.

added sid_name_use array argument to lsa_lookup_names and lsa_lookup_sids.

trying to track down issues in get_home_dir().

%U substitution should be unix user not nt user

adding svcctl parsing code

bugfix in smb_err_msg

oops, util_pwdb.c appears to be included in PASSDB_OBJ not LIB_OBJ.

adding some enumerate services code, client and server.

need to initialise global_myworkgroup

removed the SID stuff from the head branch as well.

This allows the removal of PASSDB_OBJ, RPC_CLIENT_OBJ and
RPC_PARSE_OBJ from nmbd in the head branch.

so nmbd just went on a diet :)

adding "Service Control Manager" commands to rpcclient.

Adding George's Mac extension code.
Jeremy.

Adding George's Mac extension include files.
Jeremy.

ldap_getpw() also needs to be (void)

compilation warnings due to missing (void) in ldap_close_connection.

adding srvsvc pipe.

added ldap files by Matthew Chapman.

Matthew Chapman spotted that smbpasschange was assuming the existence
of a private/smbpasswd file, this will not be the case for other
database APIs.  removed startsmbdb and endsmbpwdb calls because
add_smbpwd_entry() and mod_smbpwd_entry() don't need them.

fixed warnings (and potential errors) due to integer overflow when
creating locking masks

added text and html versions of LDAP.yo.  smb.conf.5 not generated yet
because there were parts rejected from the ldap patch file.