diff mbox

Fix for invalid sanitization of trailing byte in __builtin_strlen

Message ID 53A85C12.10804@partner.samsung.com
State New
Headers show

Commit Message

max June 23, 2014, 4:55 p.m. UTC 
Hi,

when I applied this patch (r211846), I made a little mistake in output 
test patterns. This patch fixes this.

Tested on x86_64-unknown-linux-gnu.

Ok to commit?

-Maxim

Comments

Jeff Law June 23, 2014, 6:11 p.m. UTC | #1 
On 06/23/14 10:55, Maxim Ostapenko wrote:
> Hi,
>
> when I applied this patch (r211846), I made a little mistake in output
> test patterns. This patch fixes this.
>
> Tested on x86_64-unknown-linux-gnu.
>
> Ok to commit?
OK.
jeff
max June 24, 2014, 5:56 a.m. UTC | #2 
On 06/23/2014 10:11 PM, Jeff Law wrote:
> On 06/23/14 10:55, Maxim Ostapenko wrote:
>> Hi,
>>
>> when I applied this patch (r211846), I made a little mistake in output
>> test patterns. This patch fixes this.
>>
>> Tested on x86_64-unknown-linux-gnu.
>>
>> Ok to commit?
> OK.
> jeff
>
Thanks, done in r211927.

-Maxim
diff mbox

Patch

gcc/testsuite/ChangeLog:

2014-06-23  Max Ostapenko  <m.ostapenko@partner.samsung.com>

	* c-c++-common/asan/strlen-overflow-1.c: Change match patterns.

diff --git a/gcc/testsuite/c-c++-common/asan/strlen-overflow-1.c b/gcc/testsuite/c-c++-common/asan/strlen-overflow-1.c
index bf6bf66..f58f554 100644
--- a/gcc/testsuite/c-c++-common/asan/strlen-overflow-1.c
+++ b/gcc/testsuite/c-c++-common/asan/strlen-overflow-1.c
@@ -26,4 +26,5 @@  int main () {
 }
 
 /* { dg-output "READ of size 1 at 0x\[0-9a-f\]+ thread T0.*(\n|\r\n|\r)" } */
-/* { dg-output "    #0 0x\[0-9a-f\]+ (in _*main (\[^\n\r]*strlen-overflow-1.c:24|\[^\n\r]*:0)|\[(\]).*(\n|\r\n|\r)" } */
+/* { dg-output "    #0 0x\[0-9a-f\]+ (in _*main (\[^\n\r]*strlen-overflow-1.c:25|\[^\n\r]*:0)|\[(\]).*(\n|\r\n|\r)" } */
+/* { dg-output "\[^\n\r]*0x\[0-9a-f\]+ is located 1 bytes inside of global variable" } */