a simple test program I use to test the debug message system

the first cut of the internal messaging system.

The motivation for this system is to replace the UDP message for
oplocks, but this commit only does the "set debug level" message.

Implemented DELETEPRINTERDATA (tested with Gerald's Win32 progs).
Jeremy.

Fixed up get_a_printer_driver_3 so it looks in the tdb using the correct
key name for a driver info. Version needs to be adjusted in the same
way that it is adjusted when the driver info is stored in the tdb.
AddPrinterConnection() Win32 call caught this one.
Jeremy.

Fixed typo where docname was being checked for datatype - was causing
STARTDOCPRINTER call to fail. I *love* Gerald's test code :-).
Jeremy.

Win32 API is explicit that EnumPrinterData returns ERROR_MORE_DATA not
ERROR_INSUFICIENT_BUFFER when working out what space is needed. This fix
gives us the same return that WinNT does.
Jeremy.

Fixed the printerdata code to return NO_MORE_ITEMS if a size is requested
and the parameter index doesn't exist. I think this code can be simplified
considerably, but JF needs to check to be sure.
Jeremy.

Added code to return NO_MORE_ENTRIES when trying to determine
size using "NT mega hack". I think this is the correct thing to
do but JF should also examine the.
Jeremy.

Hopefully this should fix the primary group permission problem.

Temporary fix to convert from level6 (Win2k) to level 3 (NT4)
for driver download files.
Jeremy.

Fix from John Reilly for double free of printer struct.
Jeremy.

Fix for the SID history problem when using a Win2k domain controller
with security=domain. Also fixed to dynamically allocate the SIDs and GIDs.
Jeremy.

Implemented GETFORM tested and working using Gerald's Win32 test progs....
Jeremy.

Changes from APPLIANCE_HEAD (per Tim Potter):
- make proto
- addition of function to convert from errno values to NT status codes
  (source/lib/error.c)
- purge queue done without full access permission will purge only the
  jobs owned by that user, rather than failing.
- unlock job database tdb before sending job to printer
- in print_job_start(), ensure that we don't pick a jobid with an existing
  temporary file that may be owned by another user, as it causes silent
  failures.
- fixes for printer permission checking for NT5 clients
  (source/include/rpc_spoolss.h, source/printing/nt_printing.c,
   source/printing/printing.c, source/rpc_server/srv_spoolss_nt.c)
- change from uint8 to 'enum SID_NAME_USE' (source/rpc_server/srv_lsa.c)
- fixed memory leaks for win95 driver download process
  (source/smbd/lanman.c)
- properly free prs_structs and dacl in testsuite/printing/psec.c

Fix from John Reilly @ HP for my typo with the parameters being reversed. Ooops.
Jeremy.

-U% should be an anonymous connection.  Fixed bug where
the password was being set to "" instead of NULL.

..and yes Elrond, I'll merge this into TNG.

P
jerry

Fix for swat to return correct MIME type for text files.  From "Ron
Alexander" <rcalex@home.com>

Implemented DELETEFORM tested using Gerald's Win32 test code :-).
Jeremy.

make sure a couple of variables are initialised

fixed "admin users" option with new security code

Removed compiler warning.

Fixed overrun write error when using debuglevel > 50.
Jeremy.

Fixed error returns for moving printer driver files around so generic
"Access denied" isn't always returned. More fixes found using Gerald's
wonderful Win32 test progs :-).
Jeremy.

Added vfs_unlink call to ensure vfs is used on unlink from client.
Jeremy.

Fix for bind interfaces only sometimes picking the wrong IP address
for a name query. From Steve Langasek <vorlon@netexpress.net>.
Jeremy.

Implemented AbortPrinter() from Gerald's Win32 test code. Just purge all
possible printjobs from that printer (I think this is correct).
Added error code returns for print_queue_XXX() functions.
Jeremy.

missed removing the static declaration of strwicmp()

jerry

make proto

added -P (for no prompt) and -A <authfile> options

jerry

added a #define for SPOOLSSGETPRINTERDATAEX.  Although we do not
support the RPC yte (win2k specific), I added it for documentation
purposes.

jerry

fixed previous bug spotted by Elrond in get_printerdrivernumber().
Thought I had already committed this.

jerry

added -A <authfile> option to smbclient for the reading of username /
password credentials from a file.

--jerry

needed to use strwicmp() in smbclient code, so I moved it to util_str.c
and made it non-static

--jerry

make rpcclent by default.

--jerry

added the -A <authfile> option for allowing the reading of user
name and password credentials from a file.  Designed for use
when needed to passwd credentials between scripts

--jerry

first draft at rpcclient man page for HEAD branch.  Taken heavily
from the man page in TNG.  Updated for newer spoolss commands.

