On Fri, 18 Oct 2019 09:16:56 -0700 Jakub Kicinski <jakub.kicin...@netronome.com> 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 <step...@networkplumber.org>
