Blob Blame History Raw
From 108387a2aa986a8107faa7548f3f9e9c084749d2 Mon Sep 17 00:00:00 2001
From: Chris Leech <cleech@redhat.com>
Date: Wed, 24 Mar 2021 09:50:51 -0700
Subject: [PATCH 4/5] fix regressions caused by safe_makepath change in
 libopenfcoe.c

This needs to use strlen(dname) not sizeof(dname) or the directory path
is truncated.  On a 64-bit arch an 8 byte truncation kind of looks like
a valid path '/sys/bus' which is sort of funny unless you're debugging
it.

This caused fcoeadm --fcf to fail to read in FCF info, and fcoeadm
--lesb to fail to find the link error status block counters.

Signed-off-by: Chris Leech <cleech@redhat.com>
---
 libopenfcoe.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libopenfcoe.c b/libopenfcoe.c
index 452ee803e63..cd35c868eee 100644
--- a/libopenfcoe.c
+++ b/libopenfcoe.c
@@ -58,7 +58,7 @@ out_err:
 static char *safe_makepath(char *path, size_t path_sz,
 		char *dname, char *fname)
 {
-	size_t dsz = sizeof(dname);
+	size_t dsz = strlen(dname);
 	size_t fsz = strlen(fname);
 	char *cp = path;
 
-- 
2.26.2