Merge pull request #5939 from jsquyres/pr/coverity-cid-1440332

jsquyres · web-flow · commit c2608fb5971b · 2018-10-17T14:24:42.000-07:00
opal/os_path: fix minor string overrun
diff --git a/opal/util/os_path.c b/opal/util/os_path.c
@@ -59,7 +59,7 @@ char *opal_os_path(int relative, ...)
     	if (relative) {
             path[0] = '.';
         }
-        strncat(path, path_sep, len);
+        strncat(path, path_sep, len - 1);
     	return(path);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ char *opal_os_path(int relative, ...)`
`59`	`59`	`if (relative) {`
`60`	`60`	`path[0] = '.';`
`61`	`61`	`}`
`62`		`- strncat(path, path_sep, len);`
	`62`	`+ strncat(path, path_sep, len - 1);`
`63`	`63`	`return(path);`
`64`	`64`	`}`
`65`	`65`