Move NULL check before dereference tries
CID 5912
This commit is contained in:
@@ -1018,6 +1018,11 @@ dosfs_rename(fs_volume *_vol, fs_vnode *_odir, const char *oldname,
|
|||||||
|
|
||||||
DPRINTF(0, ("dosfs_rename called: %Lx/%s->%Lx/%s\n", odir->vnid, oldname, ndir->vnid, newname));
|
DPRINTF(0, ("dosfs_rename called: %Lx/%s->%Lx/%s\n", odir->vnid, oldname, ndir->vnid, newname));
|
||||||
|
|
||||||
|
if (!oldname || !(*oldname) || !newname || !(*newname)) {
|
||||||
|
result = EINVAL;
|
||||||
|
goto bi;
|
||||||
|
}
|
||||||
|
|
||||||
if(!is_filename_legal(newname)) {
|
if(!is_filename_legal(newname)) {
|
||||||
dprintf("dosfs_rename called with invalid name '%s'\n", newname);
|
dprintf("dosfs_rename called with invalid name '%s'\n", newname);
|
||||||
result = EINVAL;
|
result = EINVAL;
|
||||||
@@ -1030,11 +1035,6 @@ dosfs_rename(fs_volume *_vol, fs_vnode *_odir, const char *oldname,
|
|||||||
goto bi;
|
goto bi;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!oldname || !(*oldname) || !newname || !(*newname)) {
|
|
||||||
result = EINVAL;
|
|
||||||
goto bi;
|
|
||||||
}
|
|
||||||
|
|
||||||
if ((odir->vnid == ndir->vnid) && !strcmp(oldname, newname)) {
|
if ((odir->vnid == ndir->vnid) && !strcmp(oldname, newname)) {
|
||||||
result = EPERM;
|
result = EPERM;
|
||||||
goto bi;
|
goto bi;
|
||||||
|
|||||||
Reference in New Issue
Block a user