All of lore.kernel.org
 help / color / mirror / Atom feed
* linux-3.16.2 queue (3.16.1+)
@ 2014-08-28 15:16 Matt
  2014-08-28 15:22 ` Greg KH
  0 siblings, 1 reply; 9+ messages in thread
From: Matt @ 2014-08-28 15:16 UTC (permalink / raw)
  To: gregkh; +Cc: Linux Kernel, ReiserFS Mailing List

Hi Greg,


please consider adding the following 2 patches to 3.16.2:

Jan Kara (1):
      reiserfs: Fix use after free in journal teardown

Jeff Mahoney (1):
      reiserfs: fix corruption introduced by balance_leaf refactor



Reason/Related:

https://bugzilla.kernel.org/show_bug.cgi?id=83121

https://bugzilla.kernel.org/show_bug.cgi?id=83321

http://forums.gentoo.org/viewtopic-t-998538-postdays-0-postorder-asc-start-0.html


Many thanks in advance


Kind Regards

Matt

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: linux-3.16.2 queue (3.16.1+)
  2014-08-28 15:16 linux-3.16.2 queue (3.16.1+) Matt
@ 2014-08-28 15:22 ` Greg KH
  2014-08-28 15:27   ` Matt
  0 siblings, 1 reply; 9+ messages in thread
From: Greg KH @ 2014-08-28 15:22 UTC (permalink / raw)
  To: Matt; +Cc: Linux Kernel, ReiserFS Mailing List

On Thu, Aug 28, 2014 at 05:16:58PM +0200, Matt wrote:
> Hi Greg,
> 
> 
> please consider adding the following 2 patches to 3.16.2:
> 
> Jan Kara (1):
>       reiserfs: Fix use after free in journal teardown
> 
> Jeff Mahoney (1):
>       reiserfs: fix corruption introduced by balance_leaf refactor
> 
> 
> 
> Reason/Related:
> 
> https://bugzilla.kernel.org/show_bug.cgi?id=83121
> 
> https://bugzilla.kernel.org/show_bug.cgi?id=83321
> 
> http://forums.gentoo.org/viewtopic-t-998538-postdays-0-postorder-asc-start-0.html
> 
> 
> Many thanks in advance

I need git commit ids of these patches in Linus's tree, can you provide
those please?

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: linux-3.16.2 queue (3.16.1+)
  2014-08-28 15:22 ` Greg KH
@ 2014-08-28 15:27   ` Matt
  2014-08-28 15:32     ` Greg KH
  0 siblings, 1 reply; 9+ messages in thread
From: Matt @ 2014-08-28 15:27 UTC (permalink / raw)
  To: Greg KH; +Cc: ReiserFS Mailing List, Linux Kernel

On Thu, Aug 28, 2014 at 5:22 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Thu, Aug 28, 2014 at 05:16:58PM +0200, Matt wrote:
>> Hi Greg,
>>
>>
>> please consider adding the following 2 patches to 3.16.2:
>>
>> Jan Kara (1):
>>       reiserfs: Fix use after free in journal teardown
>>
>> Jeff Mahoney (1):
>>       reiserfs: fix corruption introduced by balance_leaf refactor
>>
>>
>>
>> Reason/Related:
>>
>> https://bugzilla.kernel.org/show_bug.cgi?id=83121
>>
>> https://bugzilla.kernel.org/show_bug.cgi?id=83321
>>
>> http://forums.gentoo.org/viewtopic-t-998538-postdays-0-postorder-asc-start-0.html
>>
>>
>> Many thanks in advance
>
> I need git commit ids of these patches in Linus's tree, can you provide
> those please?
>
> thanks,
>
> greg k-h


Sure:

https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=27d0e5bc85f3341b9ba66f0c23627cf9d7538c9d
reiserfs: fix corruption introduced by balance_leaf refactor


https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=01777836c87081e4f68c4a43c9abe6114805f91e
reiserfs: Fix use after free in journal teardown



are checkpatch warnings usually also fixed within stable releases ?

if yes:

https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=f3fb9e27325c4e1730440820ea8a1e9d9a5af709
fs/reiserfs/xattr.c: fix blank line missing after declarations

https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=17093991af4995c4b93f6d8ac63aab68fcd9e1be
fs/reiserfs: use linux/uaccess.h

https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=53872ed07786714bff3642ee9ee61afd3f4eb749
fs/reiserfs: replace not-standard %Lu/%Ld


Greetings

Matt

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: linux-3.16.2 queue (3.16.1+)
  2014-08-28 15:27   ` Matt
@ 2014-08-28 15:32     ` Greg KH
  2014-08-28 19:18       ` Matt
  0 siblings, 1 reply; 9+ messages in thread
From: Greg KH @ 2014-08-28 15:32 UTC (permalink / raw)
  To: Matt; +Cc: ReiserFS Mailing List, Linux Kernel

On Thu, Aug 28, 2014 at 05:27:27PM +0200, Matt wrote:
> On Thu, Aug 28, 2014 at 5:22 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> > On Thu, Aug 28, 2014 at 05:16:58PM +0200, Matt wrote:
> >> Hi Greg,
> >>
> >>
> >> please consider adding the following 2 patches to 3.16.2:
> >>
> >> Jan Kara (1):
> >>       reiserfs: Fix use after free in journal teardown
> >>
> >> Jeff Mahoney (1):
> >>       reiserfs: fix corruption introduced by balance_leaf refactor
> >>
> >>
> >>
> >> Reason/Related:
> >>
> >> https://bugzilla.kernel.org/show_bug.cgi?id=83121
> >>
> >> https://bugzilla.kernel.org/show_bug.cgi?id=83321
> >>
> >> http://forums.gentoo.org/viewtopic-t-998538-postdays-0-postorder-asc-start-0.html
> >>
> >>
> >> Many thanks in advance
> >
> > I need git commit ids of these patches in Linus's tree, can you provide
> > those please?
> >
> > thanks,
> >
> > greg k-h
> 
> 
> Sure:
> 
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=27d0e5bc85f3341b9ba66f0c23627cf9d7538c9d
> reiserfs: fix corruption introduced by balance_leaf refactor
> 
> 
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=01777836c87081e4f68c4a43c9abe6114805f91e
> reiserfs: Fix use after free in journal teardown
> 
> 
> 
> are checkpatch warnings usually also fixed within stable releases ?

No, not at all, please read Documentation/stable_kernel_patches.txt for
what is acceptable for stable kernel patches.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: linux-3.16.2 queue (3.16.1+)
  2014-08-28 15:32     ` Greg KH
@ 2014-08-28 19:18       ` Matt
  2014-09-07  0:47         ` Matt
  0 siblings, 1 reply; 9+ messages in thread
From: Matt @ 2014-08-28 19:18 UTC (permalink / raw)
  To: Greg KH; +Cc: ReiserFS Mailing List, Linux Kernel

