[Xen-devel] [PATCH v3] libxl/libxl_qmp.c: fix error handling in qmp_open

2015-03-19 Thread PRAMOD DEVENDRA
From: Pramod Devendra 1. Make sure sun_path does not overflow 2. Close qmp_fd on error 3. Use goto out for error handling Signed-off-by: Pramod Devendra CC: Ian Jackson CC: Stefano Stabellini CC: Ian Campbell CC: Wei Liu --- tools/libxl/libxl_qmp.c | 26 -- 1

[Xen-devel] [PATCH v2] libxl/libxl_qmp.c: fix qmp_open

2015-03-18 Thread PRAMOD DEVENDRA
From: Pramod Devendra Signed-off-by: Pramod Devendra CC: Ian Jackson CC: Stefano Stabellini CC: Ian Campbell CC: Wei Liu --- Changed since v1: 1. Make sure sun_path does not overflow. 2. Close qmp_fd on error. --- tools/libxl/libxl_qmp.c | 26 -- 1 file changed

[Xen-devel] [PATCH 2/2] tools/libxl/libxl_qmp.c: Close fds on error path in qmp_open

2015-03-18 Thread PRAMOD DEVENDRA
From: Pramod Devendra Signed-off-by: Pramod Devendra CC: Ian Jackson CC: Stefano Stabellini CC: Ian Campbell CC: Wei Liu --- tools/libxl/libxl_qmp.c | 17 ++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/tools/libxl/libxl_qmp.c b/tools/libxl/libxl_qmp.c

[Xen-devel] [PATCH 2/3] tools/libxl/libxl_cpuid.c: Fix leak of resstr on error path

2015-03-16 Thread PRAMOD DEVENDRA
From: Pramod Devendra Signed-off-by: Pramod Devendra CC: Ian Jackson CC: Stefano Stabellini CC: Ian Campbell CC: Wei Liu --- tools/libxl/libxl_cpuid.c |8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/tools/libxl/libxl_cpuid.c b/tools/libxl/libxl_cpuid.c index

[Xen-devel] [PATCH 3/3] tools/libxc/xc_linux_osdep.c: Don't leak mmap() mapping on map_foreign_bulk() error path

2015-03-16 Thread PRAMOD DEVENDRA
From: Pramod Devendra Signed-off-by: Pramod Devendra CC: Ian Jackson CC: Stefano Stabellini CC: Ian Campbell CC: Wei Liu --- tools/libxc/xc_linux_osdep.c |1 + 1 file changed, 1 insertion(+) diff --git a/tools/libxc/xc_linux_osdep.c b/tools/libxc/xc_linux_osdep.c index b6c435a

[Xen-devel] [PATCH 1/3] tools/libxl/libxl_qmp.c: Make sure sun_path is NULL terminated in qmp_open

2015-03-16 Thread PRAMOD DEVENDRA
From: Pramod Devendra Signed-off-by: Pramod Devendra CC: Ian Jackson CC: Stefano Stabellini CC: Ian Campbell CC: Wei Liu --- tools/libxl/libxl_qmp.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tools/libxl/libxl_qmp.c b/tools/libxl/libxl_qmp.c index c7324e6