d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

metrics: fix name string termination

Browse Source 
The public API (struct rte_metric_name) includes the NULL terminator
byte in RTE_METRICS_MAX_NAME_LENGTH but the library itself internally
excludes it. This makes it possible for an application to receive an
unterminated name string. Fix be enforcing the NULL termination of all
name strings to the length that the public API expects.

Fixes: 349950ddb9 ("metrics: add information metrics library")
Cc: stable@dpdk.org

Signed-off-by: Remy Horton <remy.horton@intel.com>
This commit is contained in:
Remy Horton 2017-07-18 10:43:47 +01:00 committed by Thomas Monjalon
parent 838d6b775b
commit fa43ac7729
2 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/librte_metrics/rte_metrics.c
View File

@ -51,7 +51,7 @@
*/
struct rte_metrics_meta_s {
/** Name of metric */
char name[RTE_METRICS_MAX_NAME_LEN + 1];
char name[RTE_METRICS_MAX_NAME_LEN];
/** Current value for metric */
uint64_t value[RTE_MAX_ETHPORTS];
/** Used for global metrics */
@ -144,6 +144,8 @@ rte_metrics_reg_names(const char * const *names, uint16_t cnt_names)
entry = &stats->metadata[idx_name + stats->cnt_stats];
strncpy(entry->name, names[idx_name],
RTE_METRICS_MAX_NAME_LEN);
/* Enforce NULL-termination */
entry->name[RTE_METRICS_MAX_NAME_LEN - 1] = '\0';
memset(entry->value, 0, sizeof(entry->value));
entry->idx_next_stat = idx_name + stats->cnt_stats + 1;
}

3
lib/librte_metrics/rte_metrics.h
View File

@ -118,7 +118,8 @@ void rte_metrics_init(int socket_id);
* is required for updating said metric's value.
*
* @param name
* Metric name
* Metric name. If this exceeds RTE_METRICS_MAX_NAME_LEN (including
* the NULL terminator), it is truncated.
*
* @return
* - Zero or positive: Success (index key of new metric)