On Wed, Mar 31, 2021 at 3:13 AM Paul Moore <paul@xxxxxxxxxxxxxx> wrote:
> On Tue, Mar 30, 2021 at 9:16 AM Ondrej Mosnacek <omosnace@xxxxxxxxxx> wrote:
> >
> > This series contains a patch that fixes broken conditional AV list
> > duplication introduced by c7c556f1e81b ("selinux: refactor changing
> > booleans") and a couple "and while I'm here..." cleanup patches on top.
> >
> > Ondrej Mosnacek (3):
> > selinux: fix cond_list corruption when changing booleans
> > selinux: simplify duplicate_policydb_cond_list() by using kmemdup()
> > selinux: constify some avtab function arguments
>
> Please don't resubmit, but in the future if you are submitting a patch
> (or two (or three ...)) which is potential -stable material (and so
> far I think 1/3 qualifies) don't submit it in a patchset with trivial
> cleanup patches. Adding cleanup patches to a patchset that adds a
> feature is okay, but fixes should generally stand by themselves.
Okay, but in this case the patches are sort of interdependent, so I
didn't want to mess with a rebase and sending conflicting patches... I
did move the bugfix patch to the bottom so that at worst you can pick
it out and ask me to resubmit the rest some other way.
Since I'll need to respin the bugfix patch, will it be better if I
repost just the one patch?
--
Ondrej Mosnacek
Software Engineer, Linux Security - SELinux kernel
Red Hat, Inc.
