|
 |
e88930 |
From 38952aa277df116da0b7ef389440de34a3e61cd0 Mon Sep 17 00:00:00 2001
|
|
|
e88930 |
From: Scott Moser <smoser@brickies.net>
|
|
|
e88930 |
Date: Wed, 19 Sep 2018 17:21:48 -0400
|
|
|
e88930 |
Subject: [PATCH] Close file handles when writing pid files.
|
|
|
e88930 |
|
|
|
e88930 |
The file descriptors used for writing to the pid file were
|
|
|
e88930 |
not being closed. Just be more clean and close them.
|
|
|
e88930 |
|
|
|
e88930 |
Also fix a debug message that said 'lock file' rather than 'pid file'.
|
|
|
e88930 |
---
|
|
|
e88930 |
iscsiuio/src/unix/main.c | 3 ++-
|
|
|
e88930 |
usr/iscsid.c | 1 +
|
|
|
e88930 |
2 files changed, 3 insertions(+), 1 deletion(-)
|
|
|
e88930 |
|
|
|
e88930 |
diff --git a/iscsiuio/src/unix/main.c b/iscsiuio/src/unix/main.c
|
|
|
e88930 |
index 5cccc76ed220..457ebe340b7d 100644
|
|
|
e88930 |
--- a/iscsiuio/src/unix/main.c
|
|
|
e88930 |
+++ b/iscsiuio/src/unix/main.c
|
|
|
e88930 |
@@ -346,8 +346,9 @@ int main(int argc, char *argv[])
|
|
|
e88930 |
sprintf(buf, "%d\n", getpid());
|
|
|
e88930 |
written_bytes = write(fd, buf, strlen(buf));
|
|
|
e88930 |
if (written_bytes == -1)
|
|
|
e88930 |
- printf("Could not write lock file [%s]",
|
|
|
e88930 |
+ printf("Could not write pid file [%s]",
|
|
|
e88930 |
strerror(errno));
|
|
|
e88930 |
+ close(fd);
|
|
|
e88930 |
|
|
|
e88930 |
daemon_init();
|
|
|
e88930 |
}
|
|
|
e88930 |
diff --git a/usr/iscsid.c b/usr/iscsid.c
|
|
|
e88930 |
index b95917df3bc5..7d6032252d00 100644
|
|
|
e88930 |
--- a/usr/iscsid.c
|
|
|
e88930 |
+++ b/usr/iscsid.c
|
|
|
e88930 |
@@ -467,6 +467,7 @@ int main(int argc, char *argv[])
|
|
|
e88930 |
exit(ISCSI_ERR);
|
|
|
e88930 |
}
|
|
|
e88930 |
}
|
|
|
e88930 |
+ close(fd);
|
|
|
e88930 |
|
|
|
e88930 |
daemon_init();
|
|
|
e88930 |
} else {
|
|
|
e88930 |
--
|
|
|
e88930 |
2.17.2
|
|
|
e88930 |
|