From afc9cb3c0972fe711d1af7f57d9a8eedf201c074 Mon Sep 17 00:00:00 2001
From: Liming Sun <lsun@mellanox.com>
Date: Mon, 23 Mar 2020 14:31:56 -0400
Subject: [PATCH] Fix the coverity TAINTED_SCALAR warnings

Signed-off-by: Liming Sun <lsun@mellanox.com>
---
 src/rshim.c     | 13 ++++++++-----
 src/rshim_net.c |  2 +-
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/src/rshim.c b/src/rshim.c
index 35e9738..4940858 100644
--- a/src/rshim.c
+++ b/src/rshim.c
@@ -247,7 +247,7 @@ static int rshim_fd_full_write(int fd, void *data, int len)
 void rshim_work_signal(rshim_backend_t *bd)
 {
   if (__sync_bool_compare_and_swap(&bd->work_pending, false, true))
-    rshim_fd_full_write(rshim_work_fd[1], &bd, sizeof(bd));
+    rshim_fd_full_write(rshim_work_fd[1], &bd->index, sizeof(bd->index));
 }
 
 /*
@@ -2090,7 +2090,7 @@ bool rshim_allow_device(const char *devname)
 
 static void rshim_main(int argc, char *argv[])
 {
-  int i, fd, num, rc, epoll_fd, timer_fd;
+  int i, fd, num, rc, epoll_fd, timer_fd, index;
 #ifdef __FreeBSD__
   const int MAXEVENTS = 16;
 #else
@@ -2206,9 +2206,12 @@ static void rshim_main(int argc, char *argv[])
         rshim_fd_full_read(timer_fd, &res, sizeof(res));
         rshim_timer_run();
       } else if (fd == rshim_work_fd[0]) {
-        rc = rshim_fd_full_read(rshim_work_fd[0], &bd, sizeof(bd));
-        if (rc == sizeof(bd))
-          rshim_work_handler(bd);
+        rc = rshim_fd_full_read(rshim_work_fd[0], &index, sizeof(index));
+        if (rc == sizeof(index)) {
+          bd = rshim_devs[index];
+          if (bd)
+            rshim_work_handler(bd);
+        }
         continue;
       } else {
         uint8_t tmp;
diff --git a/src/rshim_net.c b/src/rshim_net.c
index c60cc68..626d556 100644
--- a/src/rshim_net.c
+++ b/src/rshim_net.c
@@ -68,7 +68,7 @@ static int rshim_if_open(char *ifname, int index)
 
   memset(&ifr, 0, sizeof(ifr));
   ifr.ifr_flags = IFF_TAP | IFF_NO_PI;
-  strncpy(ifr.ifr_name, ifname, sizeof(ifr.ifr_name));
+  strncpy(ifr.ifr_name, ifname, sizeof(ifr.ifr_name) - 1);
 
   rc = ioctl(fd, TUNSETIFF, (void *) &ifr);
   if (rc < 0) {