On Thu, Aug 28, 2014 at 5:32 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Thu, Aug 28, 2014 at 05:27:27PM +0200, Matt wrote:
>> On Thu, Aug 28, 2014 at 5:22 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
>> > On Thu, Aug 28, 2014 at 05:16:58PM +0200, Matt wrote:
>> >> Hi Greg,
>> >>
>> >>
>> >> please consider adding the following 2 patches to 3.16.2:
>> >>
>> >> Jan Kara (1):
>> >>       reiserfs: Fix use after free in journal teardown
>> >>
>> >> Jeff Mahoney (1):
>> >>       reiserfs: fix corruption introduced by balance_leaf refactor
>> >>
>> >>
>> >>
>> >> Reason/Related:
>> >>
>> >> https://bugzilla.kernel.org/show_bug.cgi?id=83121
>> >>
>> >> https://bugzilla.kernel.org/show_bug.cgi?id=83321
>> >>
>> >> http://forums.gentoo.org/viewtopic-t-998538-postdays-0-postorder-asc-start-0.html
>> >>
>> >>
>> >> Many thanks in advance
>> >
>> > I need git commit ids of these patches in Linus's tree, can you provide
>> > those please?
>> >
>> > thanks,
>> >
>> > greg k-h
>>
>>
>> Sure:
>>
>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=27d0e5bc85f3341b9ba66f0c23627cf9d7538c9d
>> reiserfs: fix corruption introduced by balance_leaf refactor
>>
>>
>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=01777836c87081e4f68c4a43c9abe6114805f91e
>> reiserfs: Fix use after free in journal teardown
>>
>>
>>
>> are checkpatch warnings usually also fixed within stable releases ?
>
> No, not at all, please read Documentation/stable_kernel_patches.txt for
> what is acceptable for stable kernel patches.
>
> thanks,
>
> greg k-h


okay, will do

thanks for pointing that out


Regards

Matt

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: linux-3.16.2 queue (3.16.1+)
  2014-08-28 19:18       ` Matt
@ 2014-09-07  0:47         ` Matt
  2014-09-07  3:18           ` Greg KH
  0 siblings, 1 reply; 9+ messages in thread
From: Matt @ 2014-09-07  0:47 UTC (permalink / raw)
  To: Greg KH; +Cc: Linux Kernel, ReiserFS Mailing List

On Thu, Aug 28, 2014 at 9:18 PM, Matt <jackdachef@gmail.com> wrote:
> On Thu, Aug 28, 2014 at 5:32 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
>> On Thu, Aug 28, 2014 at 05:27:27PM +0200, Matt wrote:
>>> On Thu, Aug 28, 2014 at 5:22 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
>>> > On Thu, Aug 28, 2014 at 05:16:58PM +0200, Matt wrote:
>>> >> Hi Greg,
>>> >>
>>> >>
>>> >> please consider adding the following 2 patches to 3.16.2:
>>> >>
>>> >> Jan Kara (1):
>>> >>       reiserfs: Fix use after free in journal teardown
>>> >>
>>> >> Jeff Mahoney (1):
>>> >>       reiserfs: fix corruption introduced by balance_leaf refactor
>>> >>
>>> >>
>>> >>
>>> >> Reason/Related:
>>> >>
>>> >> https://bugzilla.kernel.org/show_bug.cgi?id=83121
>>> >>
>>> >> https://bugzilla.kernel.org/show_bug.cgi?id=83321
>>> >>
>>> >> http://forums.gentoo.org/viewtopic-t-998538-postdays-0-postorder-asc-start-0.html
>>> >>
>>> >>
>>> >> Many thanks in advance
>>> >
>>> > I need git commit ids of these patches in Linus's tree, can you provide
>>> > those please?
>>> >
>>> > thanks,
>>> >
>>> > greg k-h
>>>
>>>
>>> Sure:
>>>
>>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=27d0e5bc85f3341b9ba66f0c23627cf9d7538c9d
>>> reiserfs: fix corruption introduced by balance_leaf refactor
>>>
>>>
>>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=01777836c87081e4f68c4a43c9abe6114805f91e
>>> reiserfs: Fix use after free in journal teardown
>>>
>>>
>>>
>>> are checkpatch warnings usually also fixed within stable releases ?
>>
>> No, not at all, please read Documentation/stable_kernel_patches.txt for
>> what is acceptable for stable kernel patches.
>>
>> thanks,
>>
>> greg k-h
>
>
> okay, will do
>
> thanks for pointing that out
>
>
> Regards
>
> Matt

Hi Greg,

could you please add the above mentioned two patches

https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=27d0e5bc85f3341b9ba66f0c23627cf9d7538c9d
reiserfs: fix corruption introduced by balance_leaf refactor


https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=01777836c87081e4f68c4a43c9abe6114805f91e
reiserfs: Fix use after free in journal teardown

in next stable (3.16.3) kernel ?

more and more people seem to be affected by the data corruption
introduced by the recent changes.


Reading through Documentation/stable_kernel_rules.txt,
http://cwe.mitre.org/data/definitions/416.html and
http://www.hpenterprisesecurity.com/vulncat/en/vulncat/cpp/use_after_free.html

both patches seem relevant enough (concerning data integrity
filesystem-wise and security) to be included for the stable branch


Thanks & Regards

Matt

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: linux-3.16.2 queue (3.16.1+)
  2014-09-07  0:47         ` Matt
