This commit was manufactured by cvs2svn to create tag
'release-1-9-18alpha1'.

move calls to smbhash() inside smbdes.c (for legal reasons)

update the timestamp on config files in our linked list when we notice
they have changed, otherwise we keep reading them again and again

debugging the srvsvc Net Share Enum.  some padding issues; some string length
issues.  it works!

creating NetShareEnum parsing / reply creation.  compiles, but untested.

Makefile :

added srvparse.o and pipesrvsvc.o

smb.h :

mods to the Net Share Enum stuff

srvparse.c :

Net Share Enum parsing support.  more srvsvc pipe parsing to go here...

pipenetlog.c util.c:

modified standard_sub_basic() so that you can set a global boolean
and use a different string for the %U username substitution.

proto.h:

the usual.

added the following structures:

SH_INFO_1
SH_INFO_1_STR
SHARE_INFO_1_CTR
SRV_Q_NET_SHARE_ENUM
SRV_R_NET_SHARE_ENUM

the share info1 container can handle a maximum of 32 shares.  the share info1
string structures contain two unicode strings, with 1024 uint16 characters
_each_.  we're going to _have_ to dynamically allocate the unicode strings,
else we'll run out of stack space.  rapidly.

Added comment submitted by Frank Varnavas.

pipes.c pipesrvsvc.c :

moved stub srvsvc pipe function into separate file, in preparation for
further work.

add a section on encryption and change the formatting a bit

put the NT_STATUS_ALLOTTED_SPACE_EXCEEDED error in instead of 0x99.
whatever that means.

Fixed small typo.

After talking with Andrew added "interfaces only" parameter that
turns off the filtering on the broadcast socket by default.
Jeremy (jallison@whistle.com)

.cvsignore: Added make_smbcodepage
interface.c: Added is_local_net().
locking.c: Added Fix for zero length share files from Gerald Werner <wernerg@mfldclin.edu>
           plus a race condition fix for the fix.
nameannounce.c: Made function static.
namedbresp.c: extern int ClientDGRAM removed - not used.
namedbserver.c: extern int ClientDGRAM removed - not used.
namedbsubnet.c: Added code to make sockets per subnet.
namepacket.c: Added code to read from all sockets & filter.
nameresp.c: extern int ClientDGRAM removed - not used.
nameserv.c: Indentation tidyup :-).
nameserv.h: Added sockets to struct subnet.
nameservresp.c: Improved debug message.
nmbd.c: Changed to terminte on listen_for_packets exiting.
nmbsync.c: extern int ClientDGRAM & ClientNMB removed - not used.
proto.h: The usual.
util.c: Fixed debug message.
Jeremy (jallison@whistle.com)

add a note about running testparm after modifying smb.conf

put the new example config file in here as well

Simply moved the #include for ubi_dLinkList.h from within dir.c to
includes.h.  More consistent with current practice.

make_unihdr() was setting the length to 1/2 the required value.

pipenetlog.c lsaparse.c smb.h :

SAM logon sorting.  too many buffer pointers.  added in the missing
switch value (value of 3).  dealing with the buffer pointers to the
user info structure in a slightly different way.

smbparse.c smb.h :

uni_max_len and uni_str_len are the other way round, in UNIHDR.

util.c :

increased the show_msg() data size from 256 bytes to 512 bytes:
the LSA SAM Logon response can be about 500 bytes long.

pipenetlog.c :

forgot to set the authoritative field to 1.

set the wrong info level (0x0C00 0000 instead of 0xC000 0000) so that
nt got confused and thought that the server password had been set instead
of refused.  this is for the SRV_PWSET response.

remove a spurious error message from nmbd. The message was:

WINS server did not respond to name registration!

this messages should only be produced when we are trying to register
our own names, not when we are the one with the name database and we
are doing a NAME_REGISTER_CHALLENGE on behalf of another host (for
secured wins registration)

fix the handling of negative name query responses and the handling of
packets with no answer section in general.

The fix has 2 parts:

1) set ans_name to the name we queried if nmb->answers == NULL

2) check for nmb->answers == NULL in several other places where we
currently check for nmb->answers->data

While doing this, I noticed there are lots of places in our nmbd code
where we make assumptions about the packets being well formed. Someone
could easily implement a denial of service attack on nmbd by sending a
packet that causes a null pointer dereference. Does anyone feel like
going through the code and adding checks? Probably the best solution
is to have a single function that "validates" a packet, making sure
that all the required fields are there. This will be a bit tricky as
what fields are required varies a lot between packets. A first pass
would be a function that prints "SUSPECT PACKET" when it hits a packet
that it suspects does not have a required field (or the field is badly
formatted), then we could use this on a live system to find any cases
we've missed.

