vhost/crypto: fix descriptor move
authorFan Zhang <roy.fan.zhang@intel.com>
Wed, 9 May 2018 14:08:39 +0000 (15:08 +0100)
committerFerruh Yigit <ferruh.yigit@intel.com>
Thu, 17 May 2018 10:29:05 +0000 (12:29 +0200)
This patch fixes the redundant descriptor move in the copy mode
of vhost crypto. Originally the redundant descriptor move will
cause the message parsing error.

Fixes: 3bb595ecd682 ("vhost/crypto: add request handler")

Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
lib/librte_vhost/vhost_crypto.c

index 04ff8cd..4c25628 100644 (file)
@@ -853,11 +853,6 @@ prepare_sym_chain_op(struct vhost_crypto *vcrypto, struct rte_crypto_op *op,
                op->sym->auth.digest.data = digest_addr;
                op->sym->auth.digest.phys_addr = rte_pktmbuf_iova_offset(m_dst,
                                digest_offset);
-               if (unlikely(move_desc(head, &desc,
-                               chain->para.hash_result_len) < 0)) {
-                       ret = VIRTIO_CRYPTO_ERR;
-                       goto error_exit;
-               }
                break;
        default:
                ret = VIRTIO_CRYPTO_BADMSG;