This commit was manufactured by cvs2svn to create tag
'SAMBA_TNG_2_5_GOOD'.

preparing for release of alpha.2.5

merge.

sizeof(fstring) needed, not sizeof(arg).

memory over-read.  insure strikes again.

more insure stuff.

insure is cool.

replaced Memcpy with memcpy_zero.

back to square 1 (again) - rpcclient now works.

count wrong in RNetShareEnum

more merge

merge

This commit was manufactured by cvs2svn to create branch 'SAMBA_TNG'.

split fsp specific routines out of printing.c to fix linking problem
in TNG

fixed two uninitialised memory references

trick to get full stack trace when using the free version of insure

insure caught an uninitialised memory reference - ensure it starts as
zero

avoided a memory leak in the ubi code by deleting a mangled cache
entry before adding to ensure that we don't ever add a duplicate entry

this code can be removed when ubi gets fixed

fixed a prs memory leak (weren't freeing input buffer)

if using insure then don't close fd 2

Major work on architecture of nss client, winbindd and NT server
communications.  Client state now indexed by file descriptor rather
than pid number closing a possible security hole.

The lsa and samr pipe handles now cached on a per-domain basis which
speeds things up dramatically.  However it has exposed some problems in
the rpc client code.  Use and group enumeration code and types simplified
quite a bit.

Still some work to do in moving yet more state back towards the nss
client.  The eventual goal is to have nss clients unaffected by the stopping
and restarting of the winbind daemon.  Some scalability issues still to be
addressed.

Removed LIBSURS_OBJ from winbind objects.

Major work on architecture of nss client, winbindd and NT server
communications.  Client state now indexed by file descriptor rather
than pid number closing a possible security hole.

The lsa and samr pipe handles now cached on a per-domain basis which
speeds things up dramatically.  However it has exposed some problems in
the rpc client code.  Use and group enumeration code and types simplified
quite a bit.

Still some work to do in moving yet more state back towards the nss
client.  The eventual goal is to have nss clients unaffected by the stopping
and restarting of the winbind daemon.  Some scalability issues still to be
addressed.

From HEAD:
reply.c:
> fixed overlapping strcpy() found by insure
service.c:
> don't copy a null groups list
In TNG this was already fixed by returning NULL from
memdup(NULL)

Directly from HEAD:
installscripts.sh installswat.sh makeyodldocs.sh
smbtar

moved parse_misc.o and parse_rpc.o from RPC_PARSE_OBJ2 to
LIBSMB_OBJ, cause libsmb anyway depended on them.
Now could remove $(RPC_PARSE_OBJ2) from nmbd and smbclient,
and both got 30k smaller.

Split PRINTING_OBJ into PRINTING_OBJ and PRINTBACKEND_OBJ,
like HEAD. Now swat links, but spoolssd still doesn't link.
(Of course, smbd also doesn't compile)

From HEAD:

> printing/printing.c: Convert print_fsp_open to return an allocated fsp.

Now at leat smbd/open.c compiles, but spoolssd and swat
don't link any more, cause printing.c now depends on
smbd/files.c, which in turn depends on whatever.

I guess, swat will not link in HEAD either...

From HEAD:

> nmbd/nmbd_processlogon.c: Fix for NT domain logons causing nmbd to core dump.
I didn't see this in the diff.
>                           Also fix for sidsize DOS bug.
Saw that.

Luke: Can you take a _short_ look at the diff of
nmbd_processlogon.c in TNG and HEAD, this looks like
unicode-stuff.

merge from last 2 days (not compiled).

return NULL for a zero size memdup

don't copy a null groups list

fixed overlapping strcpy() found by insure

fixed uninitialised snum

This is a *big* checkin that may break some things, but implements the
new open mechanism Andrew & I discussed.

config.sub:
configure: Included the QNX patch.

include/vfs.h:
smbd/vfs-wrap.c:
smbd/vfs.c: Added ftruncate vfs call (needed).

Note that we will also need locking calls in the vfs (to be added).

lib/util_unistr.c:
nmbd/nmbd_processlogon.c: Fix for NT domain logons causing nmbd to core dump.
                          Also fix for sidsize DOS bug.

locking/locking.c: Check value of ret before using it for memdup.

printing/printing.c: Convert print_fsp_open to return an allocated fsp.

rpc_server/srv_lsa.c: Fix for NT domain logons.

I have removed all use of lp_share_modes() from the code (although I
left the parameter in the table for backwards compatibility). It no longer makes
sense for this to exist.

smbd/close.c: Removed lp_share_modes().
smbd/fileio.c: Fixed parameters to unlock_share_entry call in panic code.
smbd/files.c: Correctly set the unix_ERR_code to ERRnofids on fsp allocation fail.

smbd/nttrans.c:
smbd/reply.c:
smbd/trans2.c: Changed all occurrences of open_file_shared/open_directory/
               open_file_stat to return an fsp from the call.

smbd/open.c: Changed all occurrences of open_file_shared/open_directory/
             open_file_stat to return an fsp from the call.

In addition I have fixed a long standing race condition in the deny mode
processing w.r.t. two smbd's creating a file. Andrew, please note that
your original idea of using open with O_EXCL in this case would not work
(I went over the races very carefully) and so we must re-check deny modes
*after* the open() call returns. This is because there is a race between
the open with O_EXCL and the lock of the share mode entry. Imagine the
case where the first smbd does the open with O_EXCL and a deny mode of DENY_ALL,
but is pre-empted before it locks the share modes and creates the deny
mode entry for DENY_ALL. A second smbd could then come in with O_RDONLY
and a deny mode of DENY_NONE and the two opens would be allowed.

The *only* way to fix this race is to lock the share modes after the
open and then do the deny mode checks *after* this lock in the case
where the file did not originally exist.

This code will need extensive testing but seems to initially work.

Jeremy.

Made some things static, some of those turned out to be
unused currently, if you need them, make them public again:
lib/msrpc-client.c, rpc_client/cli_connect.c,
rpc_parse/parse_lsa.c, rpcclient/display_dfs.c

merged a little of util_sid.c from HEAD (more cosmetic and
ordering)

dropped in profile.c from HEAD

Split WINBINDD_OBJ into WINBINDD_OBJ1, which containts the
winbindd-specific objects and from which the prototypes are
generated, and WINBINDD_OBJ, which contains WINBINDD_OBJ1
and library-objects.

Did this, cause proto.h just again included all the
prototypes for rpc_client and rpc_parse.
proto.h went down from 6466 lines / 288k
    to 4578 lines / 197k

... hope, Tim isn't too angry, that I touched his area.

More games with string handling.

Code review by tridge.

Modified daemon architecture to manage partial reads and writes of client
data properly.

Also keep client socket file descriptors open for as long as possible.

Moved client state back a level from inside winbindd_{user,group}.c into
winbindd.c

All winbindd_* functions now take a winbindd_state pointer as argument
rather than selected bits of the request structure.

Lots of code moved to winbind_utils.c

String handling cleaned up.

Code review by tridge.

Cleaned up string handling.

Cut out huge swathes of code into winbindd_util.c

Renamed a bunch of types.

Code review by tridge.

Cleaned up string handling.

Moved across group and user state for {set,get,end}{pw,gr}ent() functions.

Added externals for domain name, uid and gid lists.

Code review by tridge.

Moved across common functions from winbindd.c, winbindd_{user,group}.c and
winbindd_surs.c

Code review by tridge.

Moved functions into ntdom.c as they are now only used by the client.

Code review by tridge.

Checked result of all system calls == -1 instead of < 0.

Cleaned up string handling.

Copied across next_token() function from lib/util_string.c and removed call
to strtok() function.

Keep connection to winbindd UNIX domain socket open rather than closing it
after each transaction.

Process pid now not required to be passed to winbindd.

Removed dependency of libnss_ntdom.so to any other Samba objects.

added client-side net_enum_trust_dom_list because am trying to track down
a reliability-bug.

preparing for release of alpha.2.4.2

debugging data-transfer interface (again).

Made some things static, will look later, why protos from
ncacn_np_use.c end up in both proto.h and
rpc_client_proto.h.

new unicode map files (ranges)

oops.

OSF1 compiler can't take #defines with whitespace in front of them
(thanks helmut)

more merge.

use sys_fsusage() not disk_free() in printing.c

more merge

This commit was manufactured by cvs2svn to create branch 'SAMBA_TNG'.

split fsusage() into a separate module (to fix linking problems with
spoolssd in tng)

more merge.

middle of a merge.

patch from luke to split out lanman code from ipc.c into lanman.c

- got rid of the "passive" option
- cleaned up the standard_sub_*() calls a lot

Implemented routine to expand group members that are also groups so the
group list concerned contains only users.

Sync up with 2.0.7 w.r.t. guest users.
Jeremy.

locking/locking.c: Fixed placeholder code for POSIX locking.
printing/printing.c: Cast tdb_delete to (tdb_traverse_func) to stop warning.
                     tmpfile gives mirror warning.
smbd/groupname.c: Remember to file_lines_free() on exit.
tdb/tdb.h: Add tdb_traverse_func typedef.
Jeremy

moved cli_connect_auth() made it static.  more work needed.

made cli_close_socket() static.

BOOL storing an int result.  oops.

more merge/shuffling.

start splitting domain-related functions out of clientgen.c
removed agent redirect code.

tng changes:

- got rid of passive
- go back to head branch select call
- cleanup and simplify pdu send code

updates from the TNG branch

don't parse blank lines

fixed some crash bugs in the nt forms parsing

ReadDirName calls vfs->readdirname calls dos_readdirname.
replaced with readdirname.  ditto for OpenDir and CloseDir.

Corrected minor parsing errors.
Not adding a Dfs junction to msdfs.tdb if it doesn't have any referred paths
as parsed from the dfsmap file.

When parsing a dfs map for a service, see if the mount points exist that stand
for token junction points in the Dfs. If not, create these mount points as
subdirectories of the service directory.

cmd_sam_add_addaliasmem() function gets number of names wrong when passing
argv to lsa_lookup_names()

Don't try to free command line args in cmd_sam_add_aliasmem()

Cosmetic change to help message for 'addaliasmem'.

got rid of some more old configure tests and includes

removed some obsolete configure tests (sysv ipc etc)

moved standard_sub() and friends into a separate module

bruce tenison spotted that creategroup was being requested with
insufficient access mask rights (0x00020001, whatever that is).

Fixed bug where getgrgid() would return incorrect group membership.

The getgrent() function returns domain aliases as well as domain groups and
builtin aliases.

Removed some debugs.

Renamed loop variable to avoid name clash.

Changed magic numbers in samr_enum_dom_{aliases,groups} function calls.

Added security checks for opening and creating winbind daemon UNIX domain
socket in /tmp.

Shuffled around socket code to avoid unecessary linking of Samba libraries
with libnss_ntdom.so

Return the string "unknown error" instead of a blank message for unknown
errors in get_nt_error_msg() function.

converted a couple more functions to use a fd instead of a FILE*

added a new utility fn file_lines_slashcont() which is used to handle
files that treat a \ followed by a newline as a blank

converted a bunch more functions to use a fd instead of a FILE*

to support some of this I added the following functions in util_file.c

file_lines_pload : load lines from a pipe
file_pload : load a pipe into memory

got rid of FILE* in the unix name mapping code

got rid of all the FILE* calls in the NT print system.

this also fixes a few security holes in that code. JF - you need to be
careful about open files for writing! symlinks can trap you.

note that I always unlink() the file first, then open with
O_EXCL. That is a cheap trick to avoid the symlink problems.

added fdprintf()

this is like fprintf() but operates on a file descriptor

combined with file_load_lines() this makes it really easy to get rid
of the use of fopen() in Samba.

removed old comments

improved the error handling and added queue pause and resume

use some symbolic names for print queue status

added helper fns to change from internal status codes to nt spoolss codes

- put the job status in english not french!

- add helper fns to change from internal status codes to nt spoolss codes

added tdb_get_int() and tdb_store_int()

they are very useful for database versioning

the changes to the main smb code

------------
The following series of commits are for the new tdb based printing
backend. This completely replaces our old printing backend.

Major changes include:

- all print ops are now done in printing/*.c rather than scattered all
  over the place
- system job ids are decoupled from SMB job ids
- the lpq parsers don't need to be nearly so smart, they only need to
  parse the filename, the status and system job id
- we can store lots more info about a job, including the full job name
- the queue cache control is much better

I also added a new utility routine file_lines_load() that loads a text
file and parses it into lines. This is used in out lpq parsing and I
also want to use it to replace all of our fgets() based code in other
places.

JF and Jeremy - please have a look at what I did to the spoolss
code. It now uses the new printing backend.

------------
The following series of commits are for the new tdb based printing
backend. This completely replaces our old printing backend.

Major changes include:

- all print ops are now done in printing/*.c rather than scattered all
  over the place
- system job ids are decoupled from SMB job ids
- the lpq parsers don't need to be nearly so smart, they only need to
  parse the filename, the status and system job id
- we can store lots more info about a job, including the full job name
- the queue cache control is much better

I also added a new utility routine file_lines_load() that loads a text
file and parses it into lines. This is used in out lpq parsing and I
also want to use it to replace all of our fgets() based code in other
places.

the bulk of the changes. Also split the loadparm related code into printing/load.c
so swat can get it without linking to the backend code

------------
The following series of commits are for the new tdb based printing
backend. This completely replaces our old printing backend.

Major changes include:

- all print ops are now done in printing/*.c rather than scattered all
  over the place
- system job ids are decoupled from SMB job ids
- the lpq parsers don't need to be nearly so smart, they only need to
  parse the filename, the status and system job id
- we can store lots more info about a job, including the full job name
- the queue cache control is much better

I also added a new utility routine file_lines_load() that loads a text
file and parses it into lines. This is used in out lpq parsing and I
also want to use it to replace all of our fgets() based code in other
places.

the new file_lines_load() and file_lines_free() routines. Very useful!

------------
The following series of commits are for the new tdb based printing
backend. This completely replaces our old printing backend.

Major changes include:

- all print ops are now done in printing/*.c rather than scattered all
  over the place
- system job ids are decoupled from SMB job ids
- the lpq parsers don't need to be nearly so smart, they only need to
  parse the filename, the status and system job id
- we can store lots more info about a job, including the full job name
- the queue cache control is much better

I also added a new utility routine file_lines_load() that loads a text
file and parses it into lines. This is used in out lpq parsing and I
also want to use it to replace all of our fgets() based code in other
places.