Age | Commit message (Collapse) | Author | Files | Lines | |
---|---|---|---|---|---|
2010-06-10 | Ensure we correctly setup req->subreq before testing for async. Stops ↵ | Jeremy Allison | 1 | -4/+4 | |
crashes when XX_send functions set async to stop interim replies being sent. Jeremy. | |||||
2010-06-08 | Second part of fix for 7501 - SMB2: CREATE request replies getting mangled. | Jeremy Allison | 1 | -67/+67 | |
Based on code from Ira Cooper <samba@ira.wakeful.net>, and also advice on refactoring the patch into a function call. outbuf vectors can be reallocated by smb2 processing code, so when returning interim responses we must not make assumptions about vector size. Jeremy | |||||
2010-06-08 | Fix for bug 7501 - SMB2: CREATE request replies getting mangled. | Ira Cooper | 1 | -3/+1 | |
Code for dup_smb2_req() was duplicating the wrong vector (i, instead of i+2) when returning a non-minimal SMB2 response. | |||||
2010-06-02 | Ensure we remove SMB2 cancel requests from the active queue | Jeremy Allison | 1 | -0/+1 | |
now we don't remove them in the talloc destructor. Jeremy. | |||||
2010-06-02 | Fix a crash bug found by Ira Cooper <samba@ira.wakeful.net>. | Jeremy Allison | 1 | -4/+5 | |
A create call comes in, goes async (on the oplock request). At a later time (just before a cancel request is received) it completes, and goes through smbd_smb2_request_reply() to send the reply to the create call. However, the output socket queue is full, so when tstream_writev_queue_send() is called from smbd_smb2_request_reply(), the smb2req stays on the "being processed" queue on sconn->smb2.requests, as only when tstream_writev_queue_send() completes is smbd_smb2_request_writev_done() get called, which will TALLOC_FREE the smb2req (and thus take if off the queue). The cancel comes in, gets processed and looks through the requests on the queue, and BANG - hits the smb2req that has already been processed and is outgoing.... Remove the request from the queue once tstream_writev_queue_send() is called and not in the talloc destructor function. Jeremy. | |||||
2010-05-19 | (Finally) fix bug #7158 - SMB2 connection resets during IOZone tests from ↵ | Jeremy Allison | 1 | -1/+2 | |
64-bit Vista client It turns out that the persistent handles are used by the Microsoft redirector to index files on oplock break requests. So even if we don't do durable handles (yet) we must set the persistent handle on create. For now just use the same handle value as we use for volatile. Jeremy. | |||||
2010-05-18 | Keep track of credits we're giving out. Set initial credits to 1 (MS-SMB2 ↵ | Jeremy Allison | 1 | -1/+8 | |
spec required). Jeremy. | |||||
2010-05-07 | Make us pass all SMB2 lock tests except MULTIPLE-UNLOCK and CONTEXT. Them ↵ | Jeremy Allison | 1 | -0/+8 | |
next :-). Jeremy. | |||||
2010-04-28 | Update (C) for 2010. | Jeremy Allison | 1 | -0/+1 | |
Jeremy. | |||||
2010-04-27 | Don't return an intermediate reply on async on a pipe call (Windows doesn't). | Jeremy Allison | 1 | -0/+5 | |
Jeremy. | |||||
2010-04-24 | Implement oplocks within SMB2. Plumb into the existing SMB1 oplock system. | Jeremy Allison | 1 | -3/+2 | |
Seems to work but needs more tests (to be added). Jeremy. | |||||
2010-04-23 | When sending the initial async response, this is never part of a chain. | Jeremy Allison | 1 | -1/+1 | |
Get the reply flags correct. Jeremy. | |||||
2010-04-22 | Make deferred opens (NT_STATUS_SHARING_VIOLATION) work over SMB2. | Jeremy Allison | 1 | -8/+9 | |
Makes SMB2Create call re-entrant internally. Now this infrastructure is in place, oplocks will follow shortly. Tested with Win7 client and with W2K8R2. Jeremy. | |||||
2010-04-19 | Now SMB2 error messages are correctly being returned with the 1 byte data ↵ | Jeremy Allison | 1 | -51/+49 | |
area, smbd_smb2_request_error_ex() must call smbd_smb2_request_done_ex() in order to do the padding correctly on compound replies. Jeremy. | |||||
2010-04-19 | Ensure vectors are always allocated with consistent size. Removes one byte ↵ | Jeremy Allison | 1 | -20/+60 | |
alloc on SMB2 error packet. Always use talloc_zero_array on out vectors - fixes valgrind errors in tevent writes. Jeremy. | |||||
2010-04-19 | Remove an unused auto variable. | Jeremy Allison | 1 | -3/+0 | |
Jeremy. | |||||
2010-04-19 | s3-smb2: fix some c++ warnings. | Günther Deschner | 1 | -3/+3 | |
Guenther | |||||
2010-04-18 | Final fix for #7331 - Compound async SMB 2 requests don't work right. | Jeremy Allison | 1 | -4/+13 | |
Don't free the cancelled SMB2 req early, let the cancelation function take care of it. Return a NT_STATUS_CANCELLED when we find and cancel a request. Fix our SMB2 error returns to correctly set the structuresize to 9, and add the expected zero byte (see section 2.2.2 in the SMB2 spec.). This causes Samba to pass the test program in this bug report that W2K8R2 fails (heh heh :-). This is because we always cause compound requests to get to a cancelation point before dealing with a cancel request. Jeremy. | |||||
2010-04-17 | Second part of fix for bug #7331 - Compound async SMB 2 requests don't work ↵ | Jeremy Allison | 1 | -58/+158 | |
right. Cause us to match W2K8R2 by sending an interim compound reply followed by a async reply. Jeremy. | |||||
2010-04-17 | First part of fix for bug #7331 - Compound async SMB 2 requests don't work ↵ | Jeremy Allison | 1 | -86/+320 | |
right. Gets us handling SMB2 compound async requests similar to W2K8R2 (and triggers the same client bug in the Win7 redirector). Great thanks to Ira Cooper <samba@ira.wakeful.net> for helping with this and to Metze for the wonderful async framework. The one thing I need to fix to make us identical to W2K8R2 is that when a compound request goes async at the end W2K8R2 splits the replies up into a compound non-async reply followed by a separate async reply. Currently we're doing the whole thing in a compound reply. Jeremy. | |||||
2010-04-07 | Don't forget to initialize *p_creds_requested. | Jeremy Allison | 1 | -0/+1 | |
Jeremy. | |||||
2010-04-06 | Ensure we total up the correct number of creds requested in a compound request. | Jeremy Allison | 1 | -1/+7 | |
2010-03-31 | Make smbd_lock_socket/smbd_unlock_socket recursive with a ref_count. | Jeremy Allison | 1 | -1/+1 | |
As these always call exit_server, make that part of the function. Use _internal functions for the echo client. Metze please check ! Jeremy. | |||||
2010-02-24 | Change the credential handling so that we start with maxmux creds, | Jeremy Allison | 1 | -7/+10 | |
and then return to the client the number of credits per operation that they asked for. This is a more sensible algorithm than just blindly returning "20" on every reply, although we will probably still need more changes to this going forward. Jeremy. | |||||
2009-08-17 | s3:smbd: implement SMB2 Cancel correctly. | Stefan Metzmacher | 1 | -4/+13 | |
metze | |||||
2009-08-14 | s3:smbd: allow SMB2 Cancel to have the async flag set | Stefan Metzmacher | 1 | -6/+8 | |
metze | |||||
2009-08-14 | s3:smbd: fix parsing of the SMB2 body | Stefan Metzmacher | 1 | -5/+7 | |
Maybe there's no dynamic part on the wire. metze | |||||
2009-08-12 | libcli: move some common SMB and SMB2 stuff into libcli/smb/ | Stefan Metzmacher | 1 | -1/+1 | |
This will hold code that's shared between source3 and source4. metze | |||||
2009-08-08 | s3:smbd: rename conn => sconn for smbd_server_connection structs | Stefan Metzmacher | 1 | -56/+56 | |
This should avoid confusion between smbd_server_connection and connection_struct variables. metze | |||||
2009-07-24 | Fix a few uninitialized variable warnings | Volker Lendecke | 1 | -1/+1 | |
I know those warnings are bogus, but both Coverity and gcc don't get it. | |||||
2009-07-10 | s3:smbd: fix parsing of invalid SMB2 requests. | Stefan Metzmacher | 1 | -2/+2 | |
Because of 0 - 2 => 0xFFFFFFFE, we got EMSGSIZE from the tstream layer. And terminate the transport connection. Instead we should let the caller deal with the invalid parameter, when checking the body size. So the caller always gets at least a 2 byte body. metze | |||||
2009-07-02 | s3:smbd: add marshalling layer for SMB2 Lock support | Stefan Metzmacher | 1 | -1/+1 | |
metze | |||||
2009-07-02 | s3:smbd: add marshalling layer for SMB2 Find (QueryDirectory) support | Stefan Metzmacher | 1 | -1/+1 | |
metze | |||||
2009-07-02 | s3:smbd: add marshalling layer for SMB2 Break support (SMB 2.002 only) | Stefan Metzmacher | 1 | -1/+1 | |
metze | |||||
2009-07-02 | s3:smbd: add marshalling layer for SMB2 SetInfo support | Stefan Metzmacher | 1 | -1/+1 | |
metze | |||||
2009-07-02 | s3:smbd: add marshalling layer for SMB2 GetInfo support | Stefan Metzmacher | 1 | -1/+1 | |
metze | |||||
2009-07-02 | s3:smbd: add support for SMB2 Notify | Stefan Metzmacher | 1 | -1/+1 | |
metze | |||||
2009-06-24 | s3:smbd: add support for async interim SMB2 responses and prepare SMB2 cancel | Stefan Metzmacher | 1 | -1/+155 | |
metze | |||||
2009-06-24 | s3:smbd: keep a list of outstanding SMB2 requests | Stefan Metzmacher | 1 | -28/+65 | |
metze | |||||
2009-06-24 | s3:smbd: add smbd_smb2_send_oplock_break() | Stefan Metzmacher | 1 | -0/+88 | |
metze | |||||
2009-06-24 | s3:smbd: the SMB2-COMPOUND test shows that the related vs. unrelated flags ↵ | Stefan Metzmacher | 1 | -0/+7 | |
isn't checked first metze | |||||
2009-06-09 | s3:smbd: only check the next_status for related requests | Stefan Metzmacher | 1 | -11/+12 | |
metze | |||||
2009-06-09 | s3:smbd: response with the same SMB2 flags as given by the client | Stefan Metzmacher | 1 | -1/+4 | |
...but also add the response flag. metze | |||||
2009-06-09 | s3:smbd: only pass the chained file handle for related compounded SMB2 requests | Stefan Metzmacher | 1 | -0/+4 | |
metze | |||||
2009-06-09 | s3:smbd: make sure we pad compounded SMB2 responses to 8 bytes | Stefan Metzmacher | 1 | -2/+47 | |
metze | |||||
2009-06-09 | s3:smbd: more validation of the incoming SMB2 requests | Stefan Metzmacher | 1 | -4/+94 | |
metze | |||||
2009-06-09 | s3:smbd: for now ignore all non NBT Session Message requests for SMB2 | Stefan Metzmacher | 1 | -0/+9 | |
metze | |||||
2009-06-09 | s3:smbd: don't include 1 byte padding for the dynamic buffer of SMB2 responses | Stefan Metzmacher | 1 | -21/+9 | |
It seems that Windows 2008 and Windows 7 doesn't do this anymore. metze | |||||
2009-06-05 | s3:smbd: fix the logic for compounded requests | Stefan Metzmacher | 1 | -1/+1 | |
metze | |||||
2009-06-05 | s3:smbd: only setup the dyn iovec if a a dyn blob is given | Stefan Metzmacher | 1 | -2/+1 | |
Otherwise leave the default in there, which takes care of padding for compounded requests. metze |