Bug 99136 - Check parameter's safety first
Summary: Check parameter's safety first
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Audio (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-11 21:52 PDT by Jaehun Lim
Modified: 2012-10-12 10:33 PDT (History)
5 users (show)

See Also:

Attachments
Patch (3.38 KB, patch)
2012-10-11 21:54 PDT, Jaehun Lim 		no flags Details | Formatted Diff | Diff
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 Jaehun Lim 2012-10-11 21:52:58 PDT 
Pointer parameter was used before NULL checking.
This patch moves safety checking statements to the head of the function.
Comment 1 Jaehun Lim 2012-10-11 21:54:32 PDT 
Created attachment 168356 [details]
Patch
Comment 2 Jer Noble 2012-10-12 09:39:28 PDT 
Good catch. LGTM.
Comment 3 WebKit Review Bot 2012-10-12 10:33:31 PDT 
Comment on attachment 168356 [details]
Patch

Clearing flags on attachment: 168356

Committed r131198: <http://trac.webkit.org/changeset/131198>
Comment 4 WebKit Review Bot 2012-10-12 10:33:34 PDT 
All reviewed patches have been landed.  Closing bug.