Messages in this thread | | | Date | Fri, 22 Jun 2012 17:02:37 -0700 (PDT) | Subject | Re: Bug in net/ipv6/ip6_fib.c:fib6_dump_table() | From | David Miller <> |
| |
From: Eric Dumazet <eric.dumazet@gmail.com> Date: Fri, 22 Jun 2012 20:13:05 +0200
> On Fri, 2012-06-22 at 08:44 -0500, Josh Hunt wrote: > >> Ahh. That makes sense and is what Alexey said before I just didn't put >> it all together. So we are OK reverting this patch? I cannot find a path >> where the walker's pointers are updated without the tb6_lock write_lock. >> > > There was a bug somewhere, not sure we want to NULL dereference again.
Well:
1) Patrick McHardy has been inactive for a while, so do not expect any insight from him.
2) Ben Greear isn't even on the CC: list of this discussion yet he appears to be the person who reproduced the crash way back then and is listed in the Tested-by tag of the commit.
As a result we aren't likely to get any insight from the one person who actually could hit the crash.
I'm inclined to just revert simply because we have people active who can reproduce regressions introduced by this change and nobody can understand why the change is even necessary.
| |