examples/vhost_crypto: fix unchecked return value

Message ID 20190515162548.71498-1-roy.fan.zhang@intel.com
State New
Headers show
Series
  • examples/vhost_crypto: fix unchecked return value
Related show

Checks

Context Check Description
ci/Intel-compilation success Compilation OK
ci/intel-Performance-Testing success Performance Testing PASS
ci/mellanox-Performance-Testing success Performance Testing PASS
ci/checkpatch success coding style OK

Commit Message

Fan Zhang May 15, 2019, 4:25 p.m.
This patch tries to fix the coverity issue of unchecked
return value. Since the function that causes the problem
is unused, it is removed completely.

Coverity issue: 336816
Fixes: f5188211c721 ("examples/vhost_crypto: add sample application")
Cc: stable@dpdk.org

Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
---
 examples/vhost_crypto/main.c | 6 ------
 1 file changed, 6 deletions(-)

Comments

Jens Freimann May 16, 2019, 8:07 a.m. | #1
On Wed, May 15, 2019 at 05:25:48PM +0100, Fan Zhang wrote:
>This patch tries to fix the coverity issue of unchecked
>return value. Since the function that causes the problem
>is unused, it is removed completely.
>
>Coverity issue: 336816
>Fixes: f5188211c721 ("examples/vhost_crypto: add sample application")
>Cc: stable@dpdk.org
>
>Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
>---
> examples/vhost_crypto/main.c | 6 ------
> 1 file changed, 6 deletions(-)
>

It really is unused.

Reviewed-by: Jens Freimann <jfreimann@redhat.com>

Patch

diff --git a/examples/vhost_crypto/main.c b/examples/vhost_crypto/main.c
index ac7118e7e..cfb9dfda2 100644
--- a/examples/vhost_crypto/main.c
+++ b/examples/vhost_crypto/main.c
@@ -354,12 +354,6 @@  static const struct vhost_device_ops virtio_crypto_device_ops = {
 	.destroy_device = destroy_device,
 };
 
-__attribute__((unused))
-static void clrscr(void)
-{
-	system("@cls||clear");
-}
-
 static int
 vhost_crypto_worker(void *arg)
 {