On 2018-10-19 10:22, Jan Kara wrote:
> On Thu 18-10-18 08:27:40, Richard Guy Briggs wrote:
> > > if (chunk->dead || !(entry->flags & FSNOTIFY_MARK_FLAG_ATTACHED)) {
> > > - mutex_unlock(&entry->group->mark_mutex);
> > > - if (new)
> > > - fsnotify_put_mark(new->mark);
> > > - goto out;
> > > + mutex_unlock(&audit_tree_group->mark_mutex);
> > > + return;
> >
> > This isn't an error, but more a question of style. Since the end of the
> > function has been simplified, this could just be "goto out_mutex", or
> > remove the one remaining "goto out_mutex" after the next patch and do
> > the same as here since other exits aren't so clean.
>
> Good point, I've switch this to "goto out_mutex". The result is attached.
> Can I add your Reviewed-by tag?
Yup.
> Jan Kara <[email protected]>
- RGB
--
Richard Guy Briggs <[email protected]>
Sr. S/W Engineer, Kernel Security, Base Operating Systems
Remote, Ottawa, Red Hat Canada
IRC: rgb, SunRaycer
Voice: +1.647.777.2635, Internal: (81) 32635
--
Linux-audit mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/linux-audit
