summaryrefslogtreecommitdiffstats
path: root/source/smbd/reply.c
Commit message (Collapse)AuthorAgeFilesLines
* r25387: Sync with 3.2.0 svn treeGerald Carter2007-09-271-437/+758
|
* r25055: Add file_id_string_tosVolker Lendecke2007-09-101-2/+2
| | | | This removes file_id_string_static and file_id_string_static2
* r24702: Remove the old API pointersVolker Lendecke2007-08-271-9/+6
|
* r24660: Merge the inbuf/outbuf changesVolker Lendecke2007-08-251-1313/+1910
|
* r24548: Fix the case-changing renamesVolker Lendecke2007-08-191-14/+9
| | | | | This was broken when I changed reply_mv to wrap in a open_file_ntcreate call, unix_convert on the destination was called twice
* r24499: Move the checks for DOS error codes on open to a function.Jeremy Allison2007-08-161-44/+4
| | | | Jeremy.
* r24476: Fix the mappings in reply_opeXXX calls. Now to test renames.Jeremy Allison2007-08-161-0/+30
| | | | Jeremy.
* r24461: Fix Bug 4852, thank to anto <sistemac@prosoft.ba> for reporting it.Volker Lendecke2007-08-151-0/+10
|
* r24122: merge from SAMBA_3_2:Stefan Metzmacher2007-08-021-1/+2
| | | | | | | | | add a file_id_create() hook into the VFS layer it's needed for some cluster filesystems to overload this function. metze
* r24071: Fix a missing END_PROFILE callVolker Lendecke2007-07-301-0/+1
|
* r23992: Some constVolker Lendecke2007-07-221-2/+2
|
* r23913: Revert back to Volker's original logic to fix theJeremy Allison2007-07-171-1/+1
| | | | | | | RAW-SFILEINFO-RENAME until I can figure out what is different from the way CIFSFS drives this in the cthon tests and the way smbtorture drives it. Jeremy.
* r23911: Revert r23910 to try and fix the build farm. IJeremy Allison2007-07-171-9/+0
| | | | | | | | | need to look at this more closely tomorrow. Stevef's cthon tests definately show we're not matching Windows behaviour (as his tests pass against Windows but not SAMBA_3_2) but this isn't the fix. Jeremy.
* r23910: We used to deny renames on theJeremy Allison2007-07-171-0/+9
| | | | | | | | | source open for non-delete open. Turns out this is not the case. VL please test but this matches Windows behaviour. (I'll add a torture test tomorrow). Jeremy.
* r23909: Get closer to passing the cthon tests for delete open file.Jeremy Allison2007-07-171-3/+6
| | | | | It matters how the target is open. Jeremy.
* r23904: Remove an unused variable referenceVolker Lendecke2007-07-161-1/+0
|
* r23902: Fix uninitialized read in devicetype noticed by Volker.Jeremy Allison2007-07-161-1/+1
| | | | Jeremy
* r23858: Added srvstr_pull_buf_talloc() and srvstr_pull_talloc()Jeremy Allison2007-07-131-41/+72
| | | | | | | | | | | calls and converted reply_tcon and reply_tconX to use them - to show the boilerplate usage (valgrind tested). In conjunction with Volker's srvstr_get_path_talloc() work this should allow us to start eliminating all pstrings/fstrings out of the main path processing code. I'll watch the build farm tonight... Jeremy.
* r23784: use the GPLv3 boilerplate as recommended by the FSF and the license textAndrew Tridgell2007-07-101-2/+1
|
* r23780: Find and fix more GPL2 -> GPL3.Jeremy Allison2007-07-091-1/+1
| | | | Jeremy.
* r23752: Fix bug introduced by checkin 22920, allow largeJeremy Allison2007-07-091-3/+1
| | | | | | | | | | | | | readX. Fix from Dmitry Shatrov <dhsatrov@linux.vnet.ibm.com>. "In send_file_readX(), if startpos > sbuf.st_size, then smb_maxcnt is set to an invalid large value due to integer overflow. As for me, this resulted in MS Word hanging while trying to save a 1.5Mb document." This isn't in shipping code. Jeremy.
* r23741: Combined merge of 23726, 23727 and 23731 from 3_0:Volker Lendecke2007-07-071-31/+70
| | | | | | | | | | | | | | | | | | | | | 23726: Explicitly pass down the FLAGS2 field to srvstr_pull_buf. The next checkin will pull this up to srvstr_get_path. At that point we can get more independent of the inbuf, the base_ptr in pull_string will only be used to satisfy UCS2 alignment constraints. 23731: Explicitly pass down FLAGS2 to srvstr_get_path. Next step is to remove the bug that in the trans2 code we use the inbuf as the base pointer to decide whether we need ucs2 alignment where we need to use the beginning of the params buffer 23731: Forgot one reference to inbuf
* r23739: Merge r23724 from 3_0 without the accidential checkin:Volker Lendecke2007-07-071-22/+42
| | | | | | | | Reduce access to the global inbuf a tiny bit. Add a struct smb_request that contains some of the fields from the SMB header, removing the need to access inbuf directly. This right now is used only in the open file code & friends, and creating that header is only done when needed. This needs more work, but it is a start.
* r23538: Fix for wild-card rename: We can't return directly on error, we need toVolker Lendecke2007-06-181-2/+2
| | | | CloseDir.
* r23526: Merge 23522/23523 from 3_0:Volker Lendecke2007-06-161-30/+23
| | | | | | | | Save us a kilobyte stack space in a hot code path: I can't see a reason why check_path_syntax should not be able to run in-line. The destination pointer either walks side by side with the source pointer or is decremented. So as far as I can see s>=d is true throughout the whole routine.
* r23517: After Jeremy has given is ack on irc:Volker Lendecke2007-06-161-179/+61
| | | | | | | | Change rename_internals to open the file/directory and then call rename_internals_fsp. Two reasons: Remove code duplication and remove a race condition. The race condition was due to the fact that in can_rename the share mode check closed the file and then after that did the rename.
* r23508: Fix sync_file() to return NTSTATUS and return thisJeremy Allison2007-06-151-8/+50
| | | | | on failure in the write path. Jeremy.
* r23501: Move notify_rename before rename_internals_fsp and call it from there.Volker Lendecke2007-06-141-42/+44
|
* r23500: Two changes to survive the now activated test for rename_internals_fsp:Volker Lendecke2007-06-141-2/+13
| | | | | With the target being open we have to return NT_STATUS_ACCESS_DENIED and root_fid != 0 leads to NT_STATUS_INVALID_PARAMETER
* r23486: Ok, this time with a hopefully successful make test in the right place:Volker Lendecke2007-06-141-18/+33
| | | | Remove two local variables
* r23483: Revert 23482, I must have run 'make test' in the wrong subdir.Volker Lendecke2007-06-141-14/+18
|
* r23482: Slightly simplify the rename code: Remove two local variables that areVolker Lendecke2007-06-141-18/+14
| | | | not really needed.
* r23469: Fix a commentVolker Lendecke2007-06-131-1/+1
|
* r23467: Next little simplification: In rename_internals it's a bit pointless toVolker Lendecke2007-06-131-38/+13
| | | | | | first ask for existence of a file when we do the open_file_ntcreate in can_rename later on anyway. That also gets us the right error message in case the file is not there automatically.
* r23466: Fix RAW-NOTIFY: by using delete on close the notify is triggered deepVolker Lendecke2007-06-131-6/+0
| | | | inside close_file() already.
* r23457: After Jeremy's ack:Volker Lendecke2007-06-131-24/+29
| | | | | | | | | | | | | | The attached patch removes a little race condition for people with real kernel oplock support, and reduces some code paths. It changes reply_unlink to open_file_ntcreate, set_delete_on_close and close_file. The race condition happens if we break the oplock in can_delete via open_file_ntcreate, we close the file, someone else gets a batch oplock and we try to unlink. It reduces code paths by calling SMB_VFS_UNLINK in 2 fewer places.
* r23445: Fix suggested by Volker. Don't call rename_open_filesJeremy Allison2007-06-121-2/+0
| | | | | if the name wasn't changed. Jeremy.
* r23299: Fix the build for !WITH_SENDFILE.James Peach2007-06-011-5/+4
|
* r23183: Check in a change made by Tridge:Volker Lendecke2007-05-291-20/+19
| | | | | | | | | | | | | This replaces the internal explicit dev/ino file id representation by a "struct file_id". This is necessary as cluster file systems and NFS don't necessarily assign the same device number to the shared file system. With this structure in place we can now easily add different schemes to map a file to a unique 64-bit device node. Jeremy, you might note that I did not change the external interface of smb_share_modes.c. Volker
* r23107: Fix renames on file descriptors that are supposed to overwrite theJeremy Allison2007-05-241-3/+4
| | | | | target. Needs merging for 3.0.25a (sorry). Jeremy.
* r23100: Implement the delete on close semantics I've just tested forJeremy Allison2007-05-231-7/+47
| | | | | | in Samba4 smbtorture. Fix rename on an open file handle. Needed for 3.0.25a. Jeremy.
* r23088: Fix rename for cifsfs client. This may be needed forJeremy Allison2007-05-221-2/+2
| | | | | 3.0.25a. Jeremy.
* r23016: Remove extra & - thanks to Volker for spotting this.Jeremy Allison2007-05-191-1/+1
| | | | Jeremy.
* r23014: For all branches, ensure that if we're blocked on a POSIXJeremy Allison2007-05-191-4/+9
| | | | | | | | lock we know nothing about that we retry the lock every 10 seconds instead of waiting for the standard select timeout. This is how we used to (and are supposed to) work. Jeremy.
* r23007: Ensure we don't allow large read over the possibleJeremy Allison2007-05-191-0/+4
| | | | | packet size. Jeremy.
* r22920: Add in the UNIX capability for 24-bit readX, as discussedJeremy Allison2007-05-161-61/+100
| | | | | | with the Apple guys and Linux kernel guys. Still looking at how to do writeX as there's no recvfile(). Jeremy.
* r22846: Chunk one to replace message_send_pid with messaging_send: Deep insideVolker Lendecke2007-05-141-8/+16
| | | | | | locking/locking.c we have to send retry messages to timed lock holders. The majority of this patch passes a "struct messaging_context" down there. No functional change, survives make test.
* r22765: Fix from Alison Winters <alisonw@sgi.com> for missing returnJeremy Allison2007-05-091-0/+1
| | | | | in sendfilereadbraw. Jeremy.
* r22502: Fix bug #4536 - delete symlinks to a directory correctly.Jeremy Allison2007-04-241-1/+17
| | | | Jeremy.
* r22291: Fix off-by-one in tconX parsing.Jeremy Allison2007-04-171-2/+11
| | | | Jeremy.