diff options
author | Mathias Krause <minipli@googlemail.com> | 2014-08-27 20:28:56 +0930 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2014-08-27 21:54:11 +0930 |
commit | d93e1719a313ca960b38c5159be0106884317997 (patch) | |
tree | 10230aeafdbf5e41618c36b411bb2753e84702e7 /scripts | |
parent | 7a3ee7538598e0d60e6aa87dcf34a4e8a0adebc2 (diff) | |
download | lwn-d93e1719a313ca960b38c5159be0106884317997.tar.gz lwn-d93e1719a313ca960b38c5159be0106884317997.zip |
modpost: simplify file name generation of *.mod.c files
Avoid the variable length array (vla), just use PATH_MAX instead.
This not only makes this code clang friedly, it also leads to a
code size reduction:
text data bss dec hex filename
51765 2224 12416 66405 10365 scripts/mod/modpost.old
51677 2224 12416 66317 1030d scripts/mod/modpost.new
Signed-off-by: Mathias Krause <minipli@googlemail.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'scripts')
-rw-r--r-- | scripts/mod/modpost.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index c2ebdc788f4c..3b405c726ec5 100644 --- a/scripts/mod/modpost.c +++ b/scripts/mod/modpost.c @@ -2212,7 +2212,7 @@ int main(int argc, char **argv) err = 0; for (mod = modules; mod; mod = mod->next) { - char fname[strlen(mod->name) + 10]; + char fname[PATH_MAX]; if (mod->skip) continue; |