@ 2014-09-07  3:18           ` Greg KH
  2014-09-11  4:29             ` Jeff Mahoney
  0 siblings, 1 reply; 9+ messages in thread
From: Greg KH @ 2014-09-07  3:18 UTC (permalink / raw)
  To: Matt; +Cc: Linux Kernel, ReiserFS Mailing List

On Sun, Sep 07, 2014 at 02:47:55AM +0200, Matt wrote:
> On Thu, Aug 28, 2014 at 9:18 PM, Matt <jackdachef@gmail.com> wrote:
> > On Thu, Aug 28, 2014 at 5:32 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> >> On Thu, Aug 28, 2014 at 05:27:27PM +0200, Matt wrote:
> >>> On Thu, Aug 28, 2014 at 5:22 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> >>> > On Thu, Aug 28, 2014 at 05:16:58PM +0200, Matt wrote:
> >>> >> Hi Greg,
> >>> >>
> >>> >>
> >>> >> please consider adding the following 2 patches to 3.16.2:
> >>> >>
> >>> >> Jan Kara (1):
> >>> >>       reiserfs: Fix use after free in journal teardown
> >>> >>
> >>> >> Jeff Mahoney (1):
> >>> >>       reiserfs: fix corruption introduced by balance_leaf refactor
> >>> >>
> >>> >>
> >>> >>
> >>> >> Reason/Related:
> >>> >>
> >>> >> https://bugzilla.kernel.org/show_bug.cgi?id=83121
> >>> >>
> >>> >> https://bugzilla.kernel.org/show_bug.cgi?id=83321
> >>> >>
> >>> >> http://forums.gentoo.org/viewtopic-t-998538-postdays-0-postorder-asc-start-0.html
> >>> >>
> >>> >>
> >>> >> Many thanks in advance
> >>> >
> >>> > I need git commit ids of these patches in Linus's tree, can you provide
> >>> > those please?
> >>> >
> >>> > thanks,
> >>> >
> >>> > greg k-h
> >>>
> >>>
> >>> Sure:
> >>>
> >>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=27d0e5bc85f3341b9ba66f0c23627cf9d7538c9d
> >>> reiserfs: fix corruption introduced by balance_leaf refactor
> >>>
> >>>
> >>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=01777836c87081e4f68c4a43c9abe6114805f91e
> >>> reiserfs: Fix use after free in journal teardown
> >>>
> >>>
> >>>
> >>> are checkpatch warnings usually also fixed within stable releases ?
> >>
> >> No, not at all, please read Documentation/stable_kernel_patches.txt for
> >> what is acceptable for stable kernel patches.
> >>
> >> thanks,
> >>
> >> greg k-h
> >
> >
> > okay, will do
> >
> > thanks for pointing that out
> >
> >
> > Regards
> >
> > Matt
> 
> Hi Greg,
> 
> could you please add the above mentioned two patches
> 
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=27d0e5bc85f3341b9ba66f0c23627cf9d7538c9d
> reiserfs: fix corruption introduced by balance_leaf refactor
> 
> 
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=01777836c87081e4f68c4a43c9abe6114805f91e
> reiserfs: Fix use after free in journal teardown
> 
> in next stable (3.16.3) kernel ?
> 
> more and more people seem to be affected by the data corruption
> introduced by the recent changes.
> 
> 
> Reading through Documentation/stable_kernel_rules.txt,
> http://cwe.mitre.org/data/definitions/416.html and
> http://www.hpenterprisesecurity.com/vulncat/en/vulncat/cpp/use_after_free.html
> 
> both patches seem relevant enough (concerning data integrity
> filesystem-wise and security) to be included for the stable branch

