[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 4/4] net/virtio: return error when device_opts arg is NULL
From: |
Paolo Bonzini |
Subject: |
Re: [PULL 4/4] net/virtio: return error when device_opts arg is NULL |
Date: |
Mon, 25 Nov 2019 17:08:07 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 |
On 25/11/19 16:40, Jason Wang wrote:
> From: Jens Freimann <address@hidden>
>
> This fixes CID 1407222.
>
> Fixes: 9711cd0dfc3f ("net/virtio: add failover support")
> Signed-off-by: Jens Freimann <address@hidden>
> Reviewed-by: Michael S. Tsirkin <address@hidden>
> Signed-off-by: Jason Wang <address@hidden>
> ---
> hw/net/virtio-net.c | 9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
> index 565dea0..3c31471 100644
> --- a/hw/net/virtio-net.c
> +++ b/hw/net/virtio-net.c
> @@ -2880,9 +2880,12 @@ static int
> virtio_net_primary_should_be_hidden(DeviceListener *listener,
> QemuOpts *device_opts)
> {
> VirtIONet *n = container_of(listener, VirtIONet, primary_listener);
> - bool match_found;
> - bool hide;
> + bool match_found = false;
> + bool hide = false;
>
> + if (!device_opts) {
> + return -1;
> + }
> n->primary_device_dict = qemu_opts_to_qdict(device_opts,
> n->primary_device_dict);
> if (n->primary_device_dict) {
> @@ -2890,7 +2893,7 @@ static int
> virtio_net_primary_should_be_hidden(DeviceListener *listener,
> n->standby_id = g_strdup(qdict_get_try_str(n->primary_device_dict,
> "failover_pair_id"));
> }
> - if (device_opts && g_strcmp0(n->standby_id, n->netclient_name) == 0) {
> + if (g_strcmp0(n->standby_id, n->netclient_name) == 0) {
> match_found = true;
> } else {
> match_found = false;
>
It can't be NULL though, can it? It is called from a qemu_foreach_opt
callback on device_opts itself. This can be "re-fixed" in 5.0 though,
no hurry.
Paolo
- [PULL 0/4] Net patches, Jason Wang, 2019/11/25
- [PULL 1/4] net/virtio: fix dev_unplug_pending, Jason Wang, 2019/11/25
- [PULL 3/4] net/virtio: fix re-plugging of primary device, Jason Wang, 2019/11/25
- [PULL 4/4] net/virtio: return error when device_opts arg is NULL, Jason Wang, 2019/11/25
- Re: [PULL 4/4] net/virtio: return error when device_opts arg is NULL,
Paolo Bonzini <=
- [PULL 2/4] net/virtio: return early when failover primary alread added, Jason Wang, 2019/11/25
- Re: [PULL 0/4] Net patches, Peter Maydell, 2019/11/26