@@ -2589,7 +2589,8 @@ void osm_dump_notice_v2(IN osm_log_t * p_log,
}
}
-static void osm_dump_dr_smp_to_buf(IN const ib_smp_t * p_smp, OUT char * buf)
+static void osm_dump_dr_smp_to_buf(IN const ib_smp_t * p_smp, OUT char * buf,
+ IN size_t buf_size)
{
if (!buf || !p_smp)
return;
@@ -2607,18 +2608,18 @@ static void osm_dump_dr_smp_to_buf(IN const ib_smp_t * p_smp, OUT char * buf)
ib_get_sm_method_str(p_smp->method));
if (p_smp->mgmt_class == IB_MCLASS_SUBN_DIR) {
- n += snprintf(buf + n, sizeof(buf) - n,
+ n += snprintf(buf + n, buf_size - n,
"\t\t\t\tD bit...................0x%X\n"
"\t\t\t\tstatus..................0x%X\n",
ib_smp_is_d(p_smp),
cl_ntoh16(ib_smp_get_status(p_smp)));
} else {
- n += snprintf(buf + n, sizeof(buf) - n,
+ n += snprintf(buf + n, buf_size - n,
"\t\t\t\tstatus..................0x%X\n",
cl_ntoh16(p_smp->status));
}
- n += snprintf(buf + n, sizeof(buf) - n,
+ n += snprintf(buf + n, buf_size - n,
"\t\t\t\thop_ptr.................0x%X\n"
"\t\t\t\thop_count...............0x%X\n"
"\t\t\t\ttrans_id................0x%" PRIx64 "\n"
@@ -2636,34 +2637,34 @@ static void osm_dump_dr_smp_to_buf(IN const ib_smp_t * p_smp, OUT char * buf)
if (p_smp->mgmt_class == IB_MCLASS_SUBN_DIR) {
uint32_t i;
- n += snprintf(buf + n, sizeof(buf) - n,
+ n += snprintf(buf + n, buf_size - n,
"\t\t\t\tdr_slid.................%u\n"
"\t\t\t\tdr_dlid.................%u\n",
cl_ntoh16(p_smp->dr_slid),
cl_ntoh16(p_smp->dr_dlid));
- n += snprintf(buf + n, sizeof(buf) - n,
+ n += snprintf(buf + n, buf_size - n,
"\n\t\t\t\tInitial path: ");
- n += sprint_uint8_arr(buf + n, sizeof(buf) - n,
+ n += sprint_uint8_arr(buf + n, buf_size - n,
p_smp->initial_path,
p_smp->hop_count + 1);
- n += snprintf(buf + n, sizeof(buf) - n,
+ n += snprintf(buf + n, buf_size - n,
"\n\t\t\t\tReturn path: ");
- n += sprint_uint8_arr(buf + n, sizeof(buf) - n,
+ n += sprint_uint8_arr(buf + n, buf_size - n,
p_smp->return_path,
p_smp->hop_count + 1);
- n += snprintf(buf + n, sizeof(buf) - n,
+ n += snprintf(buf + n, buf_size - n,
"\n\t\t\t\tReserved: ");
for (i = 0; i < 7; i++) {
- n += snprintf(buf + n, sizeof(buf) - n,
+ n += snprintf(buf + n, buf_size - n,
"[%0X]", p_smp->resv1[i]);
}
- n += snprintf(buf + n, sizeof(buf) - n, "\n");
+ n += snprintf(buf + n, buf_size - n, "\n");
for (i = 0; i < 64; i += 16) {
- n += snprintf(buf + n, sizeof(buf) - n,
+ n += snprintf(buf + n, buf_size - n,
"\n\t\t\t\t%02X %02X %02X %02X "
"%02X %02X %02X %02X"
" %02X %02X %02X %02X %02X %02X %02X %02X\n",
@@ -2686,7 +2687,7 @@ static void osm_dump_dr_smp_to_buf(IN const ib_smp_t * p_smp, OUT char * buf)
}
} else {
/* not a Direct Route so provide source and destination lids */
- n += snprintf(buf + n, sizeof(buf) - n,
+ n += snprintf(buf + n, buf_size - n,
"\t\t\t\tMAD IS LID ROUTED\n");
}
}
@@ -2698,7 +2699,7 @@ void osm_dump_dr_smp(IN osm_log_t * p_log, IN const ib_smp_t * p_smp,
if (osm_log_is_active(p_log, log_level)) {
char buf[BUF_SIZE];
- osm_dump_dr_smp_to_buf(p_smp, buf);
+ osm_dump_dr_smp_to_buf(p_smp, buf, BUF_SIZE);
osm_log(p_log, log_level, buf);
}
@@ -2710,7 +2711,7 @@ void osm_dump_dr_smp_v2(IN osm_log_t * p_log, IN const ib_smp_t * p_smp,
if (osm_log_is_active_v2(p_log, log_level, file_id)) {
char buf[BUF_SIZE];
- osm_dump_dr_smp_to_buf(p_smp, buf);
+ osm_dump_dr_smp_to_buf(p_smp, buf, BUF_SIZE);
osm_log_v2(p_log, log_level, file_id, buf);
}
@@ -2791,7 +2792,7 @@ void osm_dump_sa_mad_v2(IN osm_log_t * p_log, IN const ib_sa_mad_t * p_mad,
}
static void osm_dump_dr_path_to_buf(IN const osm_dr_path_t * p_path,
- OUT char * buf)
+ OUT char * buf, IN size_t buf_size)
{
if (!buf || !p_path)
return;
@@ -2801,7 +2802,7 @@ static void osm_dump_dr_path_to_buf(IN const osm_dr_path_t * p_path,
n = sprintf(buf, "Directed Path Dump of %u hop path: "
"Path = ", p_path->hop_count);
- sprint_uint8_arr(buf + n, sizeof(buf) - n, p_path->path,
+ sprint_uint8_arr(buf + n, buf_size - n, p_path->path,
p_path->hop_count + 1);
}
}
@@ -2812,7 +2813,7 @@ void osm_dump_dr_path(IN osm_log_t * p_log, IN const osm_dr_path_t * p_path,
if (osm_log_is_active(p_log, log_level)) {
char buf[BUF_SIZE];
- osm_dump_dr_path_to_buf(p_path, buf);
+ osm_dump_dr_path_to_buf(p_path, buf, BUF_SIZE);
osm_log(p_log, log_level, buf);
}
@@ -2824,14 +2825,14 @@ void osm_dump_dr_path_v2(IN osm_log_t * p_log, IN const osm_dr_path_t * p_path,
if (osm_log_is_active_v2(p_log, log_level, file_id)) {
char buf[BUF_SIZE];
- osm_dump_dr_path_to_buf(p_path, buf);
+ osm_dump_dr_path_to_buf(p_path, buf, BUF_SIZE);
osm_log_v2(p_log, log_level, file_id, buf);
}
}
static void osm_dump_smp_dr_path_to_buf(IN const ib_smp_t * p_smp,
- OUT char * buf)
+ OUT char * buf, IN size_t buf_size)
{
if (!buf || !p_smp)
return;
@@ -2840,12 +2841,12 @@ static void osm_dump_smp_dr_path_to_buf(IN const ib_smp_t * p_smp,
n = sprintf(buf, "Received SMP on a %u hop path: "
"Initial path = ", p_smp->hop_count);
- n += sprint_uint8_arr(buf + n, sizeof(buf) - n,
+ n += sprint_uint8_arr(buf + n, buf_size - n,
p_smp->initial_path,
p_smp->hop_count + 1);
- n += sprintf(buf + n, ", Return path = ");
- n += sprint_uint8_arr(buf + n, sizeof(buf) - n,
+ n += snprintf(buf + n, ", Return path = ");
+ n += sprint_uint8_arr(buf + n, buf_size - n,
p_smp->return_path, p_smp->hop_count + 1);
}
}
@@ -2856,7 +2857,7 @@ void osm_dump_smp_dr_path(IN osm_log_t * p_log, IN const ib_smp_t * p_smp,
if (osm_log_is_active(p_log, log_level)) {
char buf[BUF_SIZE];
- osm_dump_smp_dr_path_to_buf(p_smp, buf);
+ osm_dump_smp_dr_path_to_buf(p_smp, buf, BUF_SIZE);
osm_log(p_log, log_level, buf);
}
@@ -2868,7 +2869,7 @@ void osm_dump_smp_dr_path_v2(IN osm_log_t * p_log, IN const ib_smp_t * p_smp,
if (osm_log_is_active_v2(p_log, log_level, file_id)) {
char buf[BUF_SIZE];
- osm_dump_smp_dr_path_to_buf(p_smp, buf);
+ osm_dump_smp_dr_path_to_buf(p_smp, buf, BUF_SIZE);
osm_log_v2(p_log, log_level, file_id, buf);
}
Signed-off-by: Ira Weiny <weiny2@llnl.gov> --- opensm/osm_helper.c | 53 ++++++++++++++++++++++++++------------------------- 1 files changed, 27 insertions(+), 26 deletions(-)