jerry

This doesn't compile but is a placeholder for moving the UNIX
acl code over to POSIX acls.
Jeremy.

smbd/password.c: Fixed typo in Tim's new code that caused insure overrun error.
smbd/reply.c: Fixed lowercasing UNIX character set problem.
Jeremy.

added a little description of useage and output format since this gives
more info than just what parameters are not supported by SWAT.

fixed typos spotted by Ron Alexander <rcalex@home.com>

--jerry

script written by herb for reporting which smb.conf parameters
are in loarparm.c but not supported by SWAT.  I just thought it looked fun.
:-)

--jerry

Merge bug - still getting used to dirdiff.

Oops - missed a file.

Merge from appliance branch.

yipee!

The spoolss AddJob function has an [in,out] buffer not an [in] buffer
(despite the comment in the code to the contrary). Also, we must fail
this function - not just blindly reply "no problem" as AddJob should
always fail on non-local printers.

This fixes a bug where the "print test page" failed about half the
time. I suspect it will also fix a bunch of other intermittent spoolss
bugs where the client (incorrectly) tries to use the AddJob printing
interface.

made reopen_logs() always re-open logs, not try and be smart about not
re-opening in some circumstances. This fixes a problem where a HUP
does not re-open logs and leaves the log open on a unlinked file.

arrgh.

whoever added "auto" parameters in loadparam only supported "True", "False"
and "Auto", and did not support "Yes" and "No" like booleans. This
means anyone using "preferred master = yes" didn't get what they
expected!

vfs write data fix. Fix for name matching in addtosmbpass
Jeremy.

Replaced "\\MAILSLOT\\NET\\NETLOGON" with NET_LOGON_MAILSLOT constant.

- fixed the %U macro so that the old (and documented) semantics work
again. This got broken with one of the substitute.c updates a couple
of months ago.

- also fixed %u to return the username from the current_user structure
  when called via a method that does not have direct access to the
  username. I cache the uidtoname() result to prevent thrashing nss.

Updated file with version in TNG.

Modified to use sid_to_uid and sid_to_gid. Now ready for POSIX ACL
code....
Jeremy.

Tidied up some error returns from printing calls. Still need to map UNIX
errors to NT error for print job failure returns. Patch from John Reilly at HP.
Jeremy.

Documentation on NT_USER_TOKEN. Requested by Sander.
Jeremy.

Added code to do SID to uid/gid conversion. Needed for ACL support.
Jeremy.

Patch from John Reilly <jreilly@hp.com> for not processing duplicate files
when moving printer driver to download area.

Fix for safe_free freeing static strings. From Robert August Vincent, II.
Jeremy.

oops.  must return "" string and length zero when strlen(filebuf) == 0

fixed an off by one error where I was tromping on memory
when initializing a driver_info_3 struct.  See spoolss_addprinterdriver()

jerry

fixed initialization errors (again!)  I must have forgot to
check these in before.

jerry

getfileline() - line with length of zero -> filebuf[strlen(filebuf)-1]
is NOT ok.

changing some debug message levels

j-

Fix for Win9x driver downloads when information is
stored in the TDB.  See the message I posted about
this on samba-technical

* If the admin does not desire the new NT print driver
support, nothing needs to be done.  All existing
parameters work the same.

* If the admin wants to take advantage of NT printer
driver support but does not want to migrate the 9x drivers
to the new setup, the leave the existing printers.def
file.  When smbd attempts to locate a 9x driver for the
printer in the TDB and fails it will drop down to
using the printers.def (and all associated parameters).
The make_printerdef tool will also remain for backwards
compatibility but will be moved to the "this tool is the
old way of doing it" pile.

* If the admin installs a 9x driver for a printer on
a Samba host (in the printing TDB), this information will
take precedence and the three old printing parameters
will be ignored (including print driver location).

* If an admin wants to migrate the printers.def file into
the new setup, I can write a script to do this using
rpcclient.  The main problem is the driver namespace
(we need the NT driver name first in order to
be consistent).

jerry

added comment for my own memory purposes

jerry

hack for unmarshalling unistr's on machines with BIG_ENDIAN arch
I think we have a lot more of these bugs lurking (i'm fairly
confident of it).

jerry

- fixed memcpy bug in copy_unistr2()
- init_unistr2_from_unistr() does not NULL terminate
  the buffer

--jerry

fixed unitialized struct bug in enum_users (whatever it is called)

jerry

fixed typo.

jerry

fixed error in [public] spotted by Ron Alexander <rcalex@home.com>
changed to read only = yes

jerry

Merge ... uninitialised variable fix from APPLIANCE_HEAD.

Tidied up extra ()'s.
Jeremy

