From: Maurizio Lombardi <mlombard@redhat.com>
Date: Mon, 4 Mar 2024 14:05:15 +0000 (+0100)
Subject: fabrics: check the read() return value
X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=dd22963a614533e630968f2b6557d7ed93e0973c;p=users%2Fsagi%2Flibnvme.git

fabrics: check the read() return value

Do not pass to the is_dmi_uuid_valid() function an invalid len value
if read() fails.

Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
---

diff --git a/src/nvme/fabrics.c b/src/nvme/fabrics.c
index 1f50229f..ac639bdf 100644
--- a/src/nvme/fabrics.c
+++ b/src/nvme/fabrics.c
@@ -1297,6 +1297,8 @@ static int uuid_from_dmi_entries(char *system_uuid)
 			continue;
 		len = read(f, buf, 512);
 		close(f);
+		if (len <= 0)
+			continue;
 
 		if (!is_dmi_uuid_valid(buf, len))
 			continue;