Closed Bug 839185 Opened 12 years ago Closed 12 years ago

Fix rooting hazards in jswrapper.cpp identified by static analysis

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 832299

People

(Reporter: n.nethercote, Assigned: n.nethercote)

Details

(Whiteboard: [js:t])

jswrapper.cpp has 30-odd hazards. I think a lot of them are because Wrapper is a friend API class and its methods take raw pointers. This is similar to jsproxy.cpp, so I'll adopt the same strategy of doing rooting within the relevant methods, rather than changing the methods' signatures.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.