From b4f60f55570468619100552b4e1fa96e337826f6 Mon Sep 17 00:00:00 2001 From: Tomasz Kulasek Date: Fri, 16 Jan 2015 15:27:52 +0100 Subject: [PATCH] eal/linux: check fscanf return when parsing modules list The lack of result checking of fscanf function, breaks compilation for default "-Werror=unused-result" flag. Signed-off-by: Tomasz Kulasek Acked-by: Pablo de Lara --- lib/librte_eal/linuxapp/eal/eal.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/linuxapp/eal/eal.c b/lib/librte_eal/linuxapp/eal/eal.c index 648ef81401..f99e158638 100644 --- a/lib/librte_eal/linuxapp/eal/eal.c +++ b/lib/librte_eal/linuxapp/eal/eal.c @@ -865,6 +865,7 @@ rte_eal_check_module(const char *module_name) { char mod_name[30]; /* Any module names can be longer than 30 bytes? */ int ret = 0; + int n; if (NULL == module_name) return -1; @@ -876,8 +877,8 @@ rte_eal_check_module(const char *module_name) return -1; } while (!feof(fd)) { - fscanf(fd, "%29s %*[^\n]", mod_name); - if (!strcmp(mod_name, module_name)) { + n = fscanf(fd, "%29s %*[^\n]", mod_name); + if ((n == 1) && !strcmp(mod_name, module_name)) { ret = 1; break; } -- 2.20.1