Re: [Xen-devel] [PATCH] libxl: fix unitialized variables in libxl__primary_console_find

[Olaf Hering <olaf@xxxxxxxxx>]

On Mon, Jul 23, Ian Campbell wrote:

> On Wed, 2012-07-04 at 13:17 +0100, Olaf Hering wrote:
> > # HG changeset patch
> > # User Olaf Hering <olaf@xxxxxxxxx>
> > # Date 1341404234 -7200
> > # Node ID 5668cb98eebb0a27530c8b52f3ae8cce87d3b863
> > # Parent  42f76d536b116d2ebad1b6705ae51ecd171d2581
> > libxl: fix unitialized variables in libxl__primary_console_find
> 
> Olaf,
> 
> I have a vague recollection of some discussion of this and a different
> fix but I can't actually find any such discussion and the patch is still
> in my queue. Is this still needed?

Yes.

> > gcc 4.5 as shipped with openSuSE 11.4 does not recognize the case of
> > LIBXL_DOMAIN_TYPE_INVALID properly:
> 
> That's kind of lame of gcc...
> 
> I wonder if default: abort() might be more appropriate (I'm not sure
> what our policy here ended up being).

libxl__domain_type is a simple and internal function. Perhaps a comment
about why a 'default' case exists is enough.

Olaf

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