Re: [Xen-devel] [PATCH] libxl: Don't derefence null new_name pointer in libxl_domain_rename()

2014-12-02 Thread Ian Campbell
On Mon, 2014-12-01 at 17:02 -0500, Konrad Rzeszutek Wilk wrote: > On Mon, Dec 01, 2014 at 02:50:56PM +, Ian Campbell wrote: > > On Mon, 2014-12-01 at 14:27 +, Euan Harris wrote: > > > libxl__domain_rename() unconditionally dereferences its new_name > > > parameter, to check whether it is an

Re: [Xen-devel] [PATCH] libxl: Don't derefence null new_name pointer in libxl_domain_rename()

2014-12-01 Thread Konrad Rzeszutek Wilk
On Mon, Dec 01, 2014 at 02:50:56PM +, Ian Campbell wrote: > On Mon, 2014-12-01 at 14:27 +, Euan Harris wrote: > > libxl__domain_rename() unconditionally dereferences its new_name > > parameter, to check whether it is an empty string. Add a check to > > avoid a segfault if new_name is null

Re: [Xen-devel] [PATCH] libxl: Don't derefence null new_name pointer in libxl_domain_rename()

2014-12-01 Thread Ian Campbell
On Mon, 2014-12-01 at 14:27 +, Euan Harris wrote: > libxl__domain_rename() unconditionally dereferences its new_name > parameter, to check whether it is an empty string. Add a check to > avoid a segfault if new_name is null. > > Signed-off-by: Euan Harris Acked-by: Ian Campbell I think t

[Xen-devel] [PATCH] libxl: Don't derefence null new_name pointer in libxl_domain_rename()

2014-12-01 Thread Euan Harris
libxl__domain_rename() unconditionally dereferences its new_name parameter, to check whether it is an empty string. Add a check to avoid a segfault if new_name is null. Signed-off-by: Euan Harris --- tools/libxl/libxl.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --g