On Tue, 2016-03-15 at 14:36 +0000, Al Viro wrote: > On Tue, Mar 15, 2016 at 09:43:11AM -0400, Mimi Zohar wrote: > > +void ima_post_path_mknod(struct dentry *dentry) > > +{ > > + struct integrity_iint_cache *iint; > > + struct inode *inode; > > + int must_appraise; > > + > > + if (!dentry || !dentry->d_inode) > > In which situations is that condition supposed to be true? Never, removing tests. Mimi -- 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