Re: [Qemu-devel] [PATCH] cryptodev: remove dead code

[Zhoujian (jay)]

Hi Paolo,

I've posted a patch before, pls see
https://lists.gnu.org/archive/html/qemu-devel/2018-04/msg05215.html

which reviewed by Stefan
https://lists.gnu.org/archive/html/qemu-devel/2018-05/msg01549.html

It seems that it hasn't been merged. Could you pick it up pls?

Regards,
Jay

> -----Original Message-----
> From: Qemu-devel [mailto:qemu-devel-
> address@hidden On Behalf Of Paolo Bonzini
> Sent: Monday, July 30, 2018 4:51 PM
> To: address@hidden
> Subject: [Qemu-devel] [PATCH] cryptodev: remove dead code
> 
> Reported by Coverity as CID 1390600.
> 
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
>  backends/cryptodev-vhost-user.c | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/backends/cryptodev-vhost-user.c b/backends/cryptodev-vhost-
> user.c index d52daccfcd..d539f14d59 100644
> --- a/backends/cryptodev-vhost-user.c
> +++ b/backends/cryptodev-vhost-user.c
> @@ -157,7 +157,6 @@ static void cryptodev_vhost_user_event(void *opaque, int
> event)  {
>      CryptoDevBackendVhostUser *s = opaque;
>      CryptoDevBackend *b = CRYPTODEV_BACKEND(s);
> -    Error *err = NULL;
>      int queues = b->conf.peers.queues;
> 
>      assert(queues < MAX_CRYPTO_QUEUE_NUM); @@ -174,10 +173,6 @@ static void
> cryptodev_vhost_user_event(void *opaque, int event)
>          cryptodev_vhost_user_stop(queues, s);
>          break;
>      }
> -
> -    if (err) {
> -        error_report_err(err);
> -    }
>  }
> 
>  static void cryptodev_vhost_user_init(
> --
> 2.17.1
>