Re: [GIT PULL] SELinux list corruption fix for 3.18

From: Paul Moore
Date: Wed Oct 15 2014 - 10:52:12 EST


On Wednesday, October 15, 2014 07:05:55 PM James Morris wrote:
> On Wed, 15 Oct 2014, Linus Torvalds wrote:
> > On Wed, Oct 15, 2014 at 7:27 AM, Linus Torvalds
> >
> > <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:
> > > No, the log is correct (the log always is, the back-merges can cause
> > > the trivial *diff* to be broken).
> >
> > To add some details: the commits in there are
> >
> > f6ed66087648 Merge branch 'stable-3.18' of
> >
> > git://git.infradead.org/users/pcmoore/selinux into f
> >
> > 7c66bdc72bc3 selinux: fix inode security list corruption
> > e7387395a07d selinux: normalize audit log formatting
> > 8497b78ecc9d selinux: cleanup error reporting in selinux_nlmsg_perm()
> > 6eb1ddc6bd3f selinux: make the netif cache namespace aware
> > 5e29532fed21 selinux: register nf hooks with single nf_register_hooks
> > call
> > 82341ad9b962 selinux: fix a problem with IPv6 traffic denials in
> >
> > selinux_ip_postroute()
> >
> > 04e8d6ab1fa6 selinux: Permit bounded transitions under NO_NEW_PRIVS or
> > NOSUID.>
> > and all but the list corruption fix seem to just be duplicate commits
> > of things I have already gotten elsewhere. Somebody cherry-picking
> > commits and duplicating them?
>
> Yep, I also already have these other changes under different commit IDs:
>
> commit 7b0d0b40cd78cadb525df760ee4cac151533c2b5
> Author: Stephen Smalley <sds@xxxxxxxxxxxxx>
> Date: Mon Aug 4 13:36:49 2014 -0400
>
> selinux: Permit bounded transitions under NO_NEW_PRIVS or NOSUID.
>
>
> Paul: do you have the above commit ID in your tree?

It would seem I've got the Midas touch this merge window, my apologies yet
again.

The short version is that everything should be fixed now, the stable-3.18
branch should be the single list corruption patch on top of the next branch I
sent previously. James, could you please re-pull from the stable-3.18 branch?

Thanks for your patience.

The slightly longer version is that when I created the stable-3.18 branch from
the next branch I used the '--clone' flag in stgit which updated the dates on
the patches which were new for #next (not "committed" in stgit) and caused the
commit IDs to change. Another lesson learned, I'm sorry it caused additional
problems.

-Paul

--
paul moore
www.paul-moore.com

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/