<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - NULL pointer dereference in cairo-polygon-intersect.c:1170" href="https://bugs.freedesktop.org/show_bug.cgi?id=101531#c1">Comment # 1</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - NULL pointer dereference in cairo-polygon-intersect.c:1170" href="https://bugs.freedesktop.org/show_bug.cgi?id=101531">bug 101531</a> from <a class="email" href="mailto:bryce@osg.samsung.com" title="Bryce Harrington <bryce@osg.samsung.com>"> Bryce Harrington</a> <pre>Created <a href="attachment.cgi?id=134390" name="attach_134390" title="Adds the proposed null ptr check">attachment 134390</a> <a href="attachment.cgi?id=134390&action=edit" title="Adds the proposed null ptr check">[details]</a> <a href='page.cgi?id=splinter.html&bug=101531&attachment=134390'>[review]</a> Adds the proposed null ptr check Attached is a patch to implement the proposed null ptr check. However, what's needed is an analysis of the PDF to see how exactly it's constructing this invalid data, since I seriously doubt having right==NULL is a valid software condition. So, ideally, this should be an assert() not a null ptr check.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the QA Contact for the bug.</li> </ul> </body> </html>