Bug 11553

Summary:

[patch] Missing NULL checks

Product:

xorg

Reporter:

Roland "Test-tools" Bär <roland>

Component:

App/compiz

Assignee:

David Reveman <reveman>

Status:

RESOLVED FIXED

QA Contact:

Xorg Project Team <xorg-team>

Severity:

major

Priority:

medium

Keywords:

janitor, patch

Version:

git

Hardware:

All

OS:

All

Whiteboard:

i915 platform:

i915 features:

Attachments:

Description	Flags
Adds the NULL checks	none
Fixes last patch (a mem leak was introduced there)	none
Renewed patch, applicable...	none

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.