Page MenuHomePhabricator

marxin (Martin Liška)
User

Projects

User does not belong to any projects.

User Details

User Since
Feb 24 2016, 2:50 AM (261 w, 8 h)

Recent Activity

Yesterday

marxin added a comment to D97263: [PATCH] Fix UBSAN in __ubsan::Value::getSIntValue.

Can you try to add a test?

Tue, Feb 23, 2:11 AM · Restricted Project
marxin committed rG16ede0956cb1: Fix UBSAN in __ubsan::Value::getSIntValue (authored by marxin).
Fix UBSAN in __ubsan::Value::getSIntValue
Tue, Feb 23, 2:11 AM
marxin closed D97263: [PATCH] Fix UBSAN in __ubsan::Value::getSIntValue.
Tue, Feb 23, 2:10 AM · Restricted Project
marxin requested review of D97263: [PATCH] Fix UBSAN in __ubsan::Value::getSIntValue.
Tue, Feb 23, 1:13 AM · Restricted Project

Fri, Feb 19

marxin abandoned D95864: [PATCH] libsanitizer: Support non-POSIX extension REG_STARTEND..
Fri, Feb 19, 5:50 AM
marxin added inline comments to D95864: [PATCH] libsanitizer: Support non-POSIX extension REG_STARTEND..
Fri, Feb 19, 5:48 AM

Thu, Feb 18

marxin added inline comments to D96348: [sanitizers] Fix interception of GLibc regexec.
Thu, Feb 18, 11:46 PM · Restricted Project
marxin added a comment to D95864: [PATCH] libsanitizer: Support non-POSIX extension REG_STARTEND..

There is D96348

Thu, Feb 18, 11:44 PM
marxin added a reviewer for D95864: [PATCH] libsanitizer: Support non-POSIX extension REG_STARTEND.: vitalybuka.
Thu, Feb 18, 3:13 AM

Tue, Feb 9

marxin updated the diff for D95864: [PATCH] libsanitizer: Support non-POSIX extension REG_STARTEND..
Tue, Feb 9, 8:25 AM

Tue, Feb 2

marxin updated the summary of D95864: [PATCH] libsanitizer: Support non-POSIX extension REG_STARTEND..
Tue, Feb 2, 4:51 AM
marxin requested review of D95864: [PATCH] libsanitizer: Support non-POSIX extension REG_STARTEND..
Tue, Feb 2, 4:51 AM

Oct 20 2020

marxin committed rGad2be02a833e: ASAN: Support detect_invalid_pointer_pairs=1 with… (authored by marxin).
ASAN: Support detect_invalid_pointer_pairs=1 with…
Oct 20 2020, 10:29 AM
marxin closed D89552: ASAN: Support detect_invalid_pointer_pairs=1 with detect_stack_use_after_return=1.
Oct 20 2020, 10:29 AM · Restricted Project
marxin added inline comments to D89552: ASAN: Support detect_invalid_pointer_pairs=1 with detect_stack_use_after_return=1.
Oct 20 2020, 12:41 AM · Restricted Project

Oct 19 2020

marxin updated the diff for D89552: ASAN: Support detect_invalid_pointer_pairs=1 with detect_stack_use_after_return=1.
Oct 19 2020, 1:01 AM · Restricted Project
marxin added inline comments to D89552: ASAN: Support detect_invalid_pointer_pairs=1 with detect_stack_use_after_return=1.
Oct 19 2020, 1:01 AM · Restricted Project

Oct 16 2020

marxin requested review of D89552: ASAN: Support detect_invalid_pointer_pairs=1 with detect_stack_use_after_return=1.
Oct 16 2020, 6:40 AM · Restricted Project

Jun 1 2020

marxin committed rGb638b63b99d6: Move internal_uname to #if SANITIZER_LINUX scope. (authored by marxin).
Move internal_uname to #if SANITIZER_LINUX scope.
Jun 1 2020, 12:27 PM
marxin closed D80864: Move internal_uname to #if SANITIZER_LINUX scope..

Merged to master as b638b63b99d66786cb37336292604a2ae3490cfd.

Jun 1 2020, 12:25 PM
marxin added reviewers for D80864: Move internal_uname to #if SANITIZER_LINUX scope.: vitalybuka, kcc.
Jun 1 2020, 6:23 AM

May 30 2020

marxin created D80864: Move internal_uname to #if SANITIZER_LINUX scope..
May 30 2020, 1:33 AM
marxin added a comment to D80864: Move internal_uname to #if SANITIZER_LINUX scope..

It fixes multilib build in GCC on ppc64 after libsanitizer merge.

May 30 2020, 1:33 AM

Nov 27 2019

marxin committed rG2045d2c90e24: Make memory dump same as the one in asan. (authored by marxin).
Make memory dump same as the one in asan.
Nov 27 2019, 9:50 AM
marxin closed D70707: [hwasan] Make memory dump same as the one in asan..
Nov 27 2019, 9:50 AM · Restricted Project, Restricted Project
marxin added a comment to D70707: [hwasan] Make memory dump same as the one in asan..
In D70707#1760759, @pcc wrote:

Test case? (Do none of the tests really need to be updated?)

Nov 27 2019, 9:50 AM · Restricted Project, Restricted Project

Nov 26 2019

marxin added a comment to D70707: [hwasan] Make memory dump same as the one in asan..

Old report look:

Memory tags around the buggy address (one tag corresponds to 16 bytes):
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
=> d7  00 [08] 00  00  00  00  00  00  00  00  00  00  00  00  00 <=
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00   
Tags for short granules around the buggy address (one tag corresponds to 16 bytes):
   ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..   
=> ..  .. [be] ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  .. <=
   ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..  ..
Nov 26 2019, 1:29 AM · Restricted Project, Restricted Project
marxin created D70707: [hwasan] Make memory dump same as the one in asan..
Nov 26 2019, 1:29 AM · Restricted Project, Restricted Project

Jun 13 2019

marxin added a comment to D59876: Improve error message when '=' is missing in {ASAN,...}_OPTIONS..

Thank you for review. Can you please install the commit?

Jun 13 2019, 1:01 AM · Restricted Project

Jun 12 2019

marxin added inline comments to D59876: Improve error message when '=' is missing in {ASAN,...}_OPTIONS..
Jun 12 2019, 12:58 AM · Restricted Project
marxin updated the diff for D59876: Improve error message when '=' is missing in {ASAN,...}_OPTIONS..
Jun 12 2019, 12:58 AM · Restricted Project

Jun 11 2019

marxin updated the diff for D59876: Improve error message when '=' is missing in {ASAN,...}_OPTIONS..

Updated version of the patch.

Jun 11 2019, 1:58 AM · Restricted Project
marxin added inline comments to D59876: Improve error message when '=' is missing in {ASAN,...}_OPTIONS..
Jun 11 2019, 1:58 AM · Restricted Project

Jun 10 2019

marxin added a reviewer for D59876: Improve error message when '=' is missing in {ASAN,...}_OPTIONS.: vitalybuka.
Jun 10 2019, 3:03 AM · Restricted Project

May 16 2019

marxin updated the diff for D59876: Improve error message when '=' is missing in {ASAN,...}_OPTIONS..

Updated patch with a test-case.

May 16 2019, 2:38 AM · Restricted Project

Apr 8 2019

marxin added a comment to D59876: Improve error message when '=' is missing in {ASAN,...}_OPTIONS..

May I please ping this..

Apr 8 2019, 5:19 AM · Restricted Project

Mar 28 2019

marxin updated the diff for D59876: Improve error message when '=' is missing in {ASAN,...}_OPTIONS..

I've made the error message more verbose.

Mar 28 2019, 1:37 AM · Restricted Project

Mar 27 2019

marxin created D59876: Improve error message when '=' is missing in {ASAN,...}_OPTIONS..
Mar 27 2019, 6:16 AM · Restricted Project

Mar 6 2019

marxin committed rGa25a2c7c9a7e: Always compare C++ typeinfo (based on libstdc++ implementation). (authored by marxin).
Always compare C++ typeinfo (based on libstdc++ implementation).
Mar 6 2019, 12:36 AM
marxin committed rL355488: Always compare C++ typeinfo (based on libstdc++ implementation)..
Always compare C++ typeinfo (based on libstdc++ implementation).
Mar 6 2019, 12:36 AM
marxin committed rCRT355488: Always compare C++ typeinfo (based on libstdc++ implementation)..
Always compare C++ typeinfo (based on libstdc++ implementation).
Mar 6 2019, 12:36 AM
marxin closed D58028: Always compare C++ typeinfo (based on libstdc++ implementation)..
Mar 6 2019, 12:36 AM · Restricted Project, Restricted Project

Mar 5 2019

marxin added a comment to D58028: Always compare C++ typeinfo (based on libstdc++ implementation)..

Update version where I address last comment.

Mar 5 2019, 4:06 AM · Restricted Project, Restricted Project

Mar 1 2019

marxin updated the diff for D58028: Always compare C++ typeinfo (based on libstdc++ implementation)..

Update version where I address last comment.

Mar 1 2019, 1:25 AM · Restricted Project, Restricted Project

Feb 26 2019

marxin added a comment to D58028: Always compare C++ typeinfo (based on libstdc++ implementation)..

Gentle ping.

Feb 26 2019, 2:27 AM · Restricted Project, Restricted Project

Feb 21 2019

marxin updated the diff for D58028: Always compare C++ typeinfo (based on libstdc++ implementation)..

Updated version which preserves behavior on MACOS and IOS. For other targets SANITIZER_NON_UNIQUE_TYPEINFO is set to true.

Feb 21 2019, 1:55 AM · Restricted Project, Restricted Project

Feb 19 2019

marxin added a comment to D58028: Always compare C++ typeinfo (based on libstdc++ implementation)..

I do not believe this change is correct for MacOS or iOS. The original bug does not reproduce on MacOS, and on arm64 iOS you always need to compare strings. Could you preserve the existing behavior for those platforms?

Feb 19 2019, 3:45 AM · Restricted Project, Restricted Project

Feb 14 2019

marxin added a comment to D58028: Always compare C++ typeinfo (based on libstdc++ implementation)..

There hasn't been any reply yet. May I understand that as approval?

Feb 14 2019, 1:31 AM · Restricted Project, Restricted Project

Feb 11 2019

marxin created D58028: Always compare C++ typeinfo (based on libstdc++ implementation)..
Feb 11 2019, 12:36 AM · Restricted Project, Restricted Project
marxin abandoned D56485: Always compare C++ typeinfo (based on libstdc++ implementation)..
Feb 11 2019, 12:33 AM

Feb 5 2019

marxin added a comment to D56485: Always compare C++ typeinfo (based on libstdc++ implementation)..

Hi, there's no reply from the people who were added to this review.
May I understand that as ready to be installed?

filcab is still marked as blocking this. Also I thought you were going to file a new differential that had the commits list cc’ed correctly? See above where I asked you to add a couple of relevant reviewers to the new differential (and me as a subscriber) when you do.

Feb 5 2019, 12:07 AM
marxin added reviewers for D56485: Always compare C++ typeinfo (based on libstdc++ implementation).: Bigcheese, dcoughlin.
Feb 5 2019, 12:05 AM

Feb 4 2019

marxin added a comment to D56485: Always compare C++ typeinfo (based on libstdc++ implementation)..

Hi, there's no reply from the people who were added to this review.
May I understand that as ready to be installed?

Feb 4 2019, 6:12 AM

Jan 24 2019

marxin added a comment to D56485: Always compare C++ typeinfo (based on libstdc++ implementation)..

I would suggest to revert the commit, and submit the new differential with properly subscribed lists this time.
At least that is the 'recommended' general practice.

Jan 24 2019, 12:46 AM
marxin committed rL352033: Revert https://reviews.llvm.org/D56485..
Revert https://reviews.llvm.org/D56485.
Jan 24 2019, 12:46 AM
marxin committed rCRT352033: Revert https://reviews.llvm.org/D56485..
Revert https://reviews.llvm.org/D56485.
Jan 24 2019, 12:46 AM
marxin added a comment to D56485: Always compare C++ typeinfo (based on libstdc++ implementation)..

Please wait 1 or 2 days (to see if there are objections), and then you can consider the patch LGTMed.

Except the lists aren't subscibed, thus no feedback can happen :S
So this wasn't really reviewed.

Jan 24 2019, 12:41 AM
marxin committed rCRT352032: Always compare C++ typeinfo (based on libstdc++ implementation)..
Always compare C++ typeinfo (based on libstdc++ implementation).
Jan 24 2019, 12:25 AM
marxin committed rL352032: Always compare C++ typeinfo (based on libstdc++ implementation)..
Always compare C++ typeinfo (based on libstdc++ implementation).
Jan 24 2019, 12:25 AM

Jan 21 2019

marxin added a comment to D56485: Always compare C++ typeinfo (based on libstdc++ implementation)..
In D56485#1364241, @pcc wrote:

This change would only impact the "diagnostic" (non-production) mode of CFI. In production builds we do not link the runtime library at all. So from a CFI perspective this is fine.

Well, that makes sense. CFI was my biggest problem with this, so I'm tempted to accept the patch (The * doesn't seem to be a valid character for Itanium C++ ABI names, so I think having the test there should only trigger on gcc code).
Please wait 1 or 2 days (to see if there are objections), and then you can consider the patch LGTMed. I'll try to poke it on tuesday/wednesday.

Thank you,
Filipe

Jan 21 2019, 3:28 AM

Jan 18 2019

marxin added a comment to D56485: Always compare C++ typeinfo (based on libstdc++ implementation)..

Sample code I used:

namespace { struct S { S () {} ~S () {} virtual void foo () { asm volatile (""); } }; }
namespace { struct T : public S { T () {} ~T () {} virtual void foo () { asm volatile ("nop"); } }; }
void bar (S &s) { s.foo (); }
void baz () { S s; T t; bar (s); bar (t); }
Jan 18 2019, 1:48 AM
marxin added inline comments to D56485: Always compare C++ typeinfo (based on libstdc++ implementation)..
Jan 18 2019, 1:43 AM

Jan 16 2019

marxin added a reviewer for D56485: Always compare C++ typeinfo (based on libstdc++ implementation).: filcab.
Jan 16 2019, 11:51 PM
marxin added a reviewer for D11502: [UBSan] Fix isDerivedFromAtOffset on iOS ARM64: filcab.
Jan 16 2019, 11:50 PM

Jan 14 2019

marxin added a reviewer for D56485: Always compare C++ typeinfo (based on libstdc++ implementation).: alekseyshl.
Jan 14 2019, 5:16 AM

Jan 9 2019

marxin added a reviewer for D56485: Always compare C++ typeinfo (based on libstdc++ implementation).: kcc.
Jan 9 2019, 3:36 AM
marxin created D56485: Always compare C++ typeinfo (based on libstdc++ implementation)..
Jan 9 2019, 3:36 AM
marxin closed D54856: Do not rely on that subject of ErrorAllocTypeMismatch is a heap address..

It's installed.

Jan 9 2019, 3:33 AM

Dec 27 2018

marxin committed rL350085: Do not rely on that subject of ErrorAllocTypeMismatch is a heap address..
Do not rely on that subject of ErrorAllocTypeMismatch is a heap address.
Dec 27 2018, 12:43 AM
marxin committed rCRT350085: Do not rely on that subject of ErrorAllocTypeMismatch is a heap address..
Do not rely on that subject of ErrorAllocTypeMismatch is a heap address.
Dec 27 2018, 12:42 AM

Nov 30 2018

marxin updated the diff for D54856: Do not rely on that subject of ErrorAllocTypeMismatch is a heap address..

The right patch, previous one belongs to GCC :)

Nov 30 2018, 2:57 AM
marxin updated the diff for D54856: Do not rely on that subject of ErrorAllocTypeMismatch is a heap address..

Updated version of the patch.

Nov 30 2018, 2:56 AM
marxin added a comment to D54856: Do not rely on that subject of ErrorAllocTypeMismatch is a heap address..

Thanks for testing Vitaly. I must confirm that i386 is broken on my openSUSE, thus I can't test that.

Nov 30 2018, 2:55 AM

Nov 28 2018

marxin updated the diff for D54856: Do not rely on that subject of ErrorAllocTypeMismatch is a heap address..

Updated patch that introduces 2 new tests.

Nov 28 2018, 4:53 AM

Nov 27 2018

marxin added a comment to D54856: Do not rely on that subject of ErrorAllocTypeMismatch is a heap address..

@vitalybuka: I modified the patch based on your notes.

