Subject: Re: [PATCH net v2 0/2] net: netem: fix further issues with packet corruption

From: Stephen Hemminger <[email-masked]>
Date: Fri, 18 Oct 2019 09:36:14 -0700

On Fri, 18 Oct 2019 09:16:56 -0700
Jakub Kicinski <[email-masked]> wrote:

> Hi!
> 
> This set is fixing two more issues with the netem packet corruption.
> 
> First patch (which was previously posted) avoids NULL pointer dereference
> if the first frame gets freed due to allocation or checksum failure.
> v2 improves the clarity of the code a little as requested by Cong.
> 
> Second patch ensures we don't return SUCCESS if the frame was in fact
> dropped. Thanks to this commit message for patch 1 no longer needs the
> "this will still break with a single-frame failure" disclaimer.
> 
> Jakub Kicinski (2):
>   net: netem: fix error path for corrupted GSO frames
>   net: netem: correct the parent's backlog when corrupted packet was
>     dropped
> 
>  net/sched/sch_netem.c | 11 ++++++++---
>  1 file changed, 8 insertions(+), 3 deletions(-)
> 

LGTM

Acked-by: Stephen Hemminger <[email-masked]>