Title: [94267] branches/chromium/835/Source/WebCore/webaudio/AudioNode.cpp
Revision
94267
Author
crog...@google.com
Date
2011-08-31 19:28:43 -0700 (Wed, 31 Aug 2011)

Log Message

Merge 94262 - Add defensive bounds checking for AudioNode methods
https://bugs.webkit.org/show_bug.cgi?id=67346

BUG=95012
Review URL: http://codereview.chromium.org/7821023

Modified Paths

Diff

Modified: branches/chromium/835/Source/WebCore/webaudio/AudioNode.cpp (94266 => 94267)


--- branches/chromium/835/Source/WebCore/webaudio/AudioNode.cpp	2011-09-01 02:24:41 UTC (rev 94266)
+++ branches/chromium/835/Source/WebCore/webaudio/AudioNode.cpp	2011-09-01 02:28:43 UTC (rev 94267)
@@ -100,12 +100,16 @@
 
 AudioNodeInput* AudioNode::input(unsigned i)
 {
-    return m_inputs[i].get();
+    if (i < m_inputs.size())
+        return m_inputs[i].get();
+    return 0;
 }
 
 AudioNodeOutput* AudioNode::output(unsigned i)
 {
-    return m_outputs[i].get();
+    if (i < m_outputs.size())
+        return m_outputs[i].get();
+    return 0;
 }
 
 bool AudioNode::connect(AudioNode* destination, unsigned outputIndex, unsigned inputIndex)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to