[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] net: simplify net_init_tap_one logic
From: |
Jason Wang |
Subject: |
Re: [Qemu-devel] [PATCH] net: simplify net_init_tap_one logic |
Date: |
Fri, 26 Feb 2016 15:13:30 +0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 |
On 02/26/2016 07:05 AM, Paolo Bonzini wrote:
> net_init_tap_one receives in vhostfdname a fd name from vhostfd= or
> vhostfds=, or NULL if there is no vhostfd=/vhostfds=. It is simpler
> to just check vhostfdname, than it is to check for vhostfd= or
> vhostfds=. This also calms down Coverity, which otherwise thinks
> that monitor_fd_param could dereference a NULL vhostfdname.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> net/tap.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/net/tap.c b/net/tap.c
> index cfb6831..cd7a7fc 100644
> --- a/net/tap.c
> +++ b/net/tap.c
> @@ -662,7 +662,7 @@ static void net_init_tap_one(const NetdevTapOptions *tap,
> NetClientState *peer,
> options.backend_type = VHOST_BACKEND_TYPE_KERNEL;
> options.net_backend = &s->nc;
>
> - if (tap->has_vhostfd || tap->has_vhostfds) {
> + if (vhostfdname) {
> vhostfd = monitor_fd_param(cur_mon, vhostfdname, &err);
> if (vhostfd == -1) {
> error_propagate(errp, err);
> @@ -684,7 +684,7 @@ static void net_init_tap_one(const NetdevTapOptions *tap,
> NetClientState *peer,
> "vhost-net requested but could not be initialized");
> return;
> }
> - } else if (tap->has_vhostfd || tap->has_vhostfds) {
> + } else if (vhostfdname) {
> error_setg(errp, "vhostfd= is not valid without vhost");
> }
> }
Applied to -net.
Thanks