On Wed, Oct 05, 2016 at 08:31:52PM +0300, Eyal Birger wrote: > On Wed, Oct 5, 2016 at 8:23 PM, Jiri Benc <jb...@redhat.com> wrote: > > On Wed, 5 Oct 2016 17:18:08 +0300, Eyal Birger wrote: > >> I think at this point, 'eth' may point to a freed packet. > > > > It may but how does that matter? eth is not used beyond that point. > > Definitely a nit. For sure not critical. > > Just seemed less future safe to keep a pointer to an old packet lying around.
I agree. Alternatively refresh the eth pointer.
