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

preparing for release of 1.9.18alpha5

updated to describe smbclient work recently done.

latest version

oops!

added correct client-side credential generation / checking to the LSA SAM
Logon query.  i think i even got the client-side checking of the response
credentials right!

got the SAM logon request generated, and received a SAM logon response back.
YEAH!

need to add:

- client-side credential calculation
- client-side parsing of the SAM logon response.

sam logon query creation functions.  written twice because i deleted it.

Makefile :

adding bits for new nt domain code

byteorder.h :

trying to get macros right, and not to crash on SUNOS5...

client.c :

added #ifdef NTDOMAIN, and created do_nt_login() function.  don't
want to have to recompile client.c unless absolutely necessary.

credentials.c :

moved deal_with_creds() [possibly inappropriately] into credentials.c

ipc.c reply.c server.c uid.c :

attempting to make (un)become_root() functions calleable from smbclient.
this is a little tricky: smbclient might have to be another setuid
root program, immediately setuid'ing to non-root, so that we can
reset-uid to root to get at the smbpasswd file.  or, have a secure
pipe mechanism to smbd to grab smbpasswd entries.  or the like.

smbdes.c smbencrypt.c :

created a function to generate lm and nt owf hashes.

lsaparse.c ntclient.c smbparse.c :

added nt client LSA_AUTH2 code.  it works, too!

pipenetlog.c pipentlsa.c pipesrvsvc.c :

simplification.  code-shuffling.  getting that damn offset right
for the opcode in RPC_HDR.

smb.h :

changed dcinfo xxx_creds to DOM_CRED structures instead of DOM_CHAL.
we might need to store the server times as well.

proto.h :

the usual.

update SGI build files to latest source

preparing for release of 1.9.18alpha4

preparing for release of samba-1.9.18alpha4

updated to reflect the current state of play:

- jeremy's nmbd restructure (wins client / wins server stuff)
- which parameters are documented in smb.conf and which aren't
- mentioning the new smbclient work-in-progress

default server_name if NIS server is unavailable should be the local
machine, not a NULL string.

nmblib.c :

adding some debug info

pipenetlog.c pipentlsa.c pipesrvsvc.c :

