sin/samba-mirror
1
0
Fork 0
mirror of https://github.com/samba-team/samba.git synced 2025-01-17 02:05:21 +03:00
Code

r3133: - more consistent error checking in rename and setfileinfo

Browse Source 
- add paranoid checking of device/inode change during open to detect race conditions
This commit is contained in:
Andrew Tridgell 2004-10-22 06:55:18 +00:00 committed by Gerald (Jerry) Carter
parent c33cdd0d55
commit 043361fed4
3 changed files with 21 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
source/ntvfs/posix/pvfs_rename.c
View File

@ -62,7 +62,7 @@ NTSTATUS pvfs_rename(struct ntvfs_module_context *ntvfs,
return NT_STATUS_OBJECT_NAME_COLLISION;
}
if (rename(name1->full_name, name2->full_name) != 0) {
if (rename(name1->full_name, name2->full_name) == -1) {
return pvfs_map_errno(pvfs, errno);
}

19
source/ntvfs/posix/pvfs_resolve.c
View File

@ -364,10 +364,29 @@ NTSTATUS pvfs_resolve_partial(struct pvfs_state *pvfs, TALLOC_CTX *mem_ctx,
NTSTATUS pvfs_resolve_name_fd(struct pvfs_state *pvfs, int fd,
struct pvfs_filename *name)
{
dev_t device;
ino_t inode;
if (name->exists) {
device = name->st.st_dev;
inode = name->st.st_ino;
}
if (fstat(fd, &name->st) == -1) {
return NT_STATUS_INVALID_HANDLE;
}
if (name->exists &&
(device != name->st.st_dev || inode != name->st.st_ino)) {
/* the file we are looking at has changed! this could
be someone trying to exploit a race
condition. Certainly we don't want to continue
operating on this file */
DEBUG(0,("pvfs: WARNING: file '%s' changed during resole - failing\n",
name->full_name));
return NT_STATUS_UNEXPECTED_IO_ERROR;
}
name->exists = True;
return pvfs_fill_dos_info(pvfs, name);

2
source/ntvfs/posix/pvfs_setfileinfo.c
View File

@ -43,7 +43,7 @@ NTSTATUS pvfs_setfileinfo(struct ntvfs_module_context *ntvfs,
case RAW_SFILEINFO_END_OF_FILE_INFO:
case RAW_SFILEINFO_END_OF_FILE_INFORMATION:
if (ftruncate(f->fd,
info->end_of_file_info.in.size) != 0) {
info->end_of_file_info.in.size) == -1) {
return pvfs_map_errno(pvfs, errno);
}
break;