Nov 27 2018, 6:03 AM
marxin updated the diff for D54856: Do not rely on that subject of ErrorAllocTypeMismatch is a heap address..

Updated patch, I also found a test-case that fails for clang before my patch:

Nov 27 2018, 6:02 AM

Nov 23 2018

marxin created D54856: Do not rely on that subject of ErrorAllocTypeMismatch is a heap address..
Nov 23 2018, 4:19 AM

Nov 5 2018

marxin committed rCRT346129: Fix build on sparc64-linux-gnu..
Fix build on sparc64-linux-gnu.
Nov 5 2018, 4:30 AM
marxin committed rL346129: Fix build on sparc64-linux-gnu..
Fix build on sparc64-linux-gnu.
Nov 5 2018, 4:30 AM
marxin closed D54030: Fix build on sparc64-linux-gnu..
Nov 5 2018, 4:30 AM

Nov 4 2018

marxin updated the diff for D54030: Fix build on sparc64-linux-gnu..
Nov 4 2018, 11:45 PM
marxin added a comment to D54030: Fix build on sparc64-linux-gnu..
In D54030#1286807, @ro wrote:

Overall, this looks good to me, with the nits noted.

However, I wonder how you tested this:

  • SPARC isn't among cmake/config-ix.cmake (ALL_ASAN_SUPPORTED_ARCH); in fact there's no compiler-rt support for SPARC at all.
  • In gcc, it's similar: there's no definition of TARGET_ASAN_SHADOW_OFFSET in the SPARC backend yet. I've submitted one for Solaris/SPARC in PR sanitizer/80953 and hope to finish and post it soon, but right now there's nothing.

Wondering...

Nov 4 2018, 11:45 PM

Nov 2 2018

marxin created D54030: Fix build on sparc64-linux-gnu..
Nov 2 2018, 2:13 AM

Oct 23 2018

marxin committed rL345006: Come up with MIN_PERCENTAGE_CHANGE value..
Come up with MIN_PERCENTAGE_CHANGE value.
Oct 23 2018, 12:11 AM
marxin closed D53136: [LNT] Come up with MIN_PERCENTAGE_CHANGE value.
Oct 23 2018, 12:11 AM

Oct 11 2018

marxin created D53136: [LNT] Come up with MIN_PERCENTAGE_CHANGE value.
Oct 11 2018, 5:49 AM
marxin abandoned D51609: [LNT] Enable X-Forwarded-* headers to be accepted..
Oct 11 2018, 5:47 AM

Oct 1 2018

marxin committed rL343504: Fix fallout caused by YAML files ordering..
Fix fallout caused by YAML files ordering.
Oct 1 2018, 10:23 AM
marxin closed D52723: [LNT] Fix fallout caused by YAML files ordering..
Oct 1 2018, 10:23 AM
marxin created D52723: [LNT] Fix fallout caused by YAML files ordering..
Oct 1 2018, 6:05 AM
marxin committed rL343467: Install missing files..
Install missing files.
Oct 1 2018, 5:40 AM
marxin committed rL343466: Sort fields according to order defined in YAML files..
Sort fields according to order defined in YAML files.
Oct 1 2018, 5:34 AM
marxin closed D52593: [LNT] Sort fields according to order defined in YAML files..
Oct 1 2018, 5:34 AM
marxin committed rL343465: Use unit_abbrev of Fields in reports..
Use unit_abbrev of Fields in reports.
Oct 1 2018, 5:33 AM
marxin closed D51916: [LNT] Use unit_abbrev of Fields in reports..
Oct 1 2018, 5:33 AM
marxin committed rL343464: Improve alignment of HTML report..
Improve alignment of HTML report.
Oct 1 2018, 5:28 AM
marxin closed D51915: [LNT] Improve alignment of HTML report..
Oct 1 2018, 5:28 AM
marxin committed rL343463: Print machine name in LNT graph tooltip..
Print machine name in LNT graph tooltip.
Oct 1 2018, 5:25 AM
marxin closed D52346: [LNT] Print machine name in LNT graph tooltip..
Oct 1 2018, 5:24 AM
marxin committed rL343462: Introduce Latest Runs Report..
Introduce Latest Runs Report.
Oct 1 2018, 5:22 AM