diff options
| author | Martin Sebor <msebor@redhat.com> | 2019-04-04 22:38:10 +0000 |
|---|---|---|
| committer | Martin Sebor <msebor@gcc.gnu.org> | 2019-04-04 16:38:10 -0600 |
| commit | 1a9b15a7d76ee3ee2cd960698a84e34ec10c2bf0 (patch) | |
| tree | b55605cb740d88748c0349945fe87a7b06634d12 /gcc/calls.c | |
| parent | 9a0cbb60d28042893135cee60ad28a48a7e2235d (diff) | |
| download | gcc-1a9b15a7d76ee3ee2cd960698a84e34ec10c2bf0.zip gcc-1a9b15a7d76ee3ee2cd960698a84e34ec10c2bf0.tar.gz gcc-1a9b15a7d76ee3ee2cd960698a84e34ec10c2bf0.tar.bz2 | |
PR middle-end/89957 - ICE calling strnlen with an int128_t bound in a known range
PR middle-end/89957 - ICE calling strnlen with an int128_t bound in a known range
PR middle-end/89911 - [9 Regression] ICE in get_attr_nonstring_decl
gcc/ChangeLog:
PR middle-end/89957
PR middle-end/89911
* builtins.c (expand_builtin_strnlen): Make sure wi::ltu_p operands
have the same precision since the function crashes otherwise.
* calls.c (maybe_warn_nonstring_arg): Avoid assuming strnlen() call
has non-zero arguments.
gcc/testsuite/ChangeLog:
PR middle-end/89957
PR middle-end/89911
* gcc.dg/Wstringop-overflow-13.c: New test.
From-SVN: r270154
Diffstat (limited to 'gcc/calls.c')
| -rw-r--r-- | gcc/calls.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/gcc/calls.c b/gcc/calls.c index 63c1bc5..6b22e7a 100644 --- a/gcc/calls.c +++ b/gcc/calls.c @@ -1555,7 +1555,10 @@ maybe_warn_nonstring_arg (tree fndecl, tree exp) if (TREE_NO_WARNING (exp) || !warn_stringop_overflow) return; + /* Avoid clearly invalid calls (more checking done below). */ unsigned nargs = call_expr_nargs (exp); + if (!nargs) + return; /* The bound argument to a bounded string function like strncpy. */ tree bound = NULL_TREE; |