Blame SOURCES/0032-kpartx-fail-if-dup-of-dasd-file-descriptor-fails.patch
|
 |
b7ef27 |
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
|
b7ef27 |
From: Benjamin Marzinski <bmarzins@redhat.com>
|
|
|
b7ef27 |
Date: Thu, 16 May 2019 12:55:16 -0500
|
|
|
b7ef27 |
Subject: [PATCH] kpartx: fail if dup() of dasd file descriptor fails
|
|
|
b7ef27 |
|
|
|
b7ef27 |
If kpartx fails to create a copy of the dasd file descriptor, it should
|
|
|
b7ef27 |
fail, instead of treating the error value as a valid fd. Found by
|
|
|
b7ef27 |
coverity.
|
|
|
b7ef27 |
|
|
|
b7ef27 |
Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
|
|
|
b7ef27 |
---
|
|
|
b7ef27 |
kpartx/dasd.c | 2 ++
|
|
|
b7ef27 |
1 file changed, 2 insertions(+)
|
|
|
b7ef27 |
|
|
|
b7ef27 |
diff --git a/kpartx/dasd.c b/kpartx/dasd.c
|
|
|
b7ef27 |
index 61b609a..d95d8ca 100644
|
|
|
b7ef27 |
--- a/kpartx/dasd.c
|
|
|
b7ef27 |
+++ b/kpartx/dasd.c
|
|
|
b7ef27 |
@@ -138,6 +138,8 @@ read_dasd_pt(int fd, struct slice all, struct slice *sp, int ns)
|
|
|
b7ef27 |
return -1;
|
|
|
b7ef27 |
} else {
|
|
|
b7ef27 |
fd_dasd = dup(fd);
|
|
|
b7ef27 |
+ if (fd_dasd < 0)
|
|
|
b7ef27 |
+ return -1;
|
|
|
b7ef27 |
}
|
|
|
b7ef27 |
|
|
|
b7ef27 |
if (ioctl(fd_dasd, BIODASDINFO, (unsigned long)&info) != 0) {
|
|
|
b7ef27 |
--
|
|
|
b7ef27 |
2.17.2
|
|
|
b7ef27 |
|