Re: [Xen-devel] [PATCH] tools/libxl: Fix a segment fault when mmio_hole is set in hvm.cfg

[Christopher Clark <christopher.w.clark@xxxxxxxxx>]

On Wed, Jul 12, 2017 at 3:15 AM, Wei Liu <wei.liu2@xxxxxxxxxx> wrote:
>
> On Thu, Jul 13, 2017 at 10:03:39AM +0800, Xiong Zhang wrote:
> > When valid mmio_hole is set in hvm.cfg, segment fault happens at accessing
> > localents pointer.
> >
> > Because the size of localents pointer isn't enough to store appended
> > mmio_hole_size parameter.
> >
> > Signed-off-by: Xiong Zhang <xiong.y.zhang@xxxxxxxxx>
> > ---
> >
> >     tools/libxl/libxl_create.c | 2 +-
> >     1 file changed, 1 insertion(+), 1 deletion(-)
> >
> >    diff --git a/tools/libxl/libxl_create.c b/tools/libxl/libxl_create.c
> >    index bffbc45..1158303 100644
> >    --- a/tools/libxl/libxl_create.c
> >    +++ b/tools/libxl/libxl_create.c
> >    @@ -451,7 +451,7 @@ int libxl__domain_build(libxl__gc *gc,
> >             vments[4] = "start_time";
> >             vments[5] = GCSPRINTF("%lu.%02d", 
> > start_time.tv_sec,(int)start_time.tv_usec/10000);
> >
> >    -        localents = libxl__calloc(gc, 9, sizeof(char *));
> >    +        localents = libxl__calloc(gc, 11, sizeof(char *));
> >             i = 0;
> >             localents[i++] = "platform/acpi";
> >             localents[i++] = libxl__acpi_defbool_val(info) ? "1" : "0";
>
>
> Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>
>
> Ian please backport this.


Bump: the above patch still needs to be backported into 4.9 please.

master commit: 614a14736e33fb84872eb00f08799ebbc73a96c6

thanks,

Christopher
--
http://openxt.org/

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel