On Tue, 2023-10-24 at 15:10 +0100, Paul Durrant wrote:
> On 16/10/2023 16:19, David Woodhouse wrote:
> > --- a/hw/char/xen_console.c
> > +++ b/hw/char/xen_console.c
> > @@ -468,7 +468,7 @@ static void xen_console_device_create(XenBackendInstance *backend,
> > Chardev *cd = NULL;
> > struct qemu_xs_handle *xsh = xenbus->xsh;
> >
> > - if (qemu_strtoul(name, NULL, 10, &number)) {
> > + if (qemu_strtoul(name, NULL, 10, &number) || number >= INT_MAX) {
> > error_setg(errp, "failed to parse name '%s'", name);
> > goto fail;
> > }
> I don't think this hunk belongs here, does it? Seems like it should be
> in patch 7.
Well, console#4294967295 *did* actually work before this patch started
using -1 to mean something different. But yes, I've already moved that
into the previous patch.
In fact I've just completely dropped this patch now, as the
dedeuplication needs to happen on the *frontend* nodes, since a given
frontend can be powered by a backend of different types, or in
different driver domains.
Attachment:
smime.p7s
Description: S/MIME cryptographic signature
