diff options
author | Martin Sebor <msebor@redhat.com> | 2020-11-29 15:09:30 -0700 |
---|---|---|
committer | Martin Sebor <msebor@redhat.com> | 2020-11-29 15:12:32 -0700 |
commit | eafe8ee7af13c39805ea09bbf5b4f9ab2a48304a (patch) | |
tree | 532502ae8d62a6e3ef875f76fe913efdfb146388 /gcc/testsuite/gcc.dg/Wstringop-overflow-7.c | |
parent | 4dce3b05ec96a5f76adec23dbe92b014f9db8554 (diff) | |
download | gcc-eafe8ee7af13c39805ea09bbf5b4f9ab2a48304a.zip gcc-eafe8ee7af13c39805ea09bbf5b4f9ab2a48304a.tar.gz gcc-eafe8ee7af13c39805ea09bbf5b4f9ab2a48304a.tar.bz2 |
Handle PHIs in compute_objsize.
PR middle-end/92936 - missing warning on a past-the-end store to a PHI
PR middle-end/92940 - incorrect offset and size in -Wstringop-overflow for out-of-bounds store into VLA and two offset ranges
PR middle-end/89428 - missing -Wstringop-overflow on a PHI with variable offset
gcc/ChangeLog:
PR middle-end/92936
PR middle-end/92940
PR middle-end/89428
* builtins.c (access_ref::access_ref): Initialize member.
(access_ref::phi): New function.
(access_ref::get_ref): New function.
(access_ref::add_offset): Remove duplicate assignment.
(maybe_warn_for_bound): Add "maybe" kind of warning messages.
(warn_for_access): Same.
(inform_access): Rename...
(access_ref::inform_access): ...to this. Print PHI arguments. Format
offset the same as size and simplify. Improve printing of allocation
functions and VLAs.
(check_access): Adjust to the above.
(gimple_parm_array_size): Change argument.
(handle_min_max_size): New function.
* builtins.h (class ssa_name_limit_t): Move class here from
tree-ssa-strlen.c.
(struct access_ref): Declare new members.
(gimple_parm_array_size): Change argument.
* tree-ssa-strlen.c (maybe_warn_overflow): Use access_ref and simplify.
(handle_builtin_memcpy): Correct argument passed to maybe_warn_overflow.
(handle_builtin_memset): Same.
(class ssa_name_limit_t): Move class to builtins.{h,c}.
gcc/testsuite/ChangeLog:
PR middle-end/92936
PR middle-end/92940
PR middle-end/89428
* c-c++-common/Wstringop-overflow-2.c: Adjust text of expected
informational notes.
* g++.dg/warn/Wstringop-overflow-3.C: Same.
* g++.dg/warn/Wplacement-new-size.C: Remove a test for a no longer
issued warning.
* gcc.dg/Warray-bounds-43.c: Removed unused declarations.
* gcc.dg/Wstringop-overflow-11.c: Remove xfails.
* gcc.dg/Wstringop-overflow-12.c: Same.
* gcc.dg/Wstringop-overflow-17.c: Adjust text of expected messages.
* gcc.dg/Wstringop-overflow-27.c: Same. Remove xfails.
* gcc.dg/Wstringop-overflow-28.c: Adjust text of expected messages.
* gcc.dg/Wstringop-overflow-29.c: Same.
* gcc.dg/Wstringop-overflow-37.c: Same.
* gcc.dg/Wstringop-overflow-46.c: Same.
* gcc.dg/Wstringop-overflow-47.c: Same.
* gcc.dg/Wstringop-overflow-54.c: Same.
* gcc.dg/warn-strnlen-no-nul.c: Add expected warning.
* gcc.dg/Wstringop-overflow-7.c: New test.
* gcc.dg/Wstringop-overflow-58.c: New test.
* gcc.dg/Wstringop-overflow-59.c: New test.
* gcc.dg/Wstringop-overflow-60.c: New test.
* gcc.dg/Wstringop-overflow-61.c: New test.
* gcc.dg/Wstringop-overflow-62.c: New test.
* gcc.dg/Wstringop-overflow-63.c: New test.
* gcc.dg/Wstringop-overflow-64.c: New test.
Diffstat (limited to 'gcc/testsuite/gcc.dg/Wstringop-overflow-7.c')
-rw-r--r-- | gcc/testsuite/gcc.dg/Wstringop-overflow-7.c | 124 |
1 files changed, 124 insertions, 0 deletions
diff --git a/gcc/testsuite/gcc.dg/Wstringop-overflow-7.c b/gcc/testsuite/gcc.dg/Wstringop-overflow-7.c new file mode 100644 index 0000000..cb2addf3 --- /dev/null +++ b/gcc/testsuite/gcc.dg/Wstringop-overflow-7.c @@ -0,0 +1,124 @@ +/* Test to verify that --param ssa_name_def_chain_limit can be used to + limit the maximum number of SSA_NAME assignments the warning follows. + { dg-do compile } + { dg-options "-O2 -Wall -Wno-array-bounds --param ssa-name-def-chain-limit=5" } */ + +#define NOIPA __attribute__ ((noipa)) + +void* memset (void*, int, __SIZE_TYPE__); + +char a9[9]; + +void sink (const char*, ...); + +NOIPA void g2 (int i) +{ + if (i < 1) i = 1; + + char *p0 = a9; + char *p1 = p0 + i; + char *p2 = p1 + i; + + sink (p0, p1, p2); + + memset (p2, 0, 8); // { dg-warning "\\\[-Wstringop-overflow" } +} + +NOIPA void g3 (int i) +{ + if (i < 1) i = 1; + + char *p0 = a9; + char *p1 = p0 + i; + char *p2 = p1 + i; + char *p3 = p2 + i; + + sink (p0, p1, p2, p3); + + memset (p3, 0, 7); // { dg-warning "\\\[-Wstringop-overflow" } +} + +NOIPA void g4 (int i) +{ + if (i < 1) i = 1; + + char *p0 = a9; + char *p1 = p0 + i; + char *p2 = p1 + i; + char *p3 = p2 + i; + char *p4 = p3 + i; + + sink (p0, p1, p2, p3, p4); + + memset (p4, 0, 6); // { dg-warning "\\\[-Wstringop-overflow" } +} + +NOIPA void g5 (int i) +{ + if (i < 1) i = 1; + + char *p0 = a9; + char *p1 = p0 + i; + char *p2 = p1 + i; + char *p3 = p2 + i; + char *p4 = p3 + i; + char *p5 = p4 + i; + + sink (p0, p1, p2, p3, p4, p5); + + memset (p5, 0, 5); // { dg-warning "\\\[-Wstringop-overflow" } +} + +NOIPA void g6 (int i) +{ + if (i < 1) i = 1; + + char *p0 = a9; + char *p1 = p0 + i; + char *p2 = p1 + i; + char *p3 = p2 + i; + char *p4 = p3 + i; + char *p5 = p4 + i; + char *p6 = p5 + i; + + sink (p0, p1, p2, p3, p4, p5, p6); + + memset (p6, 0, 4); +} + +NOIPA void g7 (int i) +{ + if (i < 1) i = 1; + + char *p0 = a9; + char *p1 = p0 + i; + char *p2 = p1 + i; + char *p3 = p2 + i; + char *p4 = p3 + i; + char *p5 = p4 + i; + char *p6 = p5 + i; + char *p7 = p6 + i; + + sink (p0, p1, p2, p3, p4, p5, p6, p7); + + memset (p7, 0, 4); +} + +NOIPA void g8 (int i) +{ + if (i < 1) i = 1; + + char *p0 = a9; + char *p1 = p0 + i; + char *p2 = p1 + i; + char *p3 = p2 + i; + char *p4 = p3 + i; + char *p5 = p4 + i; + char *p6 = p5 + i; + char *p7 = p6 + i; + char *p8 = p7 + i; + + sink (p0, p1, p2, p3, p4, p5, p6, p7, p8); + + memset (p8, 0, 2); +} |