new prototypes

the bulk of the changes to get rid of fd_ptr and move print open
handling to printing/printing.c

most of this was just replacing things like fsp->fd_ptr->fd with fsp->fd

the changes in open.c are quite dramatic. Most of it is removing all
the functions that handled the fd multiplexing

rather than doing print file open processing in open.c we now handle
it in print_open_file()

that removes a lot of special cases in open.c and makes the print
handling code much easier to understand.

there is still lots to do in printing.c, but this at least gets
printing separated from the mainline code

the first of a bunch of changes to code with getting rid of the fd_ptr
element in the fsp

pretty mechanical stuff, but it affects lots of files.

got rid of the file_fd_struct structure completely.

With the new br_lock() code we no longer need the fd multiplexing,
which is great because it was really horrible :)

Moved the dev, inode and delete_on_close elements into the fsp.

A nice side effect is that this has greatly simplified open.c

removed the read prediction code from the head branch. I think the
idea is fundamentally flawed anyway and it has been disabled for a
long time.

split out the lpq parsing code into a separate file
printing/lpq_parse.c

getting ready for the new printing backend

don't echo warning messages twice

utmp compile fix from peter@cadcamlab.org

Modified interfaces and added checks around *all* *alloc calls
so that errors are returned on memory allocation failure.
Jeremy.

filled the architecture table (useless for the moment).
fixed GetJob.

J.F.

fsync() returns an int rather than void.

Added *.so

Improved clean target.

JF - you might want to look at this patch.

Went through and fixed up all failure possibilities. If something (malloc,
file open etc.) fails, then this code goes through and cleans up all allocated
resources on exit. This is the equivalent of a C++ destuctor for the allocated
structures. We need code to be written like this to pass purify tests.

Jeremy.

off by one error ...

J.F.

changed all the status code to ERROR_xxx instead of NT_STATUS_xx which are
wrong in the spoolss case.

