Bug 11553 - [patch] Missing NULL checks
Summary: [patch] Missing NULL checks
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: App/compiz (show other bugs)
Version: git
Hardware: All All
: medium major
Assignee: David Reveman
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords: janitor, patch
Depends on:
Blocks:
 
Reported: 2007-07-12 03:18 UTC by Roland "Test-tools" Bär
Modified: 2007-08-02 08:46 UTC (History)
0 users

See Also:
i915 platform:
i915 features:

Attachments
Adds the NULL checks (1.77 KB, patch)
2007-07-12 03:18 UTC, Roland "Test-tools" Bär 		no flags Details | Splinter Review
Fixes last patch (a mem leak was introduced there) (478 bytes, patch)
2007-07-12 03:20 UTC, Roland "Test-tools" Bär 		no flags Details | Splinter Review
Renewed patch, applicable... (1.56 KB, patch)
2007-07-31 04:35 UTC, Roland "Test-tools" Bär 		no flags Details | Splinter Review
Show Obsolete (2) View All

Description Roland "Test-tools" Bär 2007-07-12 03:18:07 UTC 
Adds missing NULL checks where in similiar situation at other places we already have NULL checks.
Comment 1 Roland "Test-tools" Bär 2007-07-12 03:18:58 UTC 
Created attachment 10683 [details] [review]
Adds the NULL checks
Comment 2 Roland "Test-tools" Bär 2007-07-12 03:20:03 UTC 
Created attachment 10684 [details] [review]
Fixes last patch (a mem leak was introduced there)
Comment 3 Roland "Test-tools" Bär 2007-07-19 05:01:48 UTC 
Third part, file src/screen.c got just applied from here:
http://bugs.opencompositing.org/show_bug.cgi?id=236
First and second part still open.
Comment 4 Roland "Test-tools" Bär 2007-07-31 04:35:08 UTC 
Created attachment 10926 [details] [review]
Renewed patch, applicable...
Comment 5 David Reveman 2007-08-02 08:46:46 UTC 
The change to src/screen.c has already been applied and the patch is adding a duplicate check. I've applied the other changes.

Thanks.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.