b49ef5d3 (xenforeignmemory: work around bug in older privcmd) added an error path but forgot to close fd there.
Reported-by: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Wei Liu <wei.l...@citrix.com> --- Cc: Ian Jackson <ian.jack...@eu.citrix.com> --- tools/libs/foreignmemory/linux.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/libs/foreignmemory/linux.c b/tools/libs/foreignmemory/linux.c index 0368aa09f4..132875df8a 100644 --- a/tools/libs/foreignmemory/linux.c +++ b/tools/libs/foreignmemory/linux.c @@ -62,6 +62,7 @@ int osdep_xenforeignmemory_open(xenforeignmemory_handle *fmem) { xtl_log(fmem->logger, XTL_ERROR, -1, "xenforeignmemory", "privcmd ioctl should not be implemented"); + close(fd); return -1; } else -- 2.11.0 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel