diff options
| author | Halil Pasic <pasic@linux.vnet.ibm.com> | 2017-03-22 13:36:55 +0100 |
|---|---|---|
| committer | Gonglei <arei.gonglei@huawei.com> | 2017-03-23 17:22:01 +0800 |
| commit | b7bad50ae81efeb180609eeecdb086ebc7536ed7 (patch) | |
| tree | 64bac0aea7360564f831e1e1552bd36fc90d3934 /backends/cryptodev-builtin.c | |
| parent | 50d19cf3681059efc8d2efcc585d2a19b9ab37c3 (diff) | |
| download | focaccia-qemu-b7bad50ae81efeb180609eeecdb086ebc7536ed7.tar.gz focaccia-qemu-b7bad50ae81efeb180609eeecdb086ebc7536ed7.zip | |
cryptodev: fix asserting single queue
We already check for queues == 1 in cryptodev_builtin_init and when that is not true raise an error. But before that error is reported the assertion in cryptodev_builtin_cleanup kicks in (because object is being finalized and freed). Let's remove assert(queues == 1) form cryptodev_builtin_cleanup as it does only harm and no good. Reported-by: Boris Fiuczynski <fiuczy@linux.vnet.ibm.com> Signed-off-by: Halil Pasic <pasic@linux.vnet.ibm.com> Reviewed-by: Eric Blake <eblake@redhat.com> Signed-off-by: Gonglei <arei.gonglei@huawei.com>
Diffstat (limited to 'backends/cryptodev-builtin.c')
| -rw-r--r-- | backends/cryptodev-builtin.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/backends/cryptodev-builtin.c b/backends/cryptodev-builtin.c index b24a299d9e..657c0ba2f3 100644 --- a/backends/cryptodev-builtin.c +++ b/backends/cryptodev-builtin.c @@ -361,8 +361,6 @@ static void cryptodev_builtin_cleanup( } } - assert(queues == 1); - for (i = 0; i < queues; i++) { cc = backend->conf.peers.ccs[i]; if (cc) { |