telemetry: fix memory leak

reshmapattan · tmonjalo · commit 05682bddb603 · 2019-06-18T12:07:45.000+03:00
Free the `values` pointer before returning from rte_telemetry_command_ports_all_stat_values() to avoid memory leak. Fixes: c12cefa ("telemetry: fix mapping of statistics") Cc: stable@dpdk.org Signed-off-by: Reshma Pattan <reshma.pattan@intel.com> Acked-by: Kevin Laatz <kevin.laatz@intel.com>
diff --git a/lib/librte_telemetry/rte_telemetry_parser.c b/lib/librte_telemetry/rte_telemetry_parser.c
@@ -342,6 +342,7 @@ rte_telemetry_command_ports_all_stat_values(struct telemetry_impl *telemetry,
 		goto fail;
 	}
 
+	free(values);
 	return 0;
 
 fail:

Original file line number	Diff line number	Diff line change
`@@ -342,6 +342,7 @@ rte_telemetry_command_ports_all_stat_values(struct telemetry_impl *telemetry,`
`342`	`342`	`goto fail;`
`343`	`343`	`}`
`344`	`344`
	`345`	`+ free(values);`
`345`	`346`	`return 0;`
`346`	`347`
`347`	`348`	`fail:`