Closed
Bug 519813
Opened 15 years ago
Closed 15 years ago
IPDL unions need to follow C++ construction/destruction rules
Categories
(Core :: IPC, defect)
Core
IPC
Tracking
()
RESOLVED
FIXED
People
(Reporter: bent.mozilla, Assigned: cjones)
References
Details
Attachments
(1 file)
|
1.32 KB,
patch
|
Details | Diff | Splinter Review |
xul.dll!ReleaseData(void * data=0xcdcdcdcd, unsigned int flags=0xcdcdcdcd) Line 116 + 0x13 bytes C++
xul.dll!nsACString_internal::Assign(const nsACString_internal & str={...}) Line 409 + 0x12 bytes C++
> xul.dll!nsCString::operator=(const nsCString & str={...}) Line 106 + 0x12 bytes C++
xul.dll!mozilla::plugins::Variant::operator=(const nsCString & aRhs={...}) Line 317 C++
Looks like the default constructor for nsCString was never called. And cjones says that the old type needs to be placement-destructor'd too.
|
||
Updated•15 years ago
|
|
Assignee | |
Comment 1•15 years ago
|
||
Not a serious patch, just a memory-leaking workaround until the real fix is ready.
|
|
Assignee | |
Comment 2•15 years ago
|
||
Incidentally fixed by http://hg.mozilla.org/projects/electrolysis/rev/f1e79cb59195
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•