Any takers?

document -a "append log" option

added -a "append log" option

smb.h :

id_info_1 has a pointer at the front of it.  so does return credentials,
and so does the client credentials.  these are all from the sam logon.
auth_level is 16 bytes not 32 and is actually called a switch_level.

smbparse.c :

smb_io_unihdr() - uni_max_len and uni_str_len are 16 bytes not 32.
this may have a knock-on effect on smb_in_unihdr2() but we'll see...

dealing with some stack overflow bug somewhere around the use of
deal_with_credentials() by moving important code to the beginning
of the function :-) :-) :-).

the new seed (old_cred + time + 1) was getting corrupted.

inside the auth 2 response bits, i wasn't storing the received client
credentials for the calculation of the next credentials: i was storing
the auth 2 calculated credentials.

oops.

add ERRDOS/67 ERRnosuchshare to the client.c list of error strings

Added Michael Johnsons' PAM modifications <johnsonm@redhat.com>
This patch has been checked over. JHT

fixed the problem that browsing breaks if you put quote marks around
the server string option.

I fixed it by adding: trim_string(ret, "\"", "\"") to lp_string()
which means that it removes leading and trailing quotes from _all_
strings in smb.conf. I think this is what we want as I can't actually
think of any case where quoted strings are needed.

I suspect that this fix will actually fix browsing at quite a few
sites. It's not uncommon to see people putting "" marks around the
workgroup name or other essential strings.

The real reason it was causing browsing to fail is that the browse.dat
file uses " to delimit fields, which means that it wasn't being parsed
correctly. It would be nice to use a proper database format in
browse.dat sometime. In fact, we really need a generic database type
subsystem in Samba to replace the mish-mash we currrently use
(browse.dat, wins.dat, smbpasswd etc)

Moved the LGPL from the ubiqx directory into the samba root directory
and renamed it COPYING.LGPL.  This is because the ubi_* files are under
LGPL, not regular GPL.
Removed the last few things from the ubiqx directory and deleted it.

Don't need it any more.

Made changes to the dir cache functions:
- They now use the ubi_dLinkList linked list code.
  This is not a big gain, I suppose.  It would be significant if there
  were lots of doubly-linked lists in the code and I replaced them all.
  The only other advantage is that the code is more modular, which
  appeals to my own sense of order, if no one elses.  :-}
- I allocate space for the entry structure and the strings in one go,
  instead of using malloc() and separate strdup() calls.  This should
  be more efficient, and allows for a single call to free() to free the
  whole thing.
These are very minor changes, but they do serve to make me more familiar
with the code overall.

This code manages a simple doubly-linked list.

I got fed up trying to put the tree and list stuff into a subdirectory,
so I've started to move them into the main directory.

ipc.c: Adding Andrews become_root code to the main branch.
locking.c: Adding Andrews become_root code to the main branch.
pipes.c: Fixing the close_file issue.
proto.h: The usual.
reply.c: Move smb_pass into NTDOMAIN defined code. Fixing the close_file issue.
server.c: Fixing the close_file issue.
trans2.c: Fixing the close_file issue.
uid.c: Adding Andrews become_root code to the main branch.
Jeremy (jallison@whistle.com)

smb.h smbparse.c pipenetlog.c :

whoops, the SAM Logon structure was wrong.  updated this, and
cifsntdomain.txt.  more debug info in pipenetlog.c.  the crash
is somewhere around deal_with_credentials().

byteorder.h :

put in uint8, uint16 and uint32 typecasts around debug info, because
sign extending was resulting in ffffffe8 being displayed instead of e8.

credentials.c :

some debugging info, because i'm tracking a coredump.  without gdb.
nothing like making things difficult.

reply.c :

whoops, missed this (important) bit from paul's code, which tells
the NT workstation that the MACHINE$ entry doesn't already exist,
and we're going to create a default entry with a password "machine"
right now.

proto.h:

the usual.

Added ubi_sLinkList module which manages simple singly-linked lists.

added srvsvc basic pipe, straight from paul's code.  does NETSHAREENUM
and NETSERVERGETINFO.

the usual proto.h update

add the port number to a debug statement

