fixdep: remove unneeded memcpy() in parse_dep_file()
authorMasahiro Yamada <yamada.masahiro@socionext.com>
Thu, 11 Jan 2018 13:05:43 +0000 (22:05 +0900)
committerMasahiro Yamada <yamada.masahiro@socionext.com>
Thu, 18 Jan 2018 00:37:38 +0000 (09:37 +0900)
Each token in the depfile is copied to the temporary buffer 's' to
terminate the token with zero.  We do not need to do this any more
because the parsed buffer is now writable.  Insert '\0' directly in
the buffer without calling memcpy().

<limits.h> is no longer necessary. (It was needed for PATH_MAX).

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
scripts/basic/fixdep.c

index 9f9238eaec19abd5b423274172a778512b60e99a..dfba77b5d0c3ced54d366bfa6877e95f2ff9f878 100644 (file)
 #include <string.h>
 #include <stdlib.h>
 #include <stdio.h>
-#include <limits.h>
 #include <ctype.h>
 
 int insert_extra_deps;
@@ -304,7 +303,6 @@ static void *read_file(const char *filename)
 static void parse_dep_file(char *m)
 {
        char *p;
-       char s[PATH_MAX];
        int is_last, is_target;
        int saw_any_target = 0;
        int is_first_dep = 0;
@@ -330,16 +328,14 @@ static void parse_dep_file(char *m)
                        /* The /next/ file is the first dependency */
                        is_first_dep = 1;
                } else {
-                       /* Save this token/filename */
-                       memcpy(s, m, p-m);
-                       s[p - m] = 0;
+                       *p = '\0';
 
                        /* Ignore certain dependencies */
-                       if (strrcmp(s, "include/generated/autoconf.h") &&
-                           strrcmp(s, "include/generated/autoksyms.h") &&
-                           strrcmp(s, "arch/um/include/uml-config.h") &&
-                           strrcmp(s, "include/linux/kconfig.h") &&
-                           strrcmp(s, ".ver")) {
+                       if (strrcmp(m, "include/generated/autoconf.h") &&
+                           strrcmp(m, "include/generated/autoksyms.h") &&
+                           strrcmp(m, "arch/um/include/uml-config.h") &&
+                           strrcmp(m, "include/linux/kconfig.h") &&
+                           strrcmp(m, ".ver")) {
                                /*
                                 * Do not list the source file as dependency,
                                 * so that kbuild is not confused if a .c file
@@ -360,15 +356,15 @@ static void parse_dep_file(char *m)
                                        if (!saw_any_target) {
                                                saw_any_target = 1;
                                                printf("source_%s := %s\n\n",
-                                                       target, s);
+                                                       target, m);
                                                printf("deps_%s := \\\n",
                                                        target);
                                        }
                                        is_first_dep = 0;
                                } else
-                                       printf("  %s \\\n", s);
+                                       printf("  %s \\\n", m);
 
-                               buf = read_file(s);
+                               buf = read_file(m);
                                parse_config_file(buf);
                                free(buf);
                        }