Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(772)

Unified Diff: third_party/WebKit/Source/wtf/BitArray.h

Issue 1672603002: Change assert to release assert for BitArray to prevent out-of-bounds access. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/wtf/BitArray.h
diff --git a/third_party/WebKit/Source/wtf/BitArray.h b/third_party/WebKit/Source/wtf/BitArray.h
index 76258b337159938b1b34c1d57248df89f597dab2..9a9304f1c589046f26c2895f7e673c5bfc909c65 100644
--- a/third_party/WebKit/Source/wtf/BitArray.h
+++ b/third_party/WebKit/Source/wtf/BitArray.h
@@ -43,19 +43,19 @@ public:
void set(unsigned index)
{
- ASSERT_WITH_SECURITY_IMPLICATION(index < arraySize);
+ RELEASE_ASSERT(index < arraySize);
m_data[index / 8] |= 1 << (index & 7);
}
void clear(unsigned index)
{
- ASSERT_WITH_SECURITY_IMPLICATION(index < arraySize);
+ RELEASE_ASSERT(index < arraySize);
m_data[index / 8] &= ~(1 << (index & 7));
}
bool get(unsigned index) const
{
- ASSERT_WITH_SECURITY_IMPLICATION(index < arraySize);
+ RELEASE_ASSERT(index < arraySize);
return !!(m_data[index / 8] & (1 << (index & 7)));
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698