Hi On Mon, Sep 5, 2022 at 3:27 PM Paolo Bonzini <[email protected]> wrote:
> Reported by Coverity as CID 1432543. > > Signed-off-by: Paolo Bonzini <[email protected]> > --- > tests/unit/test-qga.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/tests/unit/test-qga.c b/tests/unit/test-qga.c > index a05a4628ed..d27ff94d13 100644 > --- a/tests/unit/test-qga.c > +++ b/tests/unit/test-qga.c > @@ -32,6 +32,7 @@ static int connect_qga(char *path) > g_usleep(G_USEC_PER_SEC); > } > if (i++ == 10) { > + close(s); > We may want to be a bit safer and use g_close(), which handles EINTR too. Anyway Reviewed-by: Marc-André Lureau <[email protected]> > return -1; > } > } while (ret == -1); > -- > 2.37.2 > > > -- Marc-André Lureau
