]> git.baikalelectronics.ru Git - kernel.git/commit
tools/nolibc/stdio: Add format attribute to enable printf warnings
authorAlviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Thu, 19 May 2022 17:21:16 +0000 (00:21 +0700)
committerPaul E. McKenney <paulmck@kernel.org>
Mon, 20 Jun 2022 16:43:19 +0000 (09:43 -0700)
commitda64e2b9f5f01f5d14f317e7557ea8f40acff1c8
tree07ab404e00fe2ba8d62e7413a8230ccf526abacb
parent2ffa3fafe874ab6f52ee069faf3aed9869b74c70
tools/nolibc/stdio: Add format attribute to enable printf warnings

When we use printf and fprintf functions from the nolibc, we don't
get any warning from the compiler if we have the wrong arguments.
For example, the following calls will compile silently:
```
  printf("%s %s\n", "aaa");
  fprintf(stdout, "%s %s\n", "xxx", 1);
```
(Note the wrong arguments).

Those calls are undefined behavior. The compiler can help us warn
about the above mistakes by adding a `printf` format attribute to
those functions declaration. This patch adds it, and now it yields
these warnings for those mistakes:
```
  warning: format `%s` expects a matching `char *` argument [-Wformat=]
  warning: format `%s` expects argument of type `char *`, but argument 4 has type `int` [-Wformat=]
```

  [ ammarfaizi2: Simplify the attribute placement. ]

Signed-off-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
Acked-by: Willy Tarreau <w@1wt.eu>
Signed-off-by: Paul E. McKenney <paulmck@kernel.org>
tools/include/nolibc/stdio.h