On Tue, Dec 20, 2016 at 04:18:37PM +0000, Colin King wrote: > From: Colin Ian King <colin.k...@canonical.com> > > The check for a null inode is redundant since the function > is a callback for exportfs, which will itself crash if > dentry->d_inode or parent->d_inode is NULL. Removing the > null check makes this consistent with other file systems. > > Also remove the redundant null dir check too. > > Found with static analysis by CoverityScan, CID 1389472 > > Kudos to Jeff Mahoney for reviewing and explaining the error in > my original patch (most of this explanation went into the above > commit message) and David Sterba for pointing out that the dir > check is also redundant. > > Signed-off-by: Colin Ian King <colin.k...@canonical.com>
Reviewed-by: David Sterba <dste...@suse.com> Added to cleanups queue, thanks.
