Bug 34746 - [v8] Check if returned wrapper is empty before operating on it.
Summary: [v8] Check if returned wrapper is empty before operating on it.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-09 03:36 PST by anton muhin
Modified: 2010-02-09 19:09 PST (History)
1 user (show)

See Also:

Attachments
Patch (1.21 KB, patch)
2010-02-09 03:40 PST, anton muhin 		no flags Details | Formatted Diff | Diff
Some more suspicious places (2.78 KB, patch)
2010-02-09 03:55 PST, anton muhin 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description anton muhin 2010-02-09 03:36:35 PST 
[v8] Check if returned wrapper is empty before operating on it.
Comment 1 anton muhin 2010-02-09 03:40:26 PST 
Created attachment 48402 [details]
Patch
Comment 2 anton muhin 2010-02-09 03:55:05 PST 
Created attachment 48403 [details]
Some more suspicious places
Comment 3 Nate Chapin 2010-02-09 08:19:32 PST 
Comment on attachment 48403 [details]
Some more suspicious places

Looks good, thanks for taking care of this!
Comment 4 anton muhin 2010-02-09 08:20:27 PST 
Comment on attachment 48403 [details]
Some more suspicious places

Thanks a lot for review, Nate.
Comment 5 WebKit Commit Bot 2010-02-09 19:09:31 PST 
Comment on attachment 48403 [details]
Some more suspicious places

Clearing flags on attachment: 48403

Committed r54581: <http://trac.webkit.org/changeset/54581>
Comment 6 WebKit Commit Bot 2010-02-09 19:09:37 PST 
All reviewed patches have been landed.  Closing bug.