On Thursday 23 September 2010 17:55:39 Gianni Tedesco wrote:
> On Thu, 2010-09-23 at 11:27 +0100, Christoph Egger wrote:
> > Hi!
> >
> > 'xl create' crashes due to stack corruption.
>
> ---8<-------------------------------------------
>
> xl: Fix stack corruption caused by non-terminated call to libxl__xs_writev
Confirmed. This fixes the crash. 'xl create' now hangs in a loop. I will have
a look at this later.
>
> Signed-off-by: Gianni Tedesco <gianni.tedesco@xxxxxxxxxx>
>
> diff -r 50c1cc209f8f tools/libxl/libxl.c
> --- a/tools/libxl/libxl.c Wed Sep 22 18:29:24 2010 +0100
> +++ b/tools/libxl/libxl.c Thu Sep 23 16:54:09 2010 +0100
> @@ -1718,7 +1718,7 @@ retry_transaction:
> vm_path = libxl__xs_read(&gc,t,libxl__sprintf(&gc, "%s/vm",
> p->dom_path)); if (vm_path) {
> /* Now write the vncpassword into it. */
> - pass_stuff = libxl__calloc(&gc, 2, sizeof(char *));
> + pass_stuff = libxl__calloc(&gc, 3, sizeof(char *));
> pass_stuff[0] = "vncpasswd";
> pass_stuff[1] = info->vncpasswd;
> libxl__xs_writev(&gc,t,vm_path,pass_stuff);
--
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Einsteinring 24, 85609 Dornach b. Muenchen
Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|