Re: [PATCH] nbd: Drop dead code spotted by Coverity

2022-05-30 Thread Kevin Wolf
Am 16.05.2022 um 23:05 hat Eric Blake geschrieben: > CID 1488362 points out that the second 'rc >= 0' check is now dead > code. > > Reported-by: Peter Maydell > Fixes: 172f5f1a40(nbd: remove peppering of nbd_client_connected) > Signed-off-by: Eric Blake Thanks, applied to the block branch. Kev

Re: [PATCH] nbd: Drop dead code spotted by Coverity

2022-05-17 Thread Vladimir Sementsov-Ogievskiy
17.05.2022 00:05, Eric Blake wrote: CID 1488362 points out that the second 'rc >= 0' check is now dead code. Reported-by: Peter Maydell Fixes: 172f5f1a40(nbd: remove peppering of nbd_client_connected) Signed-off-by: Eric Blake --- block/nbd.c | 8 ++-- 1 file changed, 2 insertions(+), 6

Re: [PATCH] nbd: Drop dead code spotted by Coverity

2022-05-17 Thread Peter Maydell
On Mon, 16 May 2022 at 22:05, Eric Blake wrote: > > CID 1488362 points out that the second 'rc >= 0' check is now dead > code. > > Reported-by: Peter Maydell > Fixes: 172f5f1a40(nbd: remove peppering of nbd_client_connected) > Signed-off-by: Eric Blake > --- > block/nbd.c | 8 ++-- Reviewed-

[PATCH] nbd: Drop dead code spotted by Coverity

2022-05-16 Thread Eric Blake
CID 1488362 points out that the second 'rc >= 0' check is now dead code. Reported-by: Peter Maydell Fixes: 172f5f1a40(nbd: remove peppering of nbd_client_connected) Signed-off-by: Eric Blake --- block/nbd.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/block/nbd.c