|
 |
587c20 |
From f8e7499e9f69465668fc6e5e7bcfb9ce09e8a457 Mon Sep 17 00:00:00 2001
|
|
|
587c20 |
From: Chris Leech <cleech@redhat.com>
|
|
|
587c20 |
Date: Wed, 5 Jun 2019 09:08:39 -0700
|
|
|
587c20 |
Subject: [PATCH] Coverity scan fixes
|
|
|
587c20 |
|
|
|
587c20 |
---
|
|
|
587c20 |
iscsiuio/src/unix/libs/qedi.c | 2 +-
|
|
|
587c20 |
iscsiuio/src/unix/main.c | 12 ++++++++++--
|
|
|
587c20 |
libopeniscsiusr/idbm.c | 11 +++++------
|
|
|
587c20 |
usr/idbm.c | 10 ++++------
|
|
|
587c20 |
usr/iscsid.c | 2 +-
|
|
|
587c20 |
5 files changed, 21 insertions(+), 16 deletions(-)
|
|
|
587c20 |
|
|
|
587c20 |
diff --git a/iscsiuio/src/unix/libs/qedi.c b/iscsiuio/src/unix/libs/qedi.c
|
|
|
587c20 |
index b7595d5..47300b0 100644
|
|
|
587c20 |
--- a/iscsiuio/src/unix/libs/qedi.c
|
|
|
587c20 |
+++ b/iscsiuio/src/unix/libs/qedi.c
|
|
|
587c20 |
@@ -1023,7 +1023,7 @@ static int qedi_read(nic_t *nic, packet_t *pkt)
|
|
|
587c20 |
|
|
|
587c20 |
LOG_DEBUG(PFX "%s:hw_prod %d bd_prod %d, rx_pkt_idx %d, rxlen %d",
|
|
|
587c20 |
nic->log_name, hw_prod, bd_prod, rx_bd->rx_pkt_index, len);
|
|
|
587c20 |
- LOG_DEBUG(PFX "%s: sw_con %d bd_cons %d num BD %d",
|
|
|
587c20 |
+ LOG_DEBUG(PFX "%s: sw_con %d bd_cons %d num BD %lu",
|
|
|
587c20 |
nic->log_name, sw_cons, bd_cons, QEDI_NUM_RX_BD);
|
|
|
587c20 |
|
|
|
587c20 |
if (bd_cons != bd_prod) {
|
|
|
587c20 |
diff --git a/iscsiuio/src/unix/main.c b/iscsiuio/src/unix/main.c
|
|
|
587c20 |
index 5168e4e..ed41931 100644
|
|
|
587c20 |
--- a/iscsiuio/src/unix/main.c
|
|
|
587c20 |
+++ b/iscsiuio/src/unix/main.c
|
|
|
587c20 |
@@ -340,7 +340,10 @@ int main(int argc, char *argv[])
|
|
|
587c20 |
|
|
|
587c20 |
/* parent: wait for child msg then exit */
|
|
|
587c20 |
close(pipefds[1]);
|
|
|
587c20 |
- read(pipefds[0], msgbuf, sizeof(msgbuf));
|
|
|
587c20 |
+ if (read(pipefds[0], msgbuf, sizeof(msgbuf)) < 0) {
|
|
|
587c20 |
+ fprintf(stderr, "ERR: Waiting for child process failed\n");
|
|
|
587c20 |
+ exit(1);
|
|
|
587c20 |
+ }
|
|
|
587c20 |
exit(0);
|
|
|
587c20 |
}
|
|
|
587c20 |
|
|
|
587c20 |
@@ -386,6 +389,9 @@ int main(int argc, char *argv[])
|
|
|
587c20 |
sigaddset(&set, SIGTERM);
|
|
|
587c20 |
sigaddset(&set, SIGUSR1);
|
|
|
587c20 |
rc = pthread_sigmask(SIG_SETMASK, &set, NULL);
|
|
|
587c20 |
+ if (rc != 0) {
|
|
|
587c20 |
+ LOG_ERR("Failed to set thread signal mask");
|
|
|
587c20 |
+ }
|
|
|
587c20 |
|
|
|
587c20 |
/* Spin off the signal handling thread */
|
|
|
587c20 |
pthread_attr_init(&attr);
|
|
|
587c20 |
@@ -415,7 +421,9 @@ int main(int argc, char *argv[])
|
|
|
587c20 |
if (!foreground) {
|
|
|
587c20 |
/* signal parent they can go away now */
|
|
|
587c20 |
close(pipefds[0]);
|
|
|
587c20 |
- write(pipefds[1], "ok\n", 3);
|
|
|
587c20 |
+ if (write(pipefds[1], "ok\n", 3) < 0) {
|
|
|
587c20 |
+ LOG_ERR("Failed to signal parent process of completed initialization");
|
|
|
587c20 |
+ }
|
|
|
587c20 |
close(pipefds[1]);
|
|
|
587c20 |
}
|
|
|
587c20 |
|
|
|
587c20 |
diff --git a/libopeniscsiusr/idbm.c b/libopeniscsiusr/idbm.c
|
|
|
587c20 |
index d020e6c..342aab5 100644
|
|
|
587c20 |
--- a/libopeniscsiusr/idbm.c
|
|
|
587c20 |
+++ b/libopeniscsiusr/idbm.c
|
|
|
587c20 |
@@ -287,12 +287,11 @@ int _idbm_lock(struct iscsi_context *ctx)
|
|
|
587c20 |
return 0;
|
|
|
587c20 |
}
|
|
|
587c20 |
|
|
|
587c20 |
- if (access(LOCK_DIR, F_OK) != 0) {
|
|
|
587c20 |
- if (mkdir(LOCK_DIR, 0660) != 0) {
|
|
|
587c20 |
- _error(ctx, "Could not open %s: %d %s", LOCK_DIR, errno,
|
|
|
587c20 |
- _strerror(errno, strerr_buff));
|
|
|
587c20 |
- return LIBISCSI_ERR_IDBM;
|
|
|
587c20 |
- }
|
|
|
587c20 |
+ if (((mkdir(LOCK_DIR, 0660) != 0) && (errno != EEXIST)) ||
|
|
|
587c20 |
+ (access(LOCK_DIR, F_OK) != 0)) {
|
|
|
587c20 |
+ _error(ctx, "Could not open %s: %d %s", LOCK_DIR, errno,
|
|
|
587c20 |
+ _strerror(errno, strerr_buff));
|
|
|
587c20 |
+ return LIBISCSI_ERR_IDBM;
|
|
|
587c20 |
}
|
|
|
587c20 |
|
|
|
587c20 |
fd = open(LOCK_FILE, O_RDWR | O_CREAT, 0666);
|
|
|
587c20 |
diff --git a/usr/idbm.c b/usr/idbm.c
|
|
|
587c20 |
index 89a6c27..a3343fe 100644
|
|
|
587c20 |
--- a/usr/idbm.c
|
|
|
587c20 |
+++ b/usr/idbm.c
|
|
|
587c20 |
@@ -1332,12 +1332,10 @@ int idbm_lock(void)
|
|
|
587c20 |
return 0;
|
|
|
587c20 |
}
|
|
|
587c20 |
|
|
|
587c20 |
- if (access(LOCK_DIR, F_OK) != 0) {
|
|
|
587c20 |
- if (mkdir(LOCK_DIR, 0660) != 0) {
|
|
|
587c20 |
- log_error("Could not open %s: %s", LOCK_DIR,
|
|
|
587c20 |
- strerror(errno));
|
|
|
587c20 |
- return ISCSI_ERR_IDBM;
|
|
|
587c20 |
- }
|
|
|
587c20 |
+ if (((mkdir(LOCK_DIR, 0660) != 0) && (errno != EEXIST)) ||
|
|
|
587c20 |
+ (access(LOCK_DIR, F_OK) != 0)) {
|
|
|
587c20 |
+ log_error("Could not open %s: %s", LOCK_DIR, strerror(errno));
|
|
|
587c20 |
+ return ISCSI_ERR_IDBM;
|
|
|
587c20 |
}
|
|
|
587c20 |
|
|
|
587c20 |
fd = open(LOCK_FILE, O_RDWR | O_CREAT, 0666);
|
|
|
587c20 |
diff --git a/usr/iscsid.c b/usr/iscsid.c
|
|
|
587c20 |
index 37c13b3..1891784 100644
|
|
|
587c20 |
--- a/usr/iscsid.c
|
|
|
587c20 |
+++ b/usr/iscsid.c
|
|
|
587c20 |
@@ -489,8 +489,8 @@ int main(int argc, char *argv[])
|
|
|
587c20 |
log_close(log_pid);
|
|
|
587c20 |
exit(ISCSI_ERR);
|
|
|
587c20 |
}
|
|
|
587c20 |
+ close(fd);
|
|
|
587c20 |
}
|
|
|
587c20 |
- close(fd);
|
|
|
587c20 |
|
|
|
587c20 |
if ((control_fd = ipc->ctldev_open()) < 0) {
|
|
|
587c20 |
log_close(log_pid);
|
|
|
587c20 |
--
|
|
|
587c20 |
2.21.0
|
|
|
587c20 |
|