Henrique Carvalho <henrique.carvalho@xxxxxxxx> writes: > The function open_cached_dir_by_dentry() is only called by > cifs_dentry_needs_reval(), and it always passes dentry->d_parent as the > argument to dentry. > > Since dentry->d_parent cannot be NULL, the check for dentry == NULL > is unnecessary and can be removed. > > Signed-off-by: Henrique Carvalho <henrique.carvalho@xxxxxxxx> > --- > fs/smb/client/cached_dir.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Looks good, Acked-by: Paulo Alcantara (Red Hat) <pc@xxxxxxxxxxxxx>