Commit e97321fa authored by Bob Peterson's avatar Bob Peterson
Browse files

GFS2: Don't dereference inode in gfs2_inode_lookup until it's valid 



Function gfs2_inode_lookup was dereferencing the inode, and after,
it checks for the value being NULL. We need to check that first.

Signed-off-by: default avatarBob Peterson <rpeterso@redhat.com>
parent a527b38e

fs/gfs2/inode.c

+3 −3
Original line number Diff line number Diff line
@@ -93,12 +93,12 @@ struct inode *gfs2_inode_lookup(struct super_block *sb, unsigned int type, 
	int error;



	inode = iget_locked(sb, (unsigned long)no_addr);

	ip = GFS2_I(inode);

	ip->i_no_addr = no_addr;



	if (!inode)

		return ERR_PTR(-ENOMEM);



	ip = GFS2_I(inode);

	ip->i_no_addr = no_addr;



	if (inode->i_state & I_NEW) {

		struct gfs2_sbd *sdp = GFS2_SB(inode);

		ip->i_no_formal_ino = no_formal_ino;