diff options
| author | Volker Lendecke <vl@samba.org> | 2008-02-25 13:00:24 +0100 |
|---|---|---|
| committer | Volker Lendecke <vl@samba.org> | 2008-02-25 13:05:18 +0100 |
| commit | 71641f04d4e9a41775e898f2e4ac3c8792783cd3 (patch) | |
| tree | c7d4d7b855fa981408cef7eb447fe0167f4b5e39 /source/modules | |
| parent | 4a1a138b0fe0f2200e5a37a0609481e4340a896c (diff) | |
| download | samba-71641f04d4e9a41775e898f2e4ac3c8792783cd3.tar.gz samba-71641f04d4e9a41775e898f2e4ac3c8792783cd3.tar.xz samba-71641f04d4e9a41775e898f2e4ac3c8792783cd3.zip | |
Correctly check return of rename(2)
Bug 5279 -- Thanks to Max Matveev
Diffstat (limited to 'source/modules')
| -rw-r--r-- | source/modules/vfs_default.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source/modules/vfs_default.c b/source/modules/vfs_default.c index cf135dfd030..31ebb6352a1 100644 --- a/source/modules/vfs_default.c +++ b/source/modules/vfs_default.c @@ -468,7 +468,7 @@ static int vfswrap_rename(vfs_handle_struct *handle, const char *oldname, const START_PROFILE(syscall_rename); result = rename(oldname, newname); - if (errno == EXDEV) { + if ((result == -1) && (errno == EXDEV)) { /* Rename across filesystems needed. */ result = copy_reg(oldname, newname); } |