Re: [Xen-devel] [PATCH] libxl: get_reaper_lock_and_uid: Document fd handling

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Wed, Jan 02, 2019 at 12:03:43PM +0000, Ian Jackson wrote:
> Coverity understandably complains that get_reaper_lock_and_uid leaks
> the fd and hence open-file.  But this is intentional: the lock becomes
> owned by the child process as a whole, which is entirely the property
> of libxl.
> 
> (The coding style here in this subprocess is a bit anomalous but it's
> probably not worth it to convert get_reaper_lock_and_uid to `goto out'
> style and have it explicitly return the fd number.)
> 

Coverity-ID: XXXX here?

> Signed-off-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>

Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel