ASSERT_WITH_SECURITY_IMPLICATION currently calls CRASH(). There are many other non security related code paths that call CRASH(). ASSERT_WITH_SECURITY_IMPLICATION should crash in a distinct way in order to help screen fuzzing bugs. The attached patch adds a CRASH_WITH_SECURITY_IMPLICATION which is the same as CRASH except it accesses 0xfbadbeef instead of 0xbbadbeef. The same as before, in release builds, ASSERT_WITH_SECURITY_IMPLICATION will do nothing unless it was built with ASAN, which required moving the code around a little, since before it just called ASSERT() when ASAN was not defined.
Created attachment 217641 [details] Patch
Attachment 217641 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WTF/ChangeLog', u'Source/WTF/wtf/Assertions.cpp', u'Source/WTF/wtf/Assertions.h']" exit_code: 1 Source/WTF/ChangeLog:6: Line contains tab character. [whitespace/tab] [5] Source/WTF/ChangeLog:7: Line contains tab character. [whitespace/tab] [5] Source/WTF/wtf/Assertions.h:238: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] Source/WTF/wtf/Assertions.h:279: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] Total errors found: 4 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 217643 [details] Patch
I updated the patch to fix the whitespace issues in the ChangeLog. Re: the whitespace issues in Assertions.h, the whitespace I used is exactly the same as was already used for the existing code.
Attachment 217643 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WTF/ChangeLog', u'Source/WTF/wtf/Assertions.cpp', u'Source/WTF/wtf/Assertions.h']" exit_code: 1 Source/WTF/wtf/Assertions.h:238: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] Source/WTF/wtf/Assertions.h:279: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] Total errors found: 2 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 217643 [details] Patch r=me
Comment on attachment 217643 [details] Patch Clearing flags on attachment: 217643 Committed r159799: <http://trac.webkit.org/changeset/159799>
All reviewed patches have been landed. Closing bug.