Closed Bug 655138 Opened 9 years ago Closed 9 years ago
Invalid write [@ ns
User Font Set::Replace Font Entry]
Loading layout/reftests/font-face/local-1.html under Valgrind gives me an invalid write (write after free) in nsUserFontSet::ReplaceFontEntry.
As usual, valgrind is right. Replacing the entry in mAvailableFonts can cause deletion of the old entry, so we mustn't try to use it after that.
Assignee: nobody → jfkthame
Attachment #530603 - Flags: review?(jdaggett)
Comment on attachment 530603 [details] [diff] [review] patch, don't use aOldFontEntry after it may have been released Argh, sorry I missed this when reviewing previous patches.
Attachment #530603 - Flags: review?(jdaggett) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.