Re: [Xen-devel] [PATCH] xl: close restore file fd when we're done with it

2015-08-16 Thread Ian Campbell
On Thu, 2015-08-13 at 11:15 +0100, Ian Campbell wrote: > On Thu, 2015-08-13 at 11:09 +0100, Wei Liu wrote: > > And log if close fails. > > > > Reported-by: Andrew Armenia > > Signed-off-by: Wei Liu > > --- > > Acked-by: Ian Campbell Applied. ___ Xe

Re: [Xen-devel] [PATCH] xl: close restore file fd when we're done with it

2015-08-13 Thread Ian Campbell
On Thu, 2015-08-13 at 11:09 +0100, Wei Liu wrote: > And log if close fails. > > Reported-by: Andrew Armenia > Signed-off-by: Wei Liu > --- Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH] xl: close restore file fd when we're done with it

2015-08-13 Thread Wei Liu
This patch is for 4.6. Forgot to extend the tag in subject line. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

[Xen-devel] [PATCH] xl: close restore file fd when we're done with it

2015-08-13 Thread Wei Liu
And log if close fails. Reported-by: Andrew Armenia Signed-off-by: Wei Liu --- Cc: Ian Campbell Cc: Ian Jackson Cc: Andrew Armenia For 4.6, fix leaking fd to avoid holding on to restoring file. This should also be backported to many versions that uses xl. --- tools/libxl/xl_cmdimpl.c | 9