[Bug 1580348] [NEW] mksh 52c bi_errorf(Tbadsubst) format string is not a string literal
1580348 at bugs.launchpad.net
Tue May 10 22:50:14 UTC 2016
Chih-Hung Hsieh dixit:
>Lastest mksh/histrap.c (R52c) has a warning from clang/llvm compiler.
>It is a tricky use of pointer to the middle of string literals,
>which is recognized by gcc but not clang/llvm.
Arrgh, stupid compilers, trying to be too smart and utterly failing.
I’ve recently had “fun” with newer GCC versions and those string
pooling macros, TWICE. I already decided to remove them, but for
now I’ve added GCC workarounds… turns out Clang is just as bad.
Yes, I’ll tackle this for the next release, thank you for the report.
22:20⎜<asarch> The crazy that persists in his craziness becomes a master
22:21⎜<asarch> And the distance between the craziness and geniality is
only measured by the success 18:35⎜<asarch> "Psychotics are consistently
inconsistent. The essence of sanity is to be inconsistently inconsistent
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to mksh in Ubuntu.
mksh 52c bi_errorf(Tbadsubst) format string is not a string literal
Status in mksh package in Ubuntu:
Lastest mksh/histrap.c (R52c) has a warning from clang/llvm compiler.
It is a tricky use of pointer to the middle of string literals,
which is recognized by gcc but not clang/llvm.
This warning now blocks mksh upgrade in Android open source.
Could you fix the following warning and other places that use
bi_errorf(Tbadsubst) or internal_errorf(Tbadsubst)?
histrap.c:220:15: error: format string is not a string literal (potentially insecure) [-Werror,-Wformat-security]
sh.h:891:19: note: expanded from macro 'Tbadsubst'
#define Tbadsubst (Tfg_badsubst + 10) /* "bad substitution" */
The warning can be fixed by changing
If adding a few bytes is not acceptable, could you change
where bi_errorf0 is declared as
void bi_errorf0(const char *); // without format check
and implemented as a weak alias:
void bi_errorf0(const char *s) __attribute__((weak, alias("bi_errorf")));
Attached file is a suggested patch to compile with Android.
To manage notifications about this bug go to:
More information about the foundations-bugs