fxed a bug in the job notify code (that's the polite answer), the truth is
different: there is a bug in the NT spooler service, including SP6a and
NT2K.

changed the default lpcommand in the LPRNG case.

J.F.

Added strerror at debug level 0 on file create/open fail. Helps catch
problems.
Jeremy.

Change to vfs API. POSIX states fsync should return an int, not a void.
Jeremy.

Removed strange optimisation (paranoia fix maybe ?) that stopped smbd
from returning '.' and '..' in a top level listing of a trans2 directory
scan. NT does return these entries.
Jeremy.

Snuck in some whitespace cleanup while I was visiting these files.  (-:

Removed unused parameter vuid from rpc_server api_* calls.  For the very few
functions that need to access the vuid, it can be obtained from the
current_user global.

Did some whitespace cleanup.

Fix for compiler warning about invalid structure type.
jeremy.

Hmmm - unicode codepages need adding to HEAD. Thought that'd been done ?
Jeremy.

Fix for spoolss code from Alan Hourihane <Alan.Hourihane@pinacl.co.uk>.
Jeremy.

Additional fix for nmbd and 127.0.0.1 interface.
Jeremy.

fixes for broken IRIX header file merged from 2.0 branch

IRIX include fixes.
Jeremy.

samba-bugs -> samba.
Jeremy.

Fix for coredump in testparm.
Jeremy.

Fixed spelling of Andrew's name.  (-:

I finally got sick of configure being run automaticaly when I run make
clean or in other situations. That then screws up options I pass to
configure and is generally annoying.

Instead the Makefile now emits a warning if configure needs
rerunning, but doesn't actually run it.

don't re-enable the auto running of configure on pain of death :)

Added check for LL suffix to long long ints needed by AIX 4.3.x compiler
to allow successful build.
Jeremy.

Fixed bug found by JF where if the amount of data to return in the prs_struct
was small enough to pass the SMBtrans max data test we weren't setting the
"more data needed" error. I was fogetting the RPC_HEADER_LEN.... I now
check the correct length (current pdu staging area length). More printer
spool enum stuff works now. This bug does not affect TNG.
Jeremy.

Cause nmbd to ignore loopback interface when constructing its interface                                                         list. This is done by default when interfaces are probed, but if someone
explicitly adds 127.0.0.1 to the interfaces line for smbd, then nmbd would
start to try and register names on it. This was not good :-(.
Jeremy.

split addprinterex in preparation for level 1 support and to stop it
coredump.

J.F.

rewrote getprinterdriver level 3, now correctly handle the dependent
files.
A number of memleak fixed
some error return values fixed.

J.F.

More Japanese filename fixes wrt VFS code from
Tomoki AONO <aono@cc.osaka-kyoiku.ac.jp>

damn, the test was the wrong way around for short_resuest

Win2k will only accept volume labels in UNICODE. Fixed.... :-(.
Jeremy.

Moved common code to set parameter offset field correctly oout of 'if'
statement into main function code path.
Jeremy.

Fixed crash bug in new handle_netbios_name code.
Jeremy.

changed the definition of dos_PutUniCode

the previous definition could result is us overflowing a buffer. The
null termination was always added yet the size returned did not
include the null termination.

the new function takes a BOOL null_terminate, and always returns the
total number of bytes consumed by the string.

the final part of the nmbd merge between head and tng - this gets the
GETDC stuff sorted out

fixed the hanlding of recursion desired when sending packets from
nmbd. It does NOT follow the rule that packets to the wins server have
rec_des set (I know that we have postulated this in the past). Please
don't screw with this unless you do careful testing from WinXX
clients.

Luke and I spent a couple of hours today getting this right (it was
wrong in both TNG and HEAD)

moved nmblib-specific code from util.c to nmblib.c.

rename static reload_services function to reload_nmbd_services because
it clashes with other reload_services definitions.

when doing a "secure nbns" wack response and check with owner for a
multihomed registration we were sending the packet to the wrong host!
We sent it to the person trying to register rather than to the person
who currently holds the name. That means we were not secure and we
could allow two people to register the same name.

Jeremy, you may wish to merge this change into 2.0.X

merge from tng ...

reload services can be static

load smb.conf and interfaces in masktest (to allow for netbios name
resolution)

rpc_parse/parse_spoolss.c: Added checks on mallocs/overflow checks on all prs_xx
calls.
smbd/connection.c: Fix from David Lee <T.D.Lee@durham.ac.uk>.
Jeremy.

Correctly convert from little-endian UNICODE to dos_codepage when doing
secure file create.
Jeremy.

acconfig.h configure configure.in: Added check for UT_SYSLEN for utmp code.
include/byteorder.h: Added alignment macros.
include/nameserv.h: Added defines for msg_type field options - from rfc1002.
lib/time.c: Typo fix.
lib/util_unistr.c: Updates from UNICODE branch.
printing/nt_printing.c: bzero -> memset.
smbd/connection.c: Added check for UT_SYSLEN for utmp code.

Other fixes : Rollback of unapproved commit from Luke.
Please *ask* next time before doing large changes to HEAD.

Jeremy.

disabled the getgrent() code recently added that returns all unix groups
as NT aliases.  no allowance is made for possible clashes with some
unix groups that may be mapped to NT groups, not NT aliases.

it is not the place of this code to make decisions about which unix
groups should be mapped to NT aliases or NT groups, or in fact whether
unix groups exist at all (e.g "appliance" mode).

please could the person who wrote this code contact me and i will outline
the architecture and design issues that you need to be aware of when
working in this area: i need your skills and ability -- applied in the
right places.

thank you.

disabled code from HP that replaced get_sampwd_entries() with
get_passwd_entries().  get_sampwd_entries() calls the generic
SAM pwdb API, whereas get_passwd_entries() calls the Unix database
explicitly.

this will result in LDAP, MYSQL, NISPLUS and any other future modules
(e.g "appliance" mode SAM database) failing to operate correctly.

i know what problem was attempted to be fixed, here, but this is
not the right place to attempt to fix it.

added the following message to all dce/rpc client/server code, except
the spoolss code (it's cut from TNG) and the smb-dce/rpc interface
code that jeremy has been working up to TNG-functionality.

i also want this message to go into SAMBA_2_0 and SAMBA_2_0_RELEASE,
because it is intolerable that potentially good modifications be made
to code that is going to be thrown away, and people waste their time
fixing bugs and adding enhancements that have already been carried
out already, up to two years ago in the TNG branch.

/*
 * THIS CODE IS OUT-OF-DATE BY TWO YEARS, IS LEGACY DESIGN AND VERY, VERY,
 * INCOMPLETE.  PLEASE DO NOT MAKE ANY FURTHER ENHANCEMENTS TO THIS CODE
 * UNLESS THEY ARE ALSO CARRIED OUT IN THE SAMBA_TNG BRANCH.
 *
 * PLEASE DO NOT TREAT THIS CODE AS AUTHORITATIVE IN *ANY* WAY.
 *
 * REPEAT, PLEASE DO NOT MAKE ANY MODIFICATIONS TO THIS CODE WITHOUT
 * FIRST CHECKING THE EQUIVALENT MODULE IN SAMBA_TNG, UPDATING THAT
 * FIRST, *THEN* CONSIDER MAKING THE SAME MODIFICATION IN THIS BRANCH
 *
 * YOU WILL, ALMOST GUARANTEED, FIND THAT THE BUG-FIX OR ENHANCEMENT THAT
 * YOU THINK IS NECESSARY, HAS ALREADY BEEN IMPLEMENTED IN SAMBA_TNG.
 * IF IT HAS NOT, YOUR BUG-FIX OR ENHANCEMENT *MUST* GO INTO SAMBA_TNG
 * AS THE SAMBA_TNG CODE WILL REPLACE THIS MODULE WITHOUT REFERENCE TO
 * ANYTHING IN IT, WITH THE POSSIBLE RISK THAT THE BUG-FIX OR ENHANCEMENT
 * MAY BE LOST.
 *
 * PLEASE OBSERVE AND RESPECT THIS SIMPLE REQUEST.
 *
 * THANK YOU.
 *
 * lkcl@samba.org
 */

indent update to make t easier to see setuid mods in TNG.  some
code from these modules i had to leave out (nothing to do withj setuid)

Added preliminary user list code from HP. Needs to be made dynamic
and backended with a tdb database so the list can be shared between
multiple smbds. ToDO.

Jermey.

Fix from christoph.pfisterer@rwg.de for large directory listing to OS/2
server.
Jeremy.

Fixes to add "paranoid" option to popen. Checks some basic things.
Jeremy

removed extra arguements from make_nmb_name calls

fix a bug in enumprinterdrivers

J.F.

use parameters to minimize changes in samba for irix release version

use proper parameter names instead of alias names
clear lppause and lpresume commands since they don't exist on IRIX
change spool directory

Fix from NAKAJI Hiroyuki for a trailing '\n\ in asctime.
Jeremy.

Fixed parsing issue with HP printer defs reported by HP.
Jeremy.

Adding libtool support to HEAD branch.  We're going to need this for
various things yet to be merged from TNG.  The smbwrapper stuff should
probably be ported across too.

include/smb.h:
smbd/negprot.c:
smbd/reply.c: Fixes to recognise Win2k.
param/loadparm.c: Put debug timestamp parameter back to correct default.
smbd/nttrans.c: Fix to detect Win2k unicode bug with transact create.
Jeremy.

oops ! forgot smb.h in last commit
added info level 1 parsing code for addprinter(ex)

J.F.

Cleaning up call_trans2getdfsreferral.
Instead of a #ifdef MS_DFS in call_trans2getdfsreferral()... , now using a generic call leading to some #define problems.. hopefully I've corrected the last of them.

Cleaned up call_trans2getdfsreferral for when MS_DFS is not defined.

small fix for addprinterex at level 2

I have some network dumps at level 1, but quoting MSDN: "Level [in]
Specifies the version of the structure to which pPrinter points. This
value must be 2."

I like NT so much :-)

J.F.

Interim fix for LPRng. If a lpq parsed line start with a space, we skip
it, to prevent false status return value.

Jeremy, it's a gross hack but you should consider it for 2.0.7 also.

J.F.

parse correctly getprinterdriver2
found a stupid bug in enumprinters
fixed some memleaks
found a coredump in enumprinterdata
getprinterdriverdir responds correctly now.

J.F.

moved #endif to end of function to compile when dfs is disabled.

J.F.

Added standard shell command thing using '!' as first character.

Fix stupid logic bug in detecting start-of-pdu in writeX on pipe.
Found by JF.
Jeremy.

You *must* use O_EXCL when using mktemp (security issue).
Glad this code never shipped :-).
This is not a problem in 2.0.x.
Jeremy.

Cleaned up unused variables, returns from non-void functions etc.

Cleaning warnings from configure.developer

Cleaning up the warnings from configure.developer.
Found that Jeremy had already made some of the changes.

Fix for names with spaces and some I18N filename issues from
V. Fyodorov <V.Fyodorov@VAZ.RU>.
Jeremy.

Fixups for compiles with gcc flags -Wall -Wshadow -Wstrict-prototypes -Wpointer-arith -Wcast-qual
Partially implemented rpc daemon redirect (needs more work).
Jeremy.

you know, when you do a cvs commit, you _really_ expect it to actually work.
this explains why j-f wasn't happy.

getprinter level 0: was to short, found most of the fields, undocumented,
undecoded, nothing in MSDN, but now it works :-)

cleanup of error codes.
fixed some dfs declarations function.

J.F.

removed unused variable

J.F.

restore the spoolss pipe as Jeremy replaced srv_pipe with the one from
SAMBA_2_0 :-)

J.F.

indent went wrong.  grr.

computer too slow to compile with debug info (100mins), but i need all the
warnings.

Not used. Functionality folded back into rpc_server/srv_pipe.c (where it
originally came from).
Jeremy.

Big update moving the multi-pdu support from 2.0.x into HEAD for JF
and the printer functions.
Also tidied up some header includes and got the order right so you
can now do a :

make proto
make clean
make

Jeremy.

Fixups for Win2K security descriptors from the 2.0.x branch.
Jeremy.

Corrected DEBUG messages.

dded Microsoft Dfs services.
* added a new msdfs/ directory under source/
* added msdfs sources under this directory.
* modified configure setup to add a --with-msdfs configure time option

 Modified Files:
  Makefile.in acconfig.h configure configure.in
  include/config.h.in include/includes.h include/proto.h
  include/smb.h include/smb_macros.h param/loadparm.c
  smbd/negprot.c smbd/nttrans.c smbd/process.c smbd/reply.c
  smbd/server.c smbd/trans2.c
 Added Files:
  include/msdfs.h msdfs/README msdfs/msdfs.c msdfs/msdfs_tdb.c
  msdfs/parse_dfs_map.c
 ----------------------------------------------------------------------

added a tcon test: make two tconXs with the same user/pass, open a file
with one of them and try to write to the file with the other.

Enumprinters level 1: reply *exactly* like an NT server
BTW, found a little memleak in it.

J.F.

fixed enumprinterdata.

J.F.

changed prs_unistr to parse empty and non-empty strings the same way.
fixed typo in SPOOLSS_SYNT
some cleanup of unused functions
wrote make_spoolss_enumprinter and make_spoolss_openprinterex for
rpcclient as I'm trying to keep in sync the parsing code between HEAD and
TNG.

Will commit changes to TNG after lunch.

J.F.

Tidied up fchown code, error reporting.
Jeremy.

Correctly full buffer smbpasswd i/o streams.
Jeremy.