Reverted the change Luke made. Removed IS_BITS_SET_XX macros.
Do not re-add them. These macros are unsafe as they are not understood.
Change all TNG code using them to correct '&' and '|' please.
IS_BITS_SET_ALL was being used in cmd_interp.c when IS_BITS_SET_SOME
should have been used.
Jeremy.

Merge.

fixed a few bugs in new_smb_io_relarraystr().
Marshalling code path now uses spoolss_smb_io_unistr()
also caught a NULL pointer case.

i'll update TNG later.

NT printer permissions test program.

restored IS_BITS_SET_xxx macros.

hey, jerry: turns out you _have_ converted to get_safe_nt_errmsg!!!
[got confused with which way dirdiff was going :)].

... but you forgot to update TNG's lib/cmd_interp.c...

jeremy,

please do not remove IS_BITS_SET_XXX macros just because you happen
"not to like them".  whilst still working on merging, you are not
responsible for this code, therefore do not hinder its development
whilst people are attempting to do merges.  after the merge from
TNG to cvs main is completed, you can do what the hell you like.

thank you.

jerry,

please could you consider adding underlying routines (in this case
get_safe_smb_errstr()) instead of reverting to a [less secure]
previous coding method.  thanks.

minor formatting fixes.

jerry

fix for new_smb_io_relarraystr() and new_smb_io_relstr()
to use spool_smb_io_unistr() as this does not call prs_align()
befrore parsing the UNISTR.

Parsing a void* buffer from an RPC should not be aligned
on 4 byte boundaries.  Don't think this change affects
any marshalling code, only unmarshalling...

jerry

removed the prs_align in spoolss_smb_io_unistr()  (JF, please check)
as it was throwing things off when parsing buffers passed in SPOOLSS
RPCs

fixed the unmarshalling of UNISTR arrays to be double NULL terminated

jerry

fixes to prs_unistr UNMARSHALLING only.  Problem was due to talloc's
of 0 bytes.

jerry

FIxed some unmarshalling problems with relstr and unistr arrays

jerry

fixed unistr_to_ascii to deal with NULL src strings

jerry

reworked talloc() to in order to help track down invalid
memory access.

Tridge, I don't think using 0xdeadbeef for size allocations
of 0 is going to work.  I ended up having to use NULL
as much code that works on UNISTR checks to see if the buffer ptr
is NULL.  So valid code ends up with a seg fault.

Rather than rewriting it all, I added a DEBUG_TALLOC
#ifdef in talloc.h that sets a macro BAD_PTR.
This is the value assigned to ptr for an allocation of 0 bytes.

jerry

removed (void) typecast from SMB_ASSERT

jerry

Updated by Patrick Powell <papowell@lprng.org>

Fixed the problem with UNISTR marshalling in a buffer5 struct.
The smb_io_unistr() code called a prs_align, this was not being
counted in the size or taken account of in the offset calculation.
Fixed size_ calculation to always return a size a multiple of 4
and also set the offset correctly. This fixes the problem I saw
and will hopefully fix the problem HP reported. JF please check
this change.
Jeremy.

First shot at actually *doing* WINS failover.
If libsmb/namequery.c:name_query() times out while doing a non-broadcast
query, I mark that WINS server 'dead'.  Note that I don't try the new
WINS server.  I think I can get that working too.

This is only for queries, not registrations.  The biggest problem is that
I may have to fiddle with the UNICAST SUBNET, but I need to check talk
that over with someone (Jeremy?) before I hack at it.

I can't actually test the above change, I'm 'fraid.  I'm getting:

4 errors detected in the compilation of "rpc_server/srv_spoolss_nt.c".

in head branch.

Chris -)-----

Fix case where volumename could be returned as UNIX charset or DOS codepage
depending on where it came from.
Jeremy.

Added "add_printer_hook" call to update printer for Win9x clients.
Patch from John Reilly <jreilly@hp.com>.
Jeremy.

Tidied up security rights definitions.
Jeremy.

Removed requirement that sid have an owner before being interpreted.
Thanks to Elrond for pointing this out.
Jeremy.

Working on spoolss_getprinterdriver() as it does not display all the
dependentfiles.

jery

working on the unmarshalling of dependentfiles to a DRIVER_INFO_3
struct.  Not wuite there.

jerry

code formatting cleanup

jerry

deal with allocation size of 0 in prs_unistr when UNMARSHALLING

jerry

Deal will NULL UNISTR in unistr_to_ascii

jerry

talloc returns 0xdeadbeef when asked to allocate 0 bytes

jerry

Needed to add the SHARED attribute bit when creating a printer
on a Samba host.  Also needed to add an option to pass the share
name (printer name) on the command line.

And fixed the checking of the return code for spoolss_r_addprinterex()

jerry