[PATCH v2] render: Fix crash in RenderAddGlyphs (#23645)
Keith Packard
keithp at keithp.com
Tue Sep 29 22:29:31 PDT 2009
Excerpts from Peter Hutterer's message of Tue Sep 29 22:15:21 -0700 2009:
> miGlyphs seems to deal with 0-width glyphs just fine
I guess I don't understand how this can be --
ProcRenderCompositeGlyphs passes all of the glyphs to CompositeGlyphs
which just calls down to the Glyphs function. If that is miGlyphs,
you've got a loop that looks like:
while (n--)
{
glyph = *glyphs++;
pPicture = GlyphPicture (glyph)[pScreen->myNum];
...
(uses of pPicture)
}
I don't see how pPicture won't end up as NULL for zero-sized glyphs,
which should yield a nice segfault.
Am I missing something obvious here?
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: not available
Url : http://lists.x.org/archives/xorg-devel/attachments/20090929/72395fb1/attachment.pgp
More information about the xorg-devel
mailing list