Messages in this thread | | | Date | Thu, 11 May 2017 21:42:33 -0400 (EDT) | Subject | Re: [PATCH][V2] ethernet: aquantia: remove redundant checks on error status | From | David Miller <> |
| |
From: Colin King <colin.king@canonical.com> Date: Thu, 11 May 2017 19:29:40 +0100
> From: Colin Ian King <colin.king@canonical.com> > > The error status err is initialized as zero and then being checked > several times to see if it is less than zero even when it has not > been updated. It may seem that the err should be assigned to the > return code of the call to the various *offload_en_set calls and > then we check for failure, however, these functions are void and > never actually return any status. > > Since these error checks are redundant we can remove these > as well as err and the error exit label err_exit. > > Detected by CoverityScan, CID#1398313 and CID#1398306 ("Logically > dead code") > > Signed-off-by: Colin Ian King <colin.king@canonical.com>
Applied, thank you.
| |