hv_netvsc: fix netvsc_send_completion to avoid multiple message length checks
authorSonia Sharma <sonia.sharma@linux.microsoft.com>
Mon, 9 Oct 2023 08:00:16 +0000 (01:00 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 11 Oct 2023 08:19:01 +0000 (09:19 +0100)
commit9bae5b05502210f7fb5ac24874ec2e0747401b6b
treef34f4b22c1765960b016c8208de8b4af46bf5590
parent0e594c1f5d0f0f916f7512cac92ef3ba8c4ace1c
hv_netvsc: fix netvsc_send_completion to avoid multiple message length checks

The switch statement in netvsc_send_completion() is incorrectly validating
the length of incoming network packets by falling through to the next case.
Avoid the fallthrough. Instead break after a case match and then process
the complete() call.
The current code has not caused any known failures. But nonetheless, the
code should be corrected as a different ordering of the switch cases might
cause a length check to fail when it should not.

Signed-off-by: Sonia Sharma <sonia.sharma@linux.microsoft.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/hyperv/netvsc.c