----- Original Message ----- | Something like this? (snip) | @@ -779,6 +782,11 @@ static struct dentry *__gfs2_lookup(struct inode *dir, | struct dentry *dentry, | } | | d = d_splice_alias(inode, dentry); | + if (IS_ERR(d)) { | + iput(inode); | + gfs2_glock_dq_uninit(&gh); | + return ERR_PTR(error); ---------------------------------^ Shouldn't that be ERR_PTR(d)? Bob Peterson Red Hat File Systems -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html