Am 15.10.2020 um 14:13 hat Elena Afanasova geschrieben: > Since handle_dependencies() returns 0 or -EAGAIN the following case can be > removed. > Spotted by PVS-Studio. > > Signed-off-by: Elena Afanasova <[email protected]> > --- > block/qcow2-cluster.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/block/qcow2-cluster.c b/block/qcow2-cluster.c > index aa87d3e99b..e2e0db0cc9 100644 > --- a/block/qcow2-cluster.c > +++ b/block/qcow2-cluster.c > @@ -1806,8 +1806,6 @@ again: > * structs before starting over. */ > assert(*m == NULL); > goto again; > - } else if (ret < 0) { > - return ret; > } else if (cur_bytes == 0) { > break; > } else {
Technically correct, but let's keep it for anyone who will return a different error code in the future. I think we should always check for any error even if it is currently not produced. Kevin
