lkml.org 
[lkml]   [2020]   [Sep]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 4.19 23/37] ipv4: Update exception handling for multipath routes via same device
On Sat 2020-09-26 17:46:35, Greg Kroah-Hartman wrote:
> On Fri, Sep 25, 2020 at 06:51:34PM +0200, Pavel Machek wrote:
> > Hi!
> >
> > > [ Upstream commit 2fbc6e89b2f1403189e624cabaf73e189c5e50c6 ]
> > >
> > > Kfir reported that pmtu exceptions are not created properly for
> > > deployments where multipath routes use the same device.
> >
> > This is mismerged (in a way that does not affect functionality):
> >
> >
> > > @@ -779,6 +779,8 @@ static void __ip_do_redirect(struct rtab
> > > if (fib_lookup(net, fl4, &res, 0) == 0) {
> > > struct fib_nh *nh = &FIB_RES_NH(res);
> > >
> > > + fib_select_path(net, &res, fl4, skb);
> > > + nh = &FIB_RES_NH(res);
> > > update_or_create_fnhe(nh, fl4->daddr, new_gw,
> > > 0, false,
> >
> > nh is assigned value that is never used. Mainline patch removes the
> > assignment (but variable has different type).
> >
> > 4.19 should delete the assignment, too.
>
> Ah, good catch, I'll merge this in, thanks.

Thank you!
Pavel

--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2020-09-29 14:27    [W:0.084 / U:0.028 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site