I'll queue this up when I get a chance, there are over 300 patches
pending for the stable kernels right now :(

Also, in the future, always cc stable@vger.kernel.org for any stable
requests so that they don't get lost.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: linux-3.16.2 queue (3.16.1+)
  2014-09-07  3:18           ` Greg KH
@ 2014-09-11  4:29             ` Jeff Mahoney
  2014-09-12 21:29               ` Greg KH
  0 siblings, 1 reply; 9+ messages in thread
From: Jeff Mahoney @ 2014-09-11  4:29 UTC (permalink / raw)
  To: Greg KH, Matt; +Cc: Linux Kernel, ReiserFS Mailing List

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 9/6/14, 11:18 PM, Greg KH wrote:
> On Sun, Sep 07, 2014 at 02:47:55AM +0200, Matt wrote:
>> On Thu, Aug 28, 2014 at 9:18 PM, Matt <jackdachef@gmail.com>
>> wrote:
>>> On Thu, Aug 28, 2014 at 5:32 PM, Greg KH
>>> <gregkh@linuxfoundation.org> wrote:
>>>> On Thu, Aug 28, 2014 at 05:27:27PM +0200, Matt wrote:
>>>>> On Thu, Aug 28, 2014 at 5:22 PM, Greg KH
>>>>> <gregkh@linuxfoundation.org> wrote:
>>>>>> On Thu, Aug 28, 2014 at 05:16:58PM +0200, Matt wrote:
>>>>>>> Hi Greg,
>>>>>>> 
>>>>>>> 
>>>>>>> please consider adding the following 2 patches to
>>>>>>> 3.16.2:
>>>>>>> 
>>>>>>> Jan Kara (1): reiserfs: Fix use after free in journal
>>>>>>> teardown
>>>>>>> 
>>>>>>> Jeff Mahoney (1): reiserfs: fix corruption introduced
>>>>>>> by balance_leaf refactor
>>>>>>> 
>>>>>>> 
>>>>>>> 
>>>>>>> Reason/Related:
>>>>>>> 
>>>>>>> https://bugzilla.kernel.org/show_bug.cgi?id=83121
>>>>>>> 
>>>>>>> https://bugzilla.kernel.org/show_bug.cgi?id=83321
>>>>>>> 
>>>>>>> http://forums.gentoo.org/viewtopic-t-998538-postdays-0-postorder-asc-start-0.html
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> 
Many thanks in advance
>>>>>> 
>>>>>> I need git commit ids of these patches in Linus's tree,
>>>>>> can you provide those please?
>>>>>> 
>>>>>> thanks,
>>>>>> 
>>>>>> greg k-h
>>>>> 
>>>>> 
>>>>> Sure:
>>>>> 
>>>>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=27d0e5bc85f3341b9ba66f0c23627cf9d7538c9d
>>>>>
>>>>> 
reiserfs: fix corruption introduced by balance_leaf refactor
>>>>> 
>>>>> 
>>>>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=01777836c87081e4f68c4a43c9abe6114805f91e
>>>>>
>>>>> 
reiserfs: Fix use after free in journal teardown
>>>>> 
>>>>> 
>>>>> 
>>>>> are checkpatch warnings usually also fixed within stable
>>>>> releases ?
>>>> 
>>>> No, not at all, please read
>>>> Documentation/stable_kernel_patches.txt for what is
>>>> acceptable for stable kernel patches.
>>>> 
>>>> thanks,
>>>> 
>>>> greg k-h
>>> 
>>> 
>>> okay, will do
>>> 
>>> thanks for pointing that out
>>> 
>>> 
>>> Regards
>>> 
>>> Matt
>> 
>> Hi Greg,
>> 
>> could you please add the above mentioned two patches
>> 
>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=27d0e5bc85f3341b9ba66f0c23627cf9d7538c9d
>>
>> 
reiserfs: fix corruption introduced by balance_leaf refactor
>> 
>> 
>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=01777836c87081e4f68c4a43c9abe6114805f91e
>>
>> 
reiserfs: Fix use after free in journal teardown
>> 
>> in next stable (3.16.3) kernel ?
>> 
>> more and more people seem to be affected by the data corruption 
>> introduced by the recent changes.
>> 
>> 
>> Reading through Documentation/stable_kernel_rules.txt, 
>> http://cwe.mitre.org/data/definitions/416.html and 
>> http://www.hpenterprisesecurity.com/vulncat/en/vulncat/cpp/use_after_free.html
>>
>>
>> 
both patches seem relevant enough (concerning data integrity
>> filesystem-wise and security) to be included for the stable
>> branch
> 
> I'll queue this up when I get a chance, there are over 300 patches 
> pending for the stable kernels right now :(
> 
> Also, in the future, always cc stable@vger.kernel.org for any
> stable requests so that they don't get lost.

Hi Greg -

27d0e5bc85f3341b9ba66f0c23627cf9d7538c9d
Author: Jeff Mahoney <jeffm@suse.com>
Date:   Mon Aug 4 19:51:47 2014 -0400

    reiserfs: fix corruption introduced by balance_leaf refactor

    Commits f1f007c308e (reiserfs: balance_leaf refactor, pull out
    balance_leaf_insert_left) and cf22df182bf (reiserfs: balance_leaf
    refactor, pull out balance_leaf_paste_left) missed that the `body'
    pointer was getting repositioned. Subsequent users of the pointer
    would expect it to be repositioned, and as a result, parts of the
    tree would get overwritten. The most common observed corruption
    is indirect block pointers being overwritten.

    Since the body value isn't actually used anymore in the called
routines,
    we can pass back the offset it should be shifted. We constify the body
    and ih pointers in the balance_leaf as a mostly-free preventative
measure.

    Cc: <stable@vger.kernel.org> # 3.16
    Reported-and-tested-by: Jeff Chua <jeff.chua.linux@gmail.com>
    Signed-off-by: Jeff Mahoney <jeffm@suse.com>
    Signed-off-by: Jan Kara <jack@suse.cz>

Should there have been more? I thought it was enough to add the Cc
tag. This one has been in the tree, with the tags and with
"corruption" in the Subject since 13 Aug. I know you're busy but this
seems like a pretty obvious candidate for stable inclusion.

- -Jeff

- -- 
Jeff Mahoney
SUSE Labs
-----BEGIN PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0.19 (Darwin)

iQIcBAEBAgAGBQJUESUpAAoJEB57S2MheeWyt9oQAIEnvZPojErvzzv4IcvVheSI
Ju1XChkU4YDRW3W2e8PEjAhiPd1dMP7aEJvfq6AxlKYAYENaS/S2LdSbBbeVctFa
1VwBVakDkmHduVcb2hl3ldIQlHRW0w/q/fSk+NKZavANS/maIK/mj2HE8S3Op17C
iGsGZiluqaYp56yPHJK7XDorpWFoCVXIPlHUbec8lIxnyPqeytHo2W5UtfZZVeN3
BfGICzR57i7YjOtQ/lsmusiUjp7Ym4REKX1GGnIcZ1Po5F8oX4phMVaUR0gR1NSA
eYBcTyH245iWTQBFqE9D5AR0pHLnmi6EySEbNIWU3w0OYffDCBpqU7A7Dm5O2kng
caIlNuf4TMEp7QzVC8hxCL61nxBWJ6L2RQ9NkOg9zLHXdaWhJSjHl7TdRUPV/C3V
RzNCZEWvqEpMoju145Wez7JlcE/GlsBclNFGBqypEWN364B/MprKe5vhpeXJ+1H2
yUq/qKlgQLZe5uPCwMdcyAB3xTX8mIzG4nz8RWez6WPjhAlb82xtBSl0btWjSnVM
4YlWy/5jCWgyjXzrM3hd8P3SJi+l69rVUE+UcMvOqHq3oCFBddhlUh9tHM7pn9tH
sXTo8f8s9Pe7+HvbA0bwtwbwTQ8tNxn87ovuVnAO86RLmeeM7HCfqBU/4lKEZ0WE
TG7eSSLjQVBxo+kQCZOo
=OOOa
-----END PGP SIGNATURE-----

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: linux-3.16.2 queue (3.16.1+)
  2014-09-11  4:29             ` Jeff Mahoney
@ 2014-09-12 21:29               ` Greg KH
  0 siblings, 0 replies; 9+ messages in thread
From: Greg KH @ 2014-09-12 21:29 UTC (permalink / raw)
  To: Jeff Mahoney; +Cc: Matt, Linux Kernel, ReiserFS Mailing List

On Thu, Sep 11, 2014 at 12:29:30AM -0400, Jeff Mahoney wrote:
> On 9/6/14, 11:18 PM, Greg KH wrote:
> > On Sun, Sep 07, 2014 at 02:47:55AM +0200, Matt wrote:
> >> On Thu, Aug 28, 2014 at 9:18 PM, Matt <jackdachef@gmail.com>
> >> wrote:
> >>> On Thu, Aug 28, 2014 at 5:32 PM, Greg KH
> >>> <gregkh@linuxfoundation.org> wrote:
> >>>> On Thu, Aug 28, 2014 at 05:27:27PM +0200, Matt wrote:
> >>>>> On Thu, Aug 28, 2014 at 5:22 PM, Greg KH
> >>>>> <gregkh@linuxfoundation.org> wrote:
> >>>>>> On Thu, Aug 28, 2014 at 05:16:58PM +0200, Matt wrote:
> >>>>>>> Hi Greg,
> >>>>>>> 
> >>>>>>> 
> >>>>>>> please consider adding the following 2 patches to
> >>>>>>> 3.16.2:
> >>>>>>> 
> >>>>>>> Jan Kara (1): reiserfs: Fix use after free in journal
> >>>>>>> teardown
> >>>>>>> 
> >>>>>>> Jeff Mahoney (1): reiserfs: fix corruption introduced
> >>>>>>> by balance_leaf refactor
> >>>>>>> 
> >>>>>>> 
> >>>>>>> 
> >>>>>>> Reason/Related:
> >>>>>>> 
> >>>>>>> https://bugzilla.kernel.org/show_bug.cgi?id=83121
> >>>>>>> 
> >>>>>>> https://bugzilla.kernel.org/show_bug.cgi?id=83321
> >>>>>>> 
> >>>>>>> http://forums.gentoo.org/viewtopic-t-998538-postdays-0-postorder-asc-start-0.html
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>> 
> Many thanks in advance
> >>>>>> 
> >>>>>> I need git commit ids of these patches in Linus's tree,
> >>>>>> can you provide those please?
> >>>>>> 
> >>>>>> thanks,
> >>>>>> 
> >>>>>> greg k-h
> >>>>> 
> >>>>> 
> >>>>> Sure:
> >>>>> 
> >>>>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=27d0e5bc85f3341b9ba66f0c23627cf9d7538c9d
> >>>>>
> >>>>> 
> reiserfs: fix corruption introduced by balance_leaf refactor
> >>>>> 
> >>>>> 
> >>>>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=01777836c87081e4f68c4a43c9abe6114805f91e
> >>>>>
> >>>>> 
> reiserfs: Fix use after free in journal teardown
> >>>>> 
> >>>>> 
> >>>>> 
> >>>>> are checkpatch warnings usually also fixed within stable
> >>>>> releases ?
> >>>> 
> >>>> No, not at all, please read
> >>>> Documentation/stable_kernel_patches.txt for what is
> >>>> acceptable for stable kernel patches.
> >>>> 
> >>>> thanks,
> >>>> 
> >>>> greg k-h
> >>> 
> >>> 
> >>> okay, will do
> >>> 
> >>> thanks for pointing that out
> >>> 
> >>> 
> >>> Regards
> >>> 
> >>> Matt
> >> 
> >> Hi Greg,
> >> 
> >> could you please add the above mentioned two patches
> >> 
> >> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=27d0e5bc85f3341b9ba66f0c23627cf9d7538c9d
> >>
> >> 
> reiserfs: fix corruption introduced by balance_leaf refactor
> >> 
> >> 
> >> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=01777836c87081e4f68c4a43c9abe6114805f91e
> >>
> >> 
> reiserfs: Fix use after free in journal teardown
> >> 
> >> in next stable (3.16.3) kernel ?
> >> 
> >> more and more people seem to be affected by the data corruption 
> >> introduced by the recent changes.
> >> 
> >> 
> >> Reading through Documentation/stable_kernel_rules.txt, 
> >> http://cwe.mitre.org/data/definitions/416.html and 
> >> http://www.hpenterprisesecurity.com/vulncat/en/vulncat/cpp/use_after_free.html
> >>
> >>
> >> 
> both patches seem relevant enough (concerning data integrity
> >> filesystem-wise and security) to be included for the stable
> >> branch
> > 
> > I'll queue this up when I get a chance, there are over 300 patches 
> > pending for the stable kernels right now :(
> > 
> > Also, in the future, always cc stable@vger.kernel.org for any
> > stable requests so that they don't get lost.
> 
> Hi Greg -
> 
> 27d0e5bc85f3341b9ba66f0c23627cf9d7538c9d
> Author: Jeff Mahoney <jeffm@suse.com>
> Date:   Mon Aug 4 19:51:47 2014 -0400
> 
>     reiserfs: fix corruption introduced by balance_leaf refactor
> 
>     Commits f1f007c308e (reiserfs: balance_leaf refactor, pull out
>     balance_leaf_insert_left) and cf22df182bf (reiserfs: balance_leaf
>     refactor, pull out balance_leaf_paste_left) missed that the `body'
>     pointer was getting repositioned. Subsequent users of the pointer
>     would expect it to be repositioned, and as a result, parts of the
>     tree would get overwritten. The most common observed corruption
>     is indirect block pointers being overwritten.
> 
>     Since the body value isn't actually used anymore in the called
> routines,
>     we can pass back the offset it should be shifted. We constify the body
>     and ih pointers in the balance_leaf as a mostly-free preventative
> measure.
> 
>     Cc: <stable@vger.kernel.org> # 3.16
>     Reported-and-tested-by: Jeff Chua <jeff.chua.linux@gmail.com>
>     Signed-off-by: Jeff Mahoney <jeffm@suse.com>
>     Signed-off-by: Jan Kara <jack@suse.cz>
> 
> Should there have been more? I thought it was enough to add the Cc
> tag. This one has been in the tree, with the tags and with
> "corruption" in the Subject since 13 Aug. I know you're busy but this
> seems like a pretty obvious candidate for stable inclusion.

You marked this one just fine, it's just that, again, I have over 300+
patches in the "marked for stable" queue right now, this patch is in
good company...

greg k-h

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2014-09-12 21:29 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-28 15:16 linux-3.16.2 queue (3.16.1+) Matt
2014-08-28 15:22 ` Greg KH
2014-08-28 15:27   ` Matt
2014-08-28 15:32     ` Greg KH
2014-08-28 19:18       ` Matt
2014-09-07  0:47         ` Matt
2014-09-07  3:18           ` Greg KH
2014-09-11  4:29             ` Jeff Mahoney
2014-09-12 21:29               ` Greg KH

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.