From 108387a2aa986a8107faa7548f3f9e9c084749d2 Mon Sep 17 00:00:00 2001 From: Chris Leech 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 --- 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