Re: [PATCH 4/6] tools/xenstored: use atexit to close interfaces

[Ian Jackson <iwj@xxxxxxxxxxxxxx>]

Roger Pau Monné writes ("Re: [PATCH 4/6] tools/xenstored: use atexit to close 
interfaces"):
> On Mon, Sep 20, 2021 at 11:22:15AM +0200, Juergen Gross wrote:
> > Could you please add closing of xce_handle(), too?
> 
> Sure, I somehow assumed there was a reason for not closing it related
> to live update, but I see that's not the case as you use exec to
> launch the new image and atexit handlers are not called in that case.

Are these fds marked CLOEXEC ?  I don't think they are BICBW.
It would probably be good idea to make them so.  An fd leak might
result in one fd being consumed per live update operation.

Ian.