frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Thu Aug 11 11:43:32 2005 Søren Sandmann <sandmann@redhat.com> 

Make sure we don't crash on glyphs with NULL bits. Bug 659.
This commit is contained in:
Søren Sandmann Pedersen 2005-08-11 15:46:42 +00:00
parent 3e471ddf1d
commit 13e16ee93a
1 changed files with 20 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
hw/xfree86/xaa/xaaTEText.c
View File

@ -253,6 +253,7 @@ XAAGlyphBltTEColorExpansion(
RightEdge = min(Right, pbox->x2);
if(RightEdge > LeftEdge) { /* we have something to draw */
unsigned int *fallbackBits = NULL;
ytop = max(Top, pbox->y1);
ybot = min(Bottom, pbox->y2);
@ -265,9 +266,22 @@ XAAGlyphBltTEColorExpansion(
int count;
glyphs = (unsigned int**)(infoRec->PreAllocMem);
for(count = 0; count < nglyph; count++)
for(count = 0; count < nglyph; count++) {
glyphs[count] = (unsigned int*)
FONTGLYPHBITS(gBase,*ppci++);
if (!glyphs[count]) {
/* Glyphs with NULL bits do exist in the wild.
Replace with blank bits in that case */
if (!fallbackBits) {
int fontHeight = Bottom - Top + 1;
fallbackBits = xcalloc (glyphWidth * fontHeight, 1);
if (!fallbackBits)
return;
}
glyphs[count] = fallbackBits;
}
}
/* our new unrolled TE code only writes DWORDS at a time
so it can read up to 6 characters past the last one
@ -280,12 +294,15 @@ XAAGlyphBltTEColorExpansion(
glyphs[count + 5] = glyphs[0];
}
/* x, y, w, h, skipleft, skiptop, glyphp, glyphWidth, fg, bg, rop, pm */
/* x, y, w, h, skipleft, skiptop, glyphp, glyphWidth, fg, bg, rop, pm */
(*infoRec->TEGlyphRenderer)( pScrn,
LeftEdge, ytop, RightEdge - LeftEdge, ybot - ytop,
skippix, ytop - Top, glyphs + skipglyphs, glyphWidth,
fg, bg, rop, planemask);
fg, bg, rop, planemask);
if (fallbackBits)
xfree (fallbackBits);
}
nbox--; pbox++;