Last Comment Bug 544875 - Investigate mNavigator usage in nsPluginArray and nsMimeTypeArray
: Investigate mNavigator usage in nsPluginArray and nsMimeTypeArray
Product: Core
Classification: Components
Component: DOM (show other bugs)
: unspecified
: x86 All
: -- major (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
: Andrew Overholt [:overholt]
Depends on: CVE-2010-2767
  Show dependency treegraph
Reported: 2010-02-08 07:13 PST by Olli Pettay [:smaug]
Modified: 2012-03-08 08:47 PST (History)
10 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Description Olli Pettay [:smaug] 2010-02-08 07:13:03 PST
Using raw pointers is scary!
Comment 1 Daniel Veditz [:dveditz] 2010-08-04 15:43:09 PDT
Apparently it is, Sergey Glazunov found at least one sg:critical bug in that code. Is that the only problem there or are there more?
Comment 2 Daniel Veditz [:dveditz] 2010-08-06 10:19:14 PDT
We need this "fixed" this round to make sure bug 584517 is the only problem in there. Otherwise we're just pissing away bounties as people file them one-by-one.
Comment 3 christian 2010-08-10 11:11:24 PDT
Is there any plans to do this audit before code-freeze this Thursday? I'm going to remove blocking as we wouldn't block shipping on this audit...
Comment 4 Olli Pettay [:smaug] 2010-08-10 12:55:12 PDT
I could try to audit this tomorrow, if I find a nice solution for Bug 585815
Comment 5 Blake Kaplan (:mrbkap) 2010-08-10 13:06:39 PDT
For what it's worth, I looked into this for the patch in bug 584512 . That's why that patch touches the mime type array as well as the plugin array. I'd appreciate smaug double-checking my work though!
Comment 6 Olli Pettay [:smaug] 2010-08-11 05:42:52 PDT
Without looking at Bug 584512 I found the same bug, but not anything else.
Comment 7 Josh Aas 2012-03-08 08:44:47 PST
Seems like Blake and Olli are both saying they looked into this. Can we call this fixed, that being the case? Also, can we open this up as there is no specific vulnerability discussed?
Comment 8 Blake Kaplan (:mrbkap) 2012-03-08 08:47:40 PST

Note You need to log in before you can comment on or make changes to this bug.