Closed
Bug 600064
Opened 13 years ago
Closed 13 years ago
TraceMallocOpenLogFile lets sites create files with chosen names, at chosen locations
Categories
(Core :: XPCOM, defect)
Tracking
()
RESOLVED
FIXED
mozilla2.0b7
People
(Reporter: jruderman, Assigned: dbaron)
References
(Blocks 1 open bug)
Details
(Keywords: testcase, Whiteboard: [sg:moderate] --enable-trace-malloc builds only)
Attachments
(2 files)
206 bytes,
text/html
|
Details | |
3.77 KB,
patch
|
jst
:
review+
roc
:
approval2.0+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•13 years ago
|
||
This requires more than a debug build; it requires a build with --enable-trace-malloc, which is not on by default even for debug builds.
Assignee | ||
Comment 2•13 years ago
|
||
TraceMallocDumpAllocations has the same issue.
Reporter | ||
Comment 3•13 years ago
|
||
That explains why I started seeing this when I switched from local debug builds to Tinderbox debug builds.
Assignee | ||
Comment 4•13 years ago
|
||
Probably the best thing to do here is make these functions check for UniversalXPConnect.
Assignee | ||
Comment 5•13 years ago
|
||
I think we're better off protecting people who build with trace-malloc. If somebody needs these somewhere UniversalXPConnect is hard to get, they can always comment them out.
Assignee | ||
Updated•13 years ago
|
Attachment #481344 -
Flags: review? → review?(jst)
Updated•13 years ago
|
Attachment #481344 -
Flags: review?(jst) → review+
Assignee | ||
Updated•13 years ago
|
Attachment #481344 -
Flags: approval2.0?
Attachment #481344 -
Flags: approval2.0? → approval2.0+
Assignee | ||
Comment 6•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/5ba88998be23
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
Reporter | ||
Updated•13 years ago
|
Group: core-security
Updated•13 years ago
|
Whiteboard: [sg:moderate] → [sg:moderate] --enable-trace-malloc builds only
Updated•13 years ago
|
Target Milestone: mozilla2.0b8 → mozilla2.0b7
You need to log in
before you can comment on or make changes to this bug.
Description
•