using unistrn2 instead of unistr2 in the SAM logon username.
wrong offset for command in request (use "reserved" field
not cancel_count.  AGH i'll get there)

nterr.c :

added a structure that wraps nt errors as strings and enums, so we
can do a smb_nt_error() function.

Makefile ntclient.c :

added ntclient.c, broken out nt domain stuff into a separate file.
getting fed up of compile-times and size of client.c.

fixed the do_lsa_req_chal() function.  made it read the response,
and return the challenge credentials received from the server.

next stop: do_lsa_auth_2().

client.c :

removed nt domain logon functions into a separate file.

pipenetlog.c pipentlsa.c pipesrvsvc.c smbparse.c :

i'd broken the offsets of the RPC_HDR while trying to sort out the
nt client code.  fixed it again.  added some robustness stuff.

util.c :

the unistrn2() function was null-terminating the string at one
character too many.

Big change to make nmbd code more readable/understandable.

Main change is removal of find_name_search() confusion.
This has been replaced with find_name_on_subnet() which
makes it explicit what is being searched.

Also changed wins_subnet to be wins_client_subnet in
preparation for splitting the wins subnet into client
and server pieces.

This is a big nmbd change and I'd appreciate any
bug reports.

Specific changes follow :

asyncdns.c:
     Removed wins entry from add_netbios_entry(). This is now
     explicit in the subnet_record parameter.

interface.c:
     iface_bcast(), iface_nmask(), iface_ip() return the
     default interface if none can be found. Made this
     behavior explicit - some code in nmbd incorrectly
     depended upon this (reply_name_status() for instance).

nameannounce.c:
     find_name_search changes to find_name_on_subnet.

namebrowse.c:
     wins_subnet renamed to wins_client_subnet.

namedbname.c:
     find_name_search removed. find_name_on_subnet added.
     add_netbios_entry - wins parameter removed.

namedbsubnet.c:
     find_req_subnet removed - not explicit enough.

nameelect.c:
     wins_subnet renamed to wins_client_subnet.

namepacket.c:
     listening() simplified.

nameresp.c:
     wins_subnet renamed to wins_client_subnet.

nameserv.c:
     find_name_search moved to find_name_on_subnet.

nameserv.h:
     FIND_XXX  -> changed to FIND_SELF_NAME, FIND_ANY_NAME.

nameservreply.c:
     find_name_search moved to find_name_on_subnet.
     Debug entries changed.

nameservresp.c:
     wins_subnet renamed to wins_client_subnet.

namework.c:
     wins_subnet renamed to wins_client_subnet.

nmbd.c:
     wins parameter removed from add_netbios_entry.

nmbsync:
     wins_subnet renamed to wins_client_subnet.

proto.h: The usual.

server.c:
     remove accepted fd from fd_set.

Jeremy (jallison@whistle.com)

renamed static cli_setup_pkt() to static cli_setup_packet() because it
clashed with the currently-used cli_setup_pkt() in clientutil.c

commas in the wrong places...

missed nmbsync.c when adding setup count and setup pointer arguments to
cli_call_api().  sorting this for jeremy.

there is a discrepancy between the client.c list_servers() and the nmbsync.c
add_info() calls to cli_call_api() - one has an mdrcount of 1024, the other
of zero.

i don't know what difference this makes.

general sorting out, from crashes generated by do_lsa_req_chal() in client.c

trying to set up the data parameters etc and not understanding what's going on.

in api_netlogTNP, added smb_io_rpc_hdr() call to decode the header received
(and in this instance, generated by do_lsa_req_chal()).  and then noticed
that it's two bytes out.  but i don't know how to do "byte parameters"
and it's not the same format as the LSA_REQCHAL received from nt workstations.

agh!

missed one of the files (lsarpc) in the touch and chmod instructions.

playing about, trying to SMBopenX a \PIPE\NETLOGON (should be a SMBopen)
and then send a LSA_REQ_CHAL down it.

client.c :

send to \PIPE\ not \PIPE\NETLOGON.

ipc.c :

fstring name not being bzero'd caused problems when calling named_pipe().

Makefile client.c :

adding start of undocumented options to do NT domain logons, client-side.
starting with LSA_REQCHAL.

the code here happily crashes smbd: i'll investigate this further... :-)

smbparse.c pipeutil.c lsaparse.c :

moved some of the common make_xxxx() functions out of pipeutil.c
so that the make_xxxx and (smb/lsa)_io_xxxx functions now sit
together.  makes sense, really...

added a make_q_req_chal() function.

restructured make_rpc_reply() and called it make_rpc_hdr().  created
functions create_rpc_reply() and create_rpc_response().

pipenetlog.c pipentlsa.c pipesrvsvc.c

calling new create_rpc_reply() function instead of old make_rpc_reply().

proto.h :

usual.

smb.h:

added enum for RPC_PACKET_TYPE

Makefile client.c clientutil.c clitar.c nmbsync.c :

redid the split that i did a year ago, taking the functions in client.c
out into clientutil.c.  guess what?  we could now do encrypted password
NetServerEnum2 calls in nmbd, if we wanted to.

i can now use cli_call_api() to send to different pipes.  i hope.

pipenetlog.c:

allow adding to users group _and_ to admin group.  if adding to
guest group, don't allow adding to users or admin as well.

smb.h :

added some pipe #defines (\PIPE\NETLOGON \PIPE\srvsvc ...)

proto.h :

usual.

loadparm.c :

added "domain other sids" parameter

pipenetlog.c :

using "domain other sids" parameter in SAM Logon response.
using new name_to_rid() function for r_uid and r_gid.

pipentlsa.c :

minor mods to do with new name_to_rid() function.

pipesrvsvc.c :

in the "net share enum" response, allocate some more space for the buffer.
there can be only 32 share entries in the response anyway.  this needs
to be dealt with.

pipeutil.c :

modified name_to_rid() function to use new parameters "domain admin users"
and "domain guest users", but will otherwise do unix uid + 1000.

moved make_dom_gids() here.

proto.h:

the usual.

smb.h smbparse.c :

renamed sid_no to sid_rev_num in DOM_SID, and gid to r_gid in DOM_GID.

util.c :

moved make_dom_gids() from here.
created char *unistrn2(uint16* uni_buffer, int max_len)

shared memory code cleanups (partly preparing for a possible sysV
shared memory implementation)

Implemented asynchronous DNS lookups in nmbd.

I realised this afternoon just how easy it is to add this, so I
thought I'd implement it while the idea was fresh.

nmbd forks at startup and uses a pipe to talk to its child. The child
does the DNS lookups and the file descriptor of the child is added to
the main select loop.

While I was doing this I discovered a bug in nmbd that explains why
the dns proxy option has been so expensive. The DNS cache entries in
the WINS list were never being checked, which means we always did a
DNS lookup even if we have done it before and it is in cache. I'm sure
this used to work (I tested the DNS cache when I added it) so someone
broke it :-(

Anyway, the async DNS gets rid of the problem completely. I'll commit
just the fix to the DNS cache bug to the 1.9.17 tree.

You can disable async DNS by adding -DSYNC_DNS to the compile flags.

updated tel no and email address for my entry.

don't send any statistics in the node status reply

preparing for release of 1.9.18alpha3

WHATSNEW.txt: Updated for alpha3.
server.c: int16 -> uint16 fix for port comparisons in oplock code.
          Needed for Solaris.
version.h: Updated to alpha3.
Jeremy (jallison@whistle.com)

The ubi_* modules are under LGPL, not standard GPL.  This README explains
that.

preparing for release of 1.9.18alpha2

WHATSNEW.txt: Preparing for 1.9.18alpha2 release.
smb.conf.5: Added 'bind interfaces only' parameter.
version.h: Updated to 1.9.18alpha2.
Jeremy (jallison@whistle.com)

chgpasswd.c:
includes.h: SCO changes.
server.c: Added code in open_sockets to allow 'bind interfaces only' to
          work as documented.
Jeremy (jallison@whistle.com)

loadparm.c :

added "domain admin users" parameter
added "domain guest users" parameter

these two complement the "domain groups" parameter.  the "domain groups"
parameter should be for your own groups, and well-known aliases.

util.c :

added ability to do "domain groups = power_users admin_users backup_ops"
which are well-known RID aliases, not well-known RID groups.

pipenetlog.c :

combine the "domain admin users"; "domain guest users" and "domain groups"
parameters to give an array of RID groups to include in the SAM Logon
response.

ipc.c smb.h :

moved REALLOC() into smb.h
added RID #defines.

proto.h:

usual.

damn, I spelt EACCES wrong.

actually it's wrong in the linux man page for utime ...

check for EPERM or EACCESS in file_utime()

don't check lp_alternate_permissions() in the new utime workaround
code. lp_alternate_permissions() is intended only to affect the
display of file permissions, not what you can actually do with the
files.

fix the order of become_uid() and become_gid() in become_root(). This
was a harmless bug but left log entries

code cleanup in reply_lanman2()

Updating an entry.Paul

add a note on how to map usernames with spaces in them

added a note about which netbios name is use in server level security

prototype updates

some cleanups in the clientutil.c code.

I'm actually hoping to remove this module completely once we convert
nmbd to using the new (much cleaner) clientgen.c

smbclient now also reports if it connects as a guest user

rewrote the password server code using the new clientgen.c client
interface

The new code uses a source netbios name equal to the Samba servers
name, not the client name. It also uses NetWkstaUserLogon to do a full
network logon. This means it will honour the servers logon
restrictions (such as login times etc).

This is a set of generic SMB client routines. I needed this in a hurry
to fix the password server code, so I didn't use SMBlib. This code is
fairly generic and uses a "struct cli_state" to hold the client
state.

get rid of the KEEP_PASSWORD_SERVER_OPEN define

add some debug info

Adding make_smbcodepage.1 man page.
Updated smb.conf with dos filetimes.
Jeremy (jallison@whistle.com)

loadparm.c: Changed 'interfaces only' parameter to 'bind interfaces only'. Added
            'dos filetimes' parameter for UTIME fix.
locking_shm.c: Fixed typo (sorry Andrew :-).
namepacket.c: Changed lp_interfaces_only() to lp_bind_interfaces_only().
proto.h: The usual.
reply.c: Made filetime calls use new file_utime call (wrapper for sys_utime).
server.c: Made filetime calls use new file_utime call (wrapper for sys_utime).
system.c: Added Andrew's sanity checks to times in sys_utime().
time.c: Moved set_filetime() to server.c. Made null_mtime() global.
trans2.c: Made filetime calls use new file_utime call (wrapper for sys_utime).
Jeremy (jallison@whistle.com)

added "domain groups" parameter, allowing you to specify the groups that
the user belongs to.

it would be nice to know exactly what the domain groups _are_....

added description of parameters:

domain sid
domain groups
logon drive
logon path

added copyright to headers.

NTDOMAIN.txt:

describes how to set up samba as an NT PDC.  it includes debugging
assisting info.  it is expected that this document turn into a
user's document, rather than a debugger's document.

cifsntdomain.txt :

this is the "NT Domain Authentication (draft)" white paper, current
version.  if anyone think it's a bad idea to put a copy of this in
here, i'm quite happy to remove it.

Added DNS Proxy, Printing, [netlogon] share modes = no.

updated briefly to mention NT domain support as working, but experimental.

removed home_share from user_info structure.

util.c password.c :

added automount_server() function which, if -DAUTOMOUNT is in use,
returns the server name of the NIS auto.map entry.  otherwise,
it returns local_server.

added use of automount_server() for a new substitution %N for NIS
home server.  this defaults, via automount_server(), to the same
functionality as %L if -DAUTOMOUNT is not used.

removed vuser->home_share.  moved code that grabbed the servername
into the separate function automount_server().

loadparm.c :

created "logon drive" (default of "")
created "logon home"  (default of "\\%N\%U")
changed default of "logon path" from NULL to "\\%N\%U\profile".

ipc.c pipenetlog.c :

use lp_logon_drive(), lp_logon_home() and lp_logon_path() in their
now easier-to-use form (don't have to check if *lp_logon_path() and
manually substitute a default of \\%L\%U and do a standard_sub_basic()
on the result, because the default automatically does this.

Fixed Upper/Lower case typos.

a major share modes reorganisation.

The shares modes code is now split into separate files. The shared
memory implementation is in locking_shm.c. The slow implementation is
in locking_slow.c

It is all controlled by a struct share_ops structure that has function
pointers to the implementation of all the functions needed by a share
modes implementation. An initialisation function sets up this
structure. This will make adding new implementations easy and clean.

This also allowed me to get rid of the ugly code in smbstatus. Now
status.c links to the locking code and calls methods in share_ops.

I also renamed some things and generally organised things in a much
cleaner fashion. Defines and structures specific to each
implementation have been moved to the appropriate file and out of
smb.h.

casting cleanups

I am removing these from the source code in preparation for an
upcoming 1.9.18alpha release. We can't release with this code in there
as it currently stands as it might breach ITAR export restrictions in
the US.

I've discussed ways around this with Jeremy and we can put the code
back in with appropriate modifications once it is needed.

Note that this code isn't actually used in Samba yet, so removing it
has no effect.

If anyone wants the code then look in ~samba-bugs/arcfour/ or grab it
from the CVS attic.

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.