Bug 124757 - ASSERT_WITH_SECURITY_IMPLICATION should crash in a distinct way
Summary: ASSERT_WITH_SECURITY_IMPLICATION should crash in a distinct way
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Template Framework (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Drew Yao
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-11-21 18:50 PST by Drew Yao
Modified: 2013-11-26 19:32 PST (History)
3 users (show)

See Also:


Attachments
Patch (3.33 KB, patch)
2013-11-21 18:54 PST, Drew Yao
no flags Details | Formatted Diff | Diff
Patch (3.34 KB, patch)
2013-11-21 19:10 PST, Drew Yao
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Drew Yao 2013-11-21 18:50:04 PST
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.
Comment 1 Drew Yao 2013-11-21 18:54:01 PST
Created attachment 217641 [details]
Patch
Comment 2 WebKit Commit Bot 2013-11-21 18:55:58 PST
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.
Comment 3 Drew Yao 2013-11-21 19:10:05 PST
Created attachment 217643 [details]
Patch
Comment 4 Drew Yao 2013-11-21 19:11:19 PST
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.
Comment 5 WebKit Commit Bot 2013-11-21 19:11:20 PST
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 6 Brent Fulgham 2013-11-26 19:05:20 PST
Comment on attachment 217643 [details]
Patch

r=me
Comment 7 WebKit Commit Bot 2013-11-26 19:32:37 PST
Comment on attachment 217643 [details]
Patch

Clearing flags on attachment: 217643

Committed r159799: <http://trac.webkit.org/changeset/159799>
Comment 8 WebKit Commit Bot 2013-11-26 19:32:39 PST
All reviewed patches have been landed.  Closing bug.