omfs: rename() needs to mark old_inode dirty after ctime update

we *do* mark it dirty before, but it doesn't guarantee that we don't get preempted just before assignment to ->i_ctime, with inode getting written out before we get CPU back... Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> Signed-off-by: Bob Copeland <me@bobcopeland.com>

omfs: rename() needs to mark old_inode dirty after ctime update
we *do* mark it dirty before, but it doesn't guarantee that we don't get preempted just before assignment to ->i_ctime, with inode getting written out before we get CPU back... Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> Signed-off-by: Bob Copeland <me@bobcopeland.com>
Al Viro · Bob Copeland
1 parent 85e2efbb1d
Showing 1 changed file with 1 additions and 0 deletions Side-by-side Diff
fs/omfs/dir.c
@@ -423,6 +423,7 @@
 		goto out;
  
 	old_inode->i_ctime = CURRENT_TIME_SEC;
+	mark_inode_dirty(old_inode);
 out:
 	return err;
 }
...	...	@@ -423,6 +423,7 @@
423	423	goto out;
424	424
425	425	old_inode->i_ctime = CURRENT_TIME_SEC;
	426	+ mark_inode_dirty(old_inode);
426	427	out:
427	428	return err;
428	429	}