r24347: fake_sendfile does not need Inbuf/Outbuf

In the future, we might put the new Linux splice(2) syscall here. This
should also work for reply_write, but getting that in is a bit trickier.
We need to decide very early before fetching the whole buffer that we
have a write call.

r24346: Push reply_prep_legacy into send_file_readX

r24341: Make libreplace provide offsetof.

r24340: Use standard data type uint32_t rather than tdb-specific u32.

r24332: schedule_aio_read_and_X does not need InBuf/OutBuf

r24331: Fix an error where gid_t != 32 bit

r24330: Fix a 64-bit error

r24329: Fix a 64-bit bug

enums are not necessarily represented as 32-bit uints. On assignment
(see line 1029) implicit conversion happens, but not when pointers are
taken.

r24328: Restore chain_reply error handling

Get the whole smb header from the second chained function, in particular
the error fields

r24327: First round of fixes to chain_reply

The argument to smb_setlen does not contain the nbt header of 4 bytes

The chained function might allocate outbuf itself (as now happens with
reply_read_and_X). This would erroneously overwrite the caller's outbuf.
Give it an outbuf pointer of it's own

r24326: Fix the build for Solaris CC

r24325: Push down reply_prep_legacy in reply_read_and_X

r24324: No reply_prep_legacy() in reply_pipe_read_and_X

r24323: Remove the reply_read_and_X wrapper function

r24322: Wrap reply_read_and_X in reply_prep_legacy

r24321: Attempt to fix the "hape" cc build

r24320: Fix the wct checks in reply_trans2 and the reply_trans calls

No idea what I've been smoking here.

Thanks to Stefan for closely looking

r24319: Check wct in reply_read_and_X

r24318: Print a prominent warning when reply_prep_legacy is called

This is a temporary function anyway

r24316: Fix an uninitialized read

Jerry, please check this!

Thanks,

Volker

r24314: Attempt to fix bug 4868

r24312: revision 24091 removed the 'FORCE'...

metze

r24311: add a reply_force_nterror() macro

metze

r24310: we should not call END_PROFILE(SMBntcreateX); twice...

metze

r24309: if no protocol matches, just return choise 0xffff
as we've done before revision 24002

metze

r24308: this needs a better fix that compiles...

metze

r24307: fill in choise with value -1 on error
as we've done before revision 24002

metze

r24306: the check_fsp() function has an explicit user argument
so use it...

metze

r24304: patch from Bjoern JAcke <bj@SerNet.DE>:

attached patches add EA support for Solaris. If no one disagrees, can
someone check this in please?

metze

r24302: add a module that overloads the file_id_create VFS function
and alters the device id depending on the configured algorithm.

The algorithm is configured via "fileid:algorithm":
- "fsname" (default) uses a uint64 hash over the mount point
- "fsid" uses the fsid returned from statfs()

This is needed for "clustering = yes" on some clusterfilesystems

metze

r24291: Fix Coverity ID 364

We've checked num_rids != 0 above.

r24290: fix debug statments to match function name

r24281: Fix bug found by Herb. The vuid entry in the cli_state structure gets
left as nonzero as returned by the failed cli_session_setup_spnego. When we then try
to authenticate as the user in cli_session_setup this returns an
error "Bad userid" (as seen in wireshark).
"We should only leave cli->vuid != 0 on success. Looks like it's
getting set in the cli_session_setup_blob_receive() call and not
cleared again on error."
Jeremy.

r24280: Fix the build of vfs_afsacl.c

r24279: Remove reply_prep_legacy from reply_write_and_X

r24278: Push down reply_prep_legacy in reply_write_and_X

Remove the need for reply_prep_legacy for reply_pipe_write_and_X

r24272: try to fix the build on irix...

metze

r24271: Push reply_prep_legacy into reply_write_and_X

r24270: Fix some END_PROFILE(SMBwriteX)

r24269: Check wct in reply_write_and_X

r24268: Fix two crashes for spoolss

strlcpy does not like a NULL source. Fix a use-after-free.

r24267: Fix the build farm

I had only tested with "net getlocalsid". posix_locking_init() calls this
with a NULL name...

r24266: Remove the "open files database hash size" parameter

This has been superseded by the "tdb_hashsize:<tdbname>" parameter

r24265: Add the tdb_hashsize:<tdbname> parameter

This makes it possible to set the default hashsize for any tdb. I would
like to remove the "open files database hash size" in favor of this one.

I'll check that removal in with the next commit, please complain/revert
if it's not ok.

Volker

r24254: Fix a segfault in r24102

The really pure bugfix would have replaced pparam by *pparam, but we pulled
the fid anyway.

Metze, in line 2421 the FSP_BELONGS_CONN is commented out, is that intended?
The FSCTL_CREATE_OR_GET_OBJECT_ID ioctl did not have it.

Volker

r24253:  From Jan Martin <Jan.Martin@rwedea.com>.
----------------------------------------------------------
In rare cases, Samba 3.0.25b shows directory contents at the wrong
position in the file tree when displaying a subdirectory of a DFS link.

The problem occurs whenever Windows XP asks for a DFS referral for a
subdirectory of a DFS link with a trailing backslash.
Windows does not do this very often, but we saw it several times per day
on our central DFS server.

smbd/msdfs.c, dfs_path_lookup() does the following with the requested
path:
- in line 390, the local copy 'localpath' is 'unix_convert'ed; the
trailing backslash is removed inside unix_convert
- in lines 417-20, 'dfspath' (another copy of the requested path) is
mangled another way without removing trailing backslashes

That's why the following loop (lines 435-461) that is meant to
synchronously cut off the last path component from both strings until it
comes to a DFS link, does not handle both strings the same.  When the
original path ended with a backslash, 'canon_dfspath' has always one
component more than 'localpath', so that *consumedcntp gets too big in
line 446. This value is reported to the client.
----------------------------------------------------------

Bug #4860.

Jeremy.

r24252: Dump guid of msExchMailboxGuid when returned.

Guenther

r24251: Neverending fun:

Heimdal doesn't accept all OIDs and gss_import_name() fails with
GSS_S_BAD_NAMETYPE using this one. Use the GSS_KRB5_NT_PRINCIPAL_NAME OID
instead (which works with at least MIT 1.6.1 and Heimdal 1.0.1).

Guenther

r24250: Merge W_ERROR_HAVE_NO_MEMORY macro from Samba 4.

Guenther

r24244: Complete the conversion of reply_trans[s]

Remove reply_prep/post_legacy from send_trans_replies

r24243: Remove reply_prep_legacy from api_reply

r24242: Remove reply_prep_legacy from api_rpc_trans_reply

r24241: Remove reply_prep_legacy from api_WNPHS and api_SNPHS

r24240: Push down reply_prep_legacy one level inside api_fd_reply

r24239: Push reply_prep_legacy into api_fd_reply

r24238: Push down reply_prep_legacy one level

r24237: Add send_trans_reply_new

r24236: Push reply_prep_legacy() into named_pipe()

r24235: Make handle_trans reply to errors itself

r24234: Reformatting -- SCNR

r24233: Push reply_prep_legacy into handle_trans

r24232: Push down reply_prep_legacy inside reply_transs

r24231: Push allocating InBuffer/OutBuffer into reply_transs

r24230: Push down reply_prep_legacy in reply_trans

The last checkin was for reply_trans, not reply_transs

r24229: Push allocating InBuffer/OutBuffer into reply_transs

r24225: Convert reply_flush to the new API

r24224: Check wct in reply_trans

r24223: Convert reply_echo to the new API

r24167: Fix Coverity ID 365

We checked argc>=3 some lines above

r24166: Fix Coverity ID 391

r24165: Fix Coverity ID 369.

This was not really a bug I think, but this change cleans up the code a
bit.

r24164: Fix for write cache corruption bug reported by Jean-Francois Panisset <panisset@A52.com>.
Awaiting confirmation from reporter.
Jeremy.

r24163: Remove the send_trans2_replies_new wrapper

This changes send_trans2_replies to not depend on large buffers anymore
and finishes the trans2 conversion.

r24162: send_trans2_replies is now static in trans2.c

r24161: handle_trans2 takes care of errors itself, no need to do it in reply_transs2

r24160: process_trans2 in smbd/blocking.c used send_trans2_replies.

Fake a struct smb_request here.

Volker

r24159: Convert reply_transs2 to the new API

r24158: SE_GROUP_RESOURCE in the other_sids list apparently means a
domain local group.

Fix a typo in the PAC debugging routine

r24156: Convert reply_trans2 to the new API

r24155: Check wct in reply_trans2

r24154: handle_trans2 does not reference inbuf/outbuf anymore

r24153: Convert call_trans2setfsinfo to the new API.

This *might* break the transport encryption stuff. I need to check that.

r24152: Convert call_trans2ioctl to the new API

r24151: Convert call_trans2getdfsreferral to the new API

r24150: Convert call_trans2qfsinfo to the new API

r24149: Convert call_trans2setfilepathinfo to the new API

r24148: Remove some inbuf refs from subcommands of call_trans2setfilepathinfo

r24147: Remove some inbuf refs from call_trans2setfilepathinfo

r24145: Fix a comment

r24144: Fix the build

In a function returning void the "return;" needs to stand alone on some
compilers

r24143: Convert call_trans2qfilepathinfo to the new API

r24142: Remove some direct inbuf references from call_trans2qfilepathinfo

r24141: Add check_fsp as a replacement for CHECK_FSP

r24140: Convert call_trans2findnotifyfirst/next to the new API

r24139: Convert call_trans2mkdir to the new API

r24138: Convert call_trans2findnext to the new API

r24137: Convert call_trans2findfirst to the new API

r24136: get_lanman2_dir_entry() does not need access to inbuf/outbuf

It did not use inbuf anyway. Outbuf was used for alignment and to the
flags2. The alignment works ok if we align relative to base_data because this
and outbuf are both malloc'ed, and we put send_trans2_replies aligns out_data
properly relative to the start of the outbuf.

I don't know if the explicit align_string() calls in get_lanman2_dir_entry are
really necessary. I think that srvstr_push also does the same.

Please also note that this changes dst_len in srvstr_push from -1 to the real
length remaining.

Volker

r24135: Convert call_trans2open to the new API

This itself won't help much, because send_trans2_replies_new still allocates
the big buffers, but stay tuned :-)

Also add/update my copyright on stuff I recently touched.

Volker

r24134: talloc smb_request for handle_trans2

When starting to convert the individual trans2 subcalls, I need the new
API conventions to be present there. This means that those calls fill in
req->outbuf when there's something to ship