Re: [Spice-devel] [PATCH linux vdagent 2/2] Avoid leaking udscs file descriptor

2017-11-29 Thread Frediano Ziglio
> > This is a finding from coverity. In practice it has very little impact maybe Coverity (with capital) ? > since returning NULL from this function results in the process existing > with an error. > > Signed-off-by: Jonathon Jongsma > --- > src/udscs.c | 2 ++ > 1 file changed, 2 insertions

[Spice-devel] [PATCH linux vdagent 2/2] Avoid leaking udscs file descriptor

2017-11-29 Thread Jonathon Jongsma
This is a finding from coverity. In practice it has very little impact since returning NULL from this function results in the process existing with an error. Signed-off-by: Jonathon Jongsma --- src/udscs.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/udscs.c b/src/udscs.c index 64f