Where is this patch buried?

4 messages, 2 authors, 2021-02-08 · open the first message on its own page

Where is this patch buried?

From: Schmid, Carsten <hidden>
Date: 2021-02-08 08:13:49

Hi Greg,

in kernel 4.14 i have seen a NULL pointer deref in
[65064.613465] RIP: 0010:ip_route_output_key_hash_rcu+0x755/0x850
(i have a core dump and detailed analysis)

That looks like this patch could have prevented it:

https://www.spinics.net/lists/stable-commits/msg133055.html

this one was queued for 4.14, but i can't find it in git tree?
Any idea who/what buried this one?

In 4.19 it is present.

Because our customer uses 4.14 (going to 4.14.212 in a few days) i kindly want to ask why this patch hasn't entered 4.14.

Best regards
Carsten
-----------------
Mentor Graphics (Deutschland) GmbH, Arnulfstrasse 201, 80634 München Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Frank Thürauf

Re: Where is this patch buried?

From: Greg KH <gregkh@linuxfoundation.org>
Date: 2021-02-08 08:52:30

On Mon, Feb 08, 2021 at 08:12:29AM +0000, Schmid, Carsten wrote:
Hi Greg,

in kernel 4.14 i have seen a NULL pointer deref in
[65064.613465] RIP: 0010:ip_route_output_key_hash_rcu+0x755/0x850
(i have a core dump and detailed analysis)

That looks like this patch could have prevented it:

https://www.spinics.net/lists/stable-commits/msg133055.html

this one was queued for 4.14, but i can't find it in git tree?
Any idea who/what buried this one?

In 4.19 it is present.

Because our customer uses 4.14 (going to 4.14.212 in a few days) i kindly want to ask why this patch hasn't entered 4.14.
Because it breaks the build?  Try it yourself and see what happens :)

I will gladly take a working backport if you can submit it.

thanks,

greg k-h

AW: Where is this patch buried?

From: Schmid, Carsten <hidden>
Date: 2021-02-08 09:28:21

quoted
Hi Greg,

in kernel 4.14 i have seen a NULL pointer deref in
[65064.613465] RIP: 0010:ip_route_output_key_hash_rcu+0x755/0x850
(i have a core dump and detailed analysis)

That looks like this patch could have prevented it:

https://www.spinics.net/lists/stable-commits/msg133055.html

this one was queued for 4.14, but i can't find it in git tree?
Any idea who/what buried this one?

In 4.19 it is present.

Because our customer uses 4.14 (going to 4.14.212 in a few days) i kindly want to
ask why this patch hasn't entered 4.14.
Because it breaks the build?  Try it yourself and see what happens :)
yep. rt_add_uncached_list is implemented _after_ the call :-(
I will gladly take a working backport if you can submit it.
Please find it attached - i needed to move rt_add_uncached_list before
the rt_cache_route, nothing else changed.
This is for 4.14 only, as other kernels do have this patch.

I can't reproduce the crash at will, but at least i could compile and flash it on my target.
And the target comes up .. hopefully the testing in other/newer kernels
done by the developers of the patch is also valid for 4.14.
thanks,

greg k-h
Thanks,
Carsten
-----------------
Mentor Graphics (Deutschland) GmbH, Arnulfstrasse 201, 80634 München Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Frank Thürauf

Re: Where is this patch buried?

From: Greg KH <gregkh@linuxfoundation.org>
Date: 2021-02-08 09:54:46

On Mon, Feb 08, 2021 at 09:24:11AM +0000, Schmid, Carsten wrote:
quoted
quoted
Hi Greg,

in kernel 4.14 i have seen a NULL pointer deref in
[65064.613465] RIP: 0010:ip_route_output_key_hash_rcu+0x755/0x850
(i have a core dump and detailed analysis)

That looks like this patch could have prevented it:

https://www.spinics.net/lists/stable-commits/msg133055.html

this one was queued for 4.14, but i can't find it in git tree?
Any idea who/what buried this one?

In 4.19 it is present.

Because our customer uses 4.14 (going to 4.14.212 in a few days) i kindly want to
ask why this patch hasn't entered 4.14.
Because it breaks the build?  Try it yourself and see what happens :)
yep. rt_add_uncached_list is implemented _after_ the call :-(
quoted
I will gladly take a working backport if you can submit it.
Please find it attached - i needed to move rt_add_uncached_list before
the rt_cache_route, nothing else changed.
This is for 4.14 only, as other kernels do have this patch.

I can't reproduce the crash at will, but at least i could compile and flash it on my target.
And the target comes up .. hopefully the testing in other/newer kernels
done by the developers of the patch is also valid for 4.14.
Thanks, now queued up.

greg k-h
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help