Name buffer size in ompi_report_comm_methods

devreal · devreal · commit c9ba52439912 · 2025-10-25T19:19:19.000-04:00
Signed-off-by: Joseph Schuchart &lt;joseph.schuchart@stonybrook.edu&gt;
diff --git a/ompi/mca/hook/comm_method/hook_comm_method_fns.c b/ompi/mca/hook/comm_method/hook_comm_method_fns.c
@@ -973,9 +973,9 @@ ompi_report_comm_methods(int called_from_location)
                         }
                     }
                     if (is_nonconformist) {
-                        char *str = malloc(1024);
-//                      int first = 1;
-                        snprintf(str, 1024, "  host %d:", i);
+                        const size_t bufsize = 1024;
+                        char *str = malloc(bufsize);
+                        snprintf(str, bufsize, "  host %d:", i);
                         for (k=0; k<NUM_COMM_METHODS; ++k) {
                             if (method_count[k] > 0) {
 //                              if (!first) {

Original file line number	Diff line number	Diff line change
`@@ -973,9 +973,9 @@ ompi_report_comm_methods(int called_from_location)`
`973`	`973`	`}`
`974`	`974`	`}`
`975`	`975`	`if (is_nonconformist) {`
`976`		`- char *str = malloc(1024);`
`977`		`-// int first = 1;`
`978`		`- snprintf(str, 1024, " host %d:", i);`
	`976`	`+ const size_t bufsize = 1024;`
	`977`	`+ char *str = malloc(bufsize);`
	`978`	`+ snprintf(str, bufsize, " host %d:", i);`
`979`	`979`	`for (k=0; k<NUM_COMM_METHODS; ++k) {`
`980`	`980`	`if (method_count[k] > 0) {`
`981`	`981`	`// if (!first) {`