Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-05-25 04:03:56 +00:00
Code Issues Projects Releases Packages Wiki Activity

[CHARMAP]

Browse source 
Don't overrun FillGrid() when accessing the last elements. Rewrite the code using 'for' loops, in the same style as what is done elsewhere in the code. CID 1363552
CORE-11597 #resolve

svn path=/trunk/; revision=71924
This commit is contained in:
Hermès Bélusca-Maïto 2016-07-13 15:50:44 +00:00
parent 380484c985
commit 3aa8e4c2ac
1 changed files with 3 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
reactos/base/applications/charmap/map.c
View file

@ -107,8 +107,9 @@ FillGrid(PMAP infoPtr,
i = XCELLS * infoPtr->iYStart;
added = 0;
x = y = 0;
while ((y <= YCELLS) && (x <= XCELLS))
for (y = 0; y < YCELLS; y++)
for (x = 0; x < XCELLS; x++)
{
ch = (WCHAR)infoPtr->ValidGlyphs[i];
@ -131,14 +132,6 @@ FillGrid(PMAP infoPtr,
i++;
ch = (WCHAR)i;
// move to the next cell
x++;
if (x > XCELLS - 1)
{
x = 0;
y++;
}
}
SelectObject(ps->hdc,
hOldFont);