Small suspected buglet in XML surface code

cairo-xml-surface.c




One checks here for NULL.. but

static cairo_status_t
_cairo_xml_surface_emit_clip (cairo_xml_surface_t *surface,
                              const cairo_clip_t *clip)
{
    if (clip == NULL  ||  clip->path == NULL) /* < -- edz: added  check for clip->path */
        return CAIRO_STATUS_SUCCESS;

    return _cairo_xml_surface_emit_clip_path (surface, clip->path);
}

Only checks if the clip is NULL but clip->path can be, I think, NULL too!







_________________________________________________________________
Edward C. Zimmermann
Softwareentwicklung
---
CIB software GmbH
Elektrastr. 6a
81925 München
---
T +49 (0)89 / 1 43 60 - 156
F +49 (0)89 / 1 43 60 -100
[email protected]<mailto:[email protected]>
www.cib.de<http://www.cib.de/>
---
Sitz: München
Registergericht München, HRB 123286
Geschäftsführer: Dipl.-Ing. Ulrich Brandner

-- 
cairo mailing list
[email protected]
http://lists.cairographics.org/mailman/listinfo/cairo