[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-stable] [PULL 30/53] hw/isa/superio: Fix inconsistent use of C
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-stable] [PULL 30/53] hw/isa/superio: Fix inconsistent use of Chardev->be |
Date: |
Thu, 31 May 2018 15:08:27 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 |
Hi Paolo, something messed with UTF-8...
On 05/31/2018 02:12 PM, Paolo Bonzini wrote:
> From: Philippe Mathieu-Daudé <address@hidden>
>
> 4c3119a6e3e and cd9526ab7c0 introduced an incorrect and inconsistent
> use of Chardev->be. Also, this CharBackend member is private and is
> not supposed to be accessible.
>
> Fix it by removing the inconsistent check.
>
> Cc: address@hidden
> Reported-by: Marc-André Lureau <address@hidden>
Marc-André Lureau
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Philippe Mathieu-Daudé
> Message-Id: <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> Reviewed-by: Marc-André Lureau <address@hidden>
Correct.
> ---
> hw/isa/isa-superio.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/isa/isa-superio.c b/hw/isa/isa-superio.c
> index 9359be7008..8bc2f69eaa 100644
> --- a/hw/isa/isa-superio.c
> +++ b/hw/isa/isa-superio.c
> @@ -42,7 +42,7 @@ static void isa_superio_realize(DeviceState *dev, Error
> **errp)
> if (!k->parallel.is_enabled || k->parallel.is_enabled(sio, i)) {
> /* FIXME use a qdev chardev prop instead of parallel_hds[] */
> chr = parallel_hds[i];
> - if (chr == NULL || chr->be) {
> + if (chr == NULL) {
> name = g_strdup_printf("discarding-parallel%d", i);
> chr = qemu_chr_new(name, "null");
> } else {
> @@ -82,7 +82,7 @@ static void isa_superio_realize(DeviceState *dev, Error
> **errp)
> if (!k->serial.is_enabled || k->serial.is_enabled(sio, i)) {
> /* FIXME use a qdev chardev prop instead of serial_hd() */
> chr = serial_hd(i);
> - if (chr == NULL || chr->be) {
> + if (chr == NULL) {
> name = g_strdup_printf("discarding-serial%d", i);
> chr = qemu_chr_new(name, "null");
> } else {
>