added loopback_ip. This is used to detect packets from ourselves

fixed the freeze on logout bug. The fix has several parts:

1) add a new parameter to queue_netbios_packet(), the "reply_id", this
is the id that should be used when sending a further response to the
packet (such as a response after we get back a reply to a name query
after senidnga  WACK). reply_id is 0 (meaning unused) in most cases.

2) fix the id used in the reply in add_name_respond() from
response_name_query_register()

3) remember to remove the response record at the end of
response_name_query_register()

4) get the right IP address (it was 0.0.0.0) in
response_name_query_register()

5) add a new field reply_id to struct response_record

fixed a stack overflow bug in api_lsa_req_chal()

changed the order of arguments to smbhash() in credentials.c. Luke,
when you changed from E1() to smbhash() you didn't notice that the
arguments are in a different order. This is why your new code was
failing.

NT logon still fails, but now gets to SAMLOGON. It shouldn't take much
to get it working now.

force the salt to be a maximum of 2 characters long in calls
to crypt()

This might solve some password problems, particulary on HPUX

These are the ubiqx modules, as included with the Samba distribution.
Updated the linked list module, which has new and changed macros.

First draft of a README file to explain the origins of the "ubiqx" code.

Added a very small piece of documentation to describe the binary tree
modules.

pipentlsa.c :

lsa close odd bug.

smbparse.c :

smb_io_dom_sid() _does_ need 4-byte alignment before it.

Added a copy of the LGPL to the samba/ubiqx/ directory.

byteorder.h:

created a RW_PIVAL macro which was missing.

smbparse.c:

smb_io_dom_sid() was storing its sub-authorities as uint16s instead
of uint32s.  used the DBG_RW_PIVAL macro instead of DBG_RW_PSVAL.

pipentlsa.c:

not sure.  something to do with the Query Info reply.

pipeutil.c:

make_rpc_reply() had the packed representation field set to 0x0100 0000
instead of 0x1000 0000, which had the interesting result of turning all
uint32 and uint16 field byte ordering the other way round!

credentials, query info reply.

docs/DIAGNOSIS.txt:
Added comment right up front about "your server is being unfriendly"
type error messages.

source/Makefile:
Added compile options for RedHat Linux 5.0 series.

changed *(uint32*)(clnt_cred) to SIVAL(clnt_cred, ....)

Broke my own rule about checking in uncompiled source :-(.
Jeremy.

Added fixes for become_user braindamage.
Jeremy (jallison@whistle.com)

another alignment of client challenge: this one in lsa_io_q_auth_2().
there are going to be a few of these...

lsa_io_r_req_chal() - challenge is not 4-byte aligned after the unicode
                      strings.

put a check around become_user().  doesn't authenticate the user, but doesn't
quit the whole pipe, either...

resolving some of the confusion over credentials.

split pipes.c down into util, netlog and ntlsa.

debugging... no idea what i'm doing.

reverted a change made by Luke at his request.

Luke, when you don't know what has been changed in the CVS tree I
highly recommend you point your browser at:

http://samba.anu.edu.au/cgi-bin/cvsweb/samba/source

If you click on a filename you can then see all the commits and
changes that have been made to it over time. You can also download any
version of the file or find the differences between any two versions.

All of this is not dependent on the state of your local CVS sandbox,
so it can be used to find out the "true" state of the tree at any
time.

If you suspect some sort of CVS problem (like a change getting
reverted) then please use the above URL to work out what has
happened. You should be able to see exactly who made what changes and
when.

debug info added

JHT ==> Getting ready for 1.9.18alpha1.
examples/smb.conf.default:
added line "share modes = no" to netlogon share parameter list

packaging/redhat:
updated RedHat RPM build files.
added smbadduser script (modified for RHL).

checked in a file with a compile error.  oops!

byteorder.h :

debugging output wasn't (still isn't) perfect.

credentials.c lsaparse.c smbparse.c :

added DEBUG strings.

pipes.c :

lost some changes, to do with setup of RPC headers.  arg.

JHT ==> Removed linefeed that caused problems.

align_offset() adjusted pointer to wrong location.  oops.

updated the "comment =" entry a bit

updated the fake oplocks entry a bit

getting somewhere.

ipc.c :

removed srvsvc pipe reference: have to do that.

pipes.c lsaparse.c smbparse.c :

more debugging info.  looks a bit like netmon output.

updated rpc header reply: callid wrong; alloc hint a uint32 not a uint16.
still doesn't get rid of the netlogon trans2 request with zero data.

ipc.c:

debugging info.  found that data = NULL because of short packet length
indicated from the ntlsaRPC pipe _royally_ stuffs NT's packet handling.
maybe this should go down as a service denial bug to the ntbugtraq list.

pipes.c lsaparse.c smbparse.c :

added more debug stuff.  added length of header to data_len in MSRPC
fragment_length field (0x18 bytes short) which caused the above bug
from NT 4.0.  oops.

added debugging macros (suitable eventually for use in tcpdump, hopefully)

Added Finland ftp mirror (PR#175) (Paul)

remove { and } from the list of illegal characters in filenames. The
CIFS3 spec does not list them as illegal.

This allows things like the control panel icon to be placed on a Samba
drive.

JHT ==> Luke, your forgot to change the modified date!

namelogon.c :

added NTLOGON.  (fixed the case 0:)

namepacket.c:

call to NTLOGON mailslot

nameserv.h

defines for NETLOGON and NTLOGON mailslot

util.c:

added andrew's dump_data() function.

updated Sunsite UK. (Paul)

Updated mirrors, combined Sources and Sources&Binaries as our binaries
are now with our sources.
Have requested that mirror sites remove the old 'binaries' tree.
Paul.

the usual

added api_lsa_sam_logon() and api_sam_logoff().  that's it.  lots of
run-time debugging, now.

added lsa_reply_srv_pwset()

ipc.c :

#if NTDOMAIN
call to api_netlogRPC
#endif

lsaparse.c :

renamed lsa_io_q_auth2 to lsa_io_q_auth_2.

pipes.c :

added api_lsa_reply_auth_2() and api_netlogRPC.

proto.h :

the usual.

I have to admit that I'm not very good at writing Makefiles.
This one will simply create an object file from each of the .c files
in the directory.  These could be linked into a static library, but
I'm not sure that this is worthwhile.  If anyone wants to improved on
what I've got here, please do!  crh -)-----

Makefile:

added credentials.c to smbd

credentials.c:

using credential structures instead of char*

password.c uid.c server.c:

added sid and attr to user_struct.

smbdes.c:

smbhash and str_to_key make public instead of private.

pipes.c smb.h:

lsa structures, sub-functions.

proto.h:

usual.

This is the ubiqx binary tree and linked list library.
This library is being included as part of the Samba distribution.
(Hurray!)

After generating some discussion, listening to the opinions, and thinking
about it for a while, I've decided to move the tree & list code into a
separate subdirectory.

After generating some discussion, listening to the opinions, and thinking
it over, I've decided to move the tree & list code into a subdirectory.
CRH

corrected paragraph giving the impression that samba can be a domain master
for multiple workgroups.

fixed the log wrapping bug.

This is a very nasty bug that I think explains quite a few
intermittent problems people have been having with Samba.

It may be worth checking on other cases where errno can be overwritten
by seemingly innocuous things (in this case a DEBUG() line)

nmblookup.c: Added -A ability to do status on ip address.
smb.h: Added defines we will need for NT SMB calls.
trans2.c: Fixed SMB_QUERY_FILE_ALT_NAME_INFO return - this is only
for short name returns (and only used when you negotiate NT SMB calls
to boot !).
Jeremy (jallison@whistle.com)

Added in oplock info.
Jeremy (jallison@whistle.com)

Added definition of oplocks parameter.
Jeremy (jallison@whistle.com)

local.h: Fix spelling mistake :-).
namedbsubnet.c: Stop registering 1x name unless we can be a local master.
reply.c: Remove ERRbaddirectory code.
server.c: Remove abort() - use exit_server() instead.
trans2.c: Remove ERRbaddirectory code.
Jeremy (jallison@whistle.com)

added #ifdef NTDOMAIN.  added call to api_ntLsarpc instead of api_Lsarpc
in ipc.c iff NTDOMAIN is defined.

moved the array macros outside of the CAREFUL_ALIGNMENT define.

pipes.c:

added api_ntlsarpcTNP() function.  hooray!

smb.h:

added LSA #defines needed by above function.

credentials.c:

use UTIME structure (defined and commented in smb.h to be time, secs,
                         since 01jan1970)

pipes.c:

another sub-function.

util.c:

added char *unistr2(uint16 *buff) function.  same as unistr except
    it takes uint16* instead of char*.

smbparse.c smb.h:

more structure sorting.

proto.h:

the usual.