Browse Source

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>
Abhi Das 11 years ago
parent
commit
48f8f711ed
1 changed files with 4 additions and 0 deletions
  1. 4 0
      fs/gfs2/inode.c

+ 4 - 0
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);
 		tmp = gfs2_lookupi(dir, &gfs2_qdotdot, 1);
+		if (!tmp) {
+			error = -ENOENT;
+			break;
+		}
 		if (IS_ERR(tmp)) {
 		if (IS_ERR(tmp)) {
 			error = PTR_ERR(tmp);
 			error = PTR_ERR(tmp);
 			break;
 			break;