diff options
| author | Abhi Das <adas@redhat.com> | 2014-03-12 03:41:44 -0500 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2014-03-12 09:50:27 +0000 |
| commit | 48f8f711edf3868fe4faa28a19f07acb43532c4a (patch) | |
| tree | 0e6f5f96792c6173792d16b1b7fe9c92b5c08998 | |
| parent | cb94eb066e089da461d37fea39779606512e144a (diff) | |
| download | op-kernel-dev-48f8f711edf3868fe4faa28a19f07acb43532c4a.zip op-kernel-dev-48f8f711edf3868fe4faa28a19f07acb43532c4a.tar.gz | |
GFS2: check NULL return value in gfs2_ok_to_move
gfs2_lookupi() can return NULL if the path to the root is broken by
another rename/rmdir. In this case gfs2_ok_to_move() must check for
this NULL pointer and return error.
Resolves: rhbz#1060246
Signed-off-by: Abhi Das <adas@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
| -rw-r--r-- | fs/gfs2/inode.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c index ec455b9..b52ebf8 100644 --- a/fs/gfs2/inode.c +++ b/fs/gfs2/inode.c @@ -1299,6 +1299,10 @@ static int gfs2_ok_to_move(struct gfs2_inode *this, struct gfs2_inode *to) } tmp = gfs2_lookupi(dir, &gfs2_qdotdot, 1); + if (!tmp) { + error = -ENOENT; + break; + } if (IS_ERR(tmp)) { error = PTR_ERR(tmp); break; |
