diff mbox

[dpdk-dev,v1,1/1] app/procinfo: resource leak fix.

Message ID 1493995649-3220-1-git-send-email-romanx.korynkevych@intel.com (mailing list archive)
State Changes Requested, archived
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Roman Korynkevych May 5, 2017, 2:47 p.m. UTC 
  Coverity issue: 143256
Fixes: 2deb6b5246d7706448d070335b329d1acb754cee ("app/procinfo: add collectd format and host id")
Cc: stable@dpdk.org

Signed-off-by: Roman Korynkevych <romanx.korynkevych@intel.com>
---
 app/proc_info/main.c | 21 ++++++++++++++++-----
 1 file changed, 16 insertions(+), 5 deletions(-)

Comments

Pattan, Reshma May 9, 2017, 1:38 p.m. UTC | #1 
Hi,

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Roman Korynkevych
> Sent: Friday, May 5, 2017 3:47 PM
> To: dev@dpdk.org
> Cc: Van Haaren, Harry <harry.van.haaren@intel.com>; Tahhan, Maryam
> <maryam.tahhan@intel.com>; Korynkevych, RomanX
> <romanx.korynkevych@intel.com>; stable@dpdk.org
> Subject: [dpdk-dev] [PATCH v1 1/1] app/procinfo: resource leak fix.
> 
> Coverity issue: 143256
> Fixes: 2deb6b5246d7706448d070335b329d1acb754cee ("app/procinfo: add
> collectd format and host id")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Roman Korynkevych <romanx.korynkevych@intel.com>
> ---
>  app/proc_info/main.c | 21 ++++++++++++++++-----
>  1 file changed, 16 insertions(+), 5 deletions(-)
> 
> diff --git a/app/proc_info/main.c b/app/proc_info/main.c index
> 17a1c87..3d0b621 100644
> --- a/app/proc_info/main.c
> +++ b/app/proc_info/main.c

1)You need to close the stdout_fd even inside proc_info_preparse_args() before return -1;
(or) 

inside below piece of code in main().
ret = proc_info_preparse_args(argc, argv);
        if (ret < 0) {
                printf("Failed to parse arguments\n");
                return -1;
        }

2)Also if  stdout_fd = dup(STDOUT_FILENO);  can return negative value, You may need to exit then? If so can you fix that code as well.

Thanks,
Reshma
diff mbox

Patch

diff --git a/app/proc_info/main.c b/app/proc_info/main.c
index 17a1c87..3d0b621 100644
--- a/app/proc_info/main.c
+++ b/app/proc_info/main.c
@@ -75,7 +75,7 @@  static uint32_t enable_xstats;
 /**< Enable collectd format*/
 static uint32_t enable_collectd_format;
 /**< FD to send collectd format messages to STDOUT*/
-static int stdout_fd;
+static int stdout_fd = -1;
 /**< Host id process is running on */
 static char host_id[MAX_LONG_OPT_SZ];
 /**< Enable metrics. */
@@ -637,28 +637,37 @@  main(int argc, char **argv)
 	argc += 3;
 
 	ret = rte_eal_init(argc, argp);
-	if (ret < 0)
+	if (ret < 0) {
+		close(stdout_fd);
 		rte_panic("Cannot init EAL\n");
+	}
 
 	argc -= ret;
 	argv += (ret - 3);
 
-	if (!rte_eal_primary_proc_alive(NULL))
+	if (!rte_eal_primary_proc_alive(NULL)) {
+		close(stdout_fd);
 		rte_exit(EXIT_FAILURE, "No primary DPDK process is running.\n");
+	}
 
 	/* parse app arguments */
 	ret = proc_info_parse_args(argc, argv);
-	if (ret < 0)
+	if (ret < 0) {
+		close(stdout_fd);
 		rte_exit(EXIT_FAILURE, "Invalid argument\n");
+	}
 
 	if (mem_info) {
 		meminfo_display();
+		close(stdout_fd);
 		return 0;
 	}
 
 	nb_ports = rte_eth_dev_count();
-	if (nb_ports == 0)
+	if (nb_ports == 0) {
+		close(stdout_fd);
 		rte_exit(EXIT_FAILURE, "No Ethernet ports - bye\n");
+	}
 
 	/* If no port mask was specified*/
 	if (enabled_port_mask == 0)
@@ -688,5 +697,7 @@  main(int argc, char **argv)
 	if (enable_metrics)
 		metrics_display(RTE_METRICS_GLOBAL);
 
+	close(stdout_fd);
+
 	return 0;
 }