[Bug sanitizer/82501] AddressSanitizer does not handle negative offset for first global variable

2019-04-03 Thread a.drobyshev at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82501 --- Comment #31 from Andrey Drobyshev --- (In reply to Jakub Jelinek from comment #30) > in a couple of most common data sections In which sections exactly? If we cover only the most common ones (thus leaving other sections which might need prot

[Bug sanitizer/82501] AddressSanitizer does not handle negative offset for first global variable

2019-03-21 Thread a.drobyshev at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82501 --- Comment #29 from Andrey Drobyshev --- (In reply to Segher Boessenkool from comment #28) > Patches should go to gcc-patches@. That is where reviews happen, too. That's still a patch candidate. I want to hear other folks' opinion before it go

[Bug sanitizer/82501] AddressSanitizer does not handle negative offset for first global variable

2019-03-21 Thread a.drobyshev at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82501 Andrey Drobyshev changed: What|Removed |Added Attachment #45751|0 |1 is obsolete|

[Bug sanitizer/82501] AddressSanitizer does not handle negative offset for first global variable

2019-03-18 Thread a.drobyshev at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82501 --- Comment #26 from Andrey Drobyshev --- (In reply to Jakub Jelinek from comment #24) > (In reply to Martin Liška from comment #23) > > (In reply to Andrey Drobyshev from comment #22) > > > Created attachment 45851 [details] > > > Work-in-progre

[Bug sanitizer/82501] AddressSanitizer does not handle negative offset for first global variable

2019-03-18 Thread a.drobyshev at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82501 --- Comment #25 from Andrey Drobyshev --- (In reply to Martin Liška from comment #23) > (In reply to Andrey Drobyshev from comment #22) > > Created attachment 45851 [details] > > Work-in-progress fix considering relocations > > > > I'm a bit stu

[Bug sanitizer/82501] AddressSanitizer does not handle negative offset for first global variable

2019-02-28 Thread a.drobyshev at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82501 --- Comment #22 from Andrey Drobyshev --- Created attachment 45851 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45851&action=edit Work-in-progress fix considering relocations I'm a bit stuck. I managed to precompute reloc value for the g

[Bug sanitizer/82501] AddressSanitizer does not handle negative offset for first global variable

2019-02-25 Thread a.drobyshev at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82501 --- Comment #19 from Andrey Drobyshev --- (In reply to Martin Liška from comment #17) > > 2. What should we do with sections like .data.rel.ro, .data.rel.ro.local? > > They suffer from this bug too, but it's not that easy to put globals there, >

[Bug sanitizer/82501] AddressSanitizer does not handle negative offset for first global variable

2019-02-25 Thread a.drobyshev at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82501 --- Comment #18 from Andrey Drobyshev --- (In reply to Martin Liška from comment #16) > Created attachment 45797 [details] > Patch candidate > > Patch candidate where I made some refactoring and come up with tests. > Works fine on x86_64, on ppc

[Bug sanitizer/82501] AddressSanitizer does not handle negative offset for first global variable

2019-02-21 Thread a.drobyshev at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82501 --- Comment #14 from Andrey Drobyshev --- (In reply to Martin Liška from comment #12) > However: > g++-8 -fsanitize=address global4.c -fno-common && ./a.out > = > ==12713==ERROR: A

[Bug sanitizer/82501] AddressSanitizer does not handle negative offset for first global variable

2019-02-18 Thread a.drobyshev at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82501 --- Comment #11 from Andrey Drobyshev --- (In reply to Martin Liška from comment #9) > (In reply to Andrey Drobyshev from comment #8) > > Great you've been working on that Andrey. > > > I recently started to work on this issue as well. I manage

[Bug sanitizer/82501] AddressSanitizer does not handle negative offset for first global variable

2019-02-18 Thread a.drobyshev at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82501 --- Comment #10 from Andrey Drobyshev --- Created attachment 45751 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45751&action=edit Work-in-progress fix This patch is pretty raw. It only handles .data, .rodata and .bss. It does not handle

[Bug sanitizer/82501] AddressSanitizer does not handle negative offset for first global variable

2019-01-29 Thread a.drobyshev at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82501 Andrey Drobyshev changed: What|Removed |Added CC||a.drobyshev at samsung dot com