diff options
| author | Wengang Wang <wen.gang.wang@oracle.com> | 2011-07-12 16:43:14 +0800 |
|---|---|---|
| committer | Joel Becker <jlbec@evilplan.org> | 2011-11-17 01:46:46 -0800 |
| commit | 8298524803339a9a8df053ebdfebc2975ec55be9 (patch) | |
| tree | 2bea3c08eac9239ea596707fadfefed7f1bab2d7 /fs/ocfs2/dir.c | |
| parent | fc9f899483435935c1cd7005df29681929d1c99b (diff) | |
| download | talos-obmc-linux-8298524803339a9a8df053ebdfebc2975ec55be9.tar.gz talos-obmc-linux-8298524803339a9a8df053ebdfebc2975ec55be9.zip | |
ocfs2: make direntry invalid when deleting it
When we deleting a direntry from a directory, if it's the first in a block we
invalid it by setting inode to 0; otherwise, we merge the deleted one to the
prior and contiguous direntry. And we don't truncate directories.
There is a problem for the later case since inode is not set to 0.
This problem happens when the caller passes a file position as parameter to
ocfs2_dir_foreach_blk(). If the position happens to point to a stale(not
the first, deleted in betweens of ocfs2_dir_foreach_blk()s) direntry, we are
not able to recognize its staleness. So that we treat it as a live one wrongly.
The fix is to set inode to 0 in both cases indicating the direntry is stale.
This won't introduce additional IOs.
Signed-off-by: Wengang Wang <wen.gang.wang@oracle.com>
Signed-off-by: Joel Becker <jlbec@evilplan.org>
Diffstat (limited to 'fs/ocfs2/dir.c')
| -rw-r--r-- | fs/ocfs2/dir.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/ocfs2/dir.c b/fs/ocfs2/dir.c index 8582e3f4f120..3302088e1f04 100644 --- a/fs/ocfs2/dir.c +++ b/fs/ocfs2/dir.c @@ -1184,8 +1184,7 @@ static int __ocfs2_delete_entry(handle_t *handle, struct inode *dir, if (pde) le16_add_cpu(&pde->rec_len, le16_to_cpu(de->rec_len)); - else - de->inode = 0; + de->inode = 0; dir->i_version++; ocfs2_journal_dirty(handle, bh); goto bail; |
