clang emits a warning in this function as we're using a boolean as the third argument to strncmp. Indeed, this function only checks the first byte of the path as is, so files beginning with . will be incorrectly included from the current working directory instead of the include directory.
Fixes: f92a1a5c4a8761c39a76b264859f0ff2c8f71179 Signed-off-by: Anatole Denis <[email protected]> --- src/scanner.l | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/scanner.l b/src/scanner.l index 625023f..bf88955 100644 --- a/src/scanner.l +++ b/src/scanner.l @@ -629,8 +629,8 @@ err: static bool search_in_include_path(const char *filename) { - return (strncmp(filename, "./", strlen("./") != 0) && - strncmp(filename, "../", strlen("../") != 0) && + return (strncmp(filename, "./", strlen("./")) != 0 && + strncmp(filename, "../", strlen("../")) != 0 && filename[0] != '/'); } -- 2.11.0 -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
