diff mbox series

[v2] app/proc-info:fix port mask parse bug

Message ID 1541473466-27118-1-git-send-email-han.li1@zte.com.cn (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series [v2] app/proc-info:fix port mask parse bug |

Checks

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

Commit Message

Li Han Nov. 6, 2018, 3:04 a.m. UTC 
  parse_portmask return type is int,but global variable
"enabled_port_mask" type is uint32_t.so in proc_info_parse_args
function,when parse_portmask return -1,"enabled_port_mask" will
get a huge value and "if (enabled_port_mask == 0)" will never happen.

Signed-off-by: Li Han <han.li1@zte.com.cn>

---

v2:fix typecast issue
---
 app/proc-info/main.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

Comments

Pattan, Reshma Nov. 6, 2018, 10:39 a.m. UTC | #1 
Hi,

> -----Original Message-----
> From: Li Han [mailto:han.li1@zte.com.cn]
> Sent: Tuesday, November 6, 2018 3:04 AM
> To: Pattan, Reshma <reshma.pattan@intel.com>
> Cc: dev@dpdk.org; Li Han <han.li1@zte.com.cn>
> Subject: [PATCH v2] [dpdk-dev] app/proc-info:fix port mask parse bug
> 
> parse_portmask return type is int,but global variable "enabled_port_mask" type
> is uint32_t.so in proc_info_parse_args function,when parse_portmask return -
> 1,"enabled_port_mask" will get a huge value and "if (enabled_port_mask == 0)"
> will never happen.

1) Please add Fixes line for the commit id that introduced this bug. 
Refer below link to know  more on Fixes line, http://doc.dpdk.org/guides/contributing/patches.html
Ex: Fixes: commit id ("commit message")

2) Title of the patch just can be  "fix port mask parsing".  No need to say bug. 
	And give space after  ': ' in the heading.

When I ran below script after applying your patches above 2 issues been reported, so you should make sure to run the below script before sending the patch. 

$devtools/check-git-log.sh
Wrong headline format:
        app/proc-info:fix port mask parse bug
Missing 'Fixes' tag:
        app/proc-info:fix port mask parse bug

Other than that , Please keep my ack in next version

Acked-by: Reshma Pattan <reshma.pattan@intel.com>

Thanks,
Reshma
diff mbox series

Patch

diff --git a/app/proc-info/main.c b/app/proc-info/main.c
index c20effa..650d599 100644
--- a/app/proc-info/main.c
+++ b/app/proc-info/main.c
@@ -37,7 +37,7 @@ 
 #define MAX_STRING_LEN 256
 
 /**< mask of enabled ports */
-static uint32_t enabled_port_mask;
+static uint64_t enabled_port_mask;
 /**< Enable stats. */
 static uint32_t enable_stats;
 /**< Enable xstats. */
@@ -90,7 +90,7 @@ 
 /*
  * Parse the portmask provided at run time.
  */
-static int
+static unsigned long
 parse_portmask(const char *portmask)
 {
 	char *end = NULL;
@@ -103,12 +103,9 @@ 
 	if ((portmask[0] == '\0') || (end == NULL) || (*end != '\0') ||
 		(errno != 0)) {
 		printf("%s ERROR parsing the port mask\n", __func__);
-		return -1;
+		return 0;
 	}
 
-	if (pm == 0)
-		return -1;
-
 	return pm;
 
 }