[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1218440487.32543.7.camel@ukdev-lin-jag>
Date: Mon, 11 Aug 2008 08:41:27 +0100
From: John Gumb <john.gumb@...dberg.com>
To: Brian Haley <brian.haley@...com>
Cc: Alexey Dobriyan <adobriyan@...il.com>,
linux-kernel@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: OOPS, ip -f inet6 route get fec0::1,
linux-2.6.26, ip6_route_output, rt6_fill_node+0x175
looks good to me.
thanks people.
John
On Fri, 2008-08-08 at 00:57 -0400, Brian Haley wrote:
> Alexey Dobriyan wrote:
> > On Thu, Aug 07, 2008 at 07:00:56PM +0200, John Gumb wrote:
> >> Scenario: no ipv6 default route set.
> >
> >> # ip -f inet6 route get fec0::1
> >>
> >> BUG: unable to handle kernel NULL pointer dereference at 00000000
> >> IP: [<c0369b85>] rt6_fill_node+0x175/0x3b0
> >> EIP is at rt6_fill_node+0x175/0x3b0
> >
> > 0xffffffff80424dd3 is in rt6_fill_node (net/ipv6/route.c:2191).
> > 2186 } else
> > 2187 #endif
> > 2188 NLA_PUT_U32(skb, RTA_IIF, iif);
> > 2189 } else if (dst) {
> > 2190 struct in6_addr saddr_buf;
> > 2191 ====> if (ipv6_dev_get_saddr(ip6_dst_idev(&rt->u.dst)->dev,
> > ^^^^^^^^^^^^^^^^^^^^^^^^
> > NULL
> >
> > 2192 dst, 0, &saddr_buf) == 0)
> > 2193 NLA_PUT(skb, RTA_PREFSRC, 16, &saddr_buf);
> > 2194 }
>
> The commit that changed this can't be reverted easily, but the patch
> below works for me.
>
> Fix NULL de-reference in rt6_fill_node() when there's no IPv6 input
> device present in the dst entry.
>
> Signed-off-by: Brian Haley <brian.haley@...com>
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists