bugWarzone 2100 Project - Bugs: bug #10840, division by zero in font code

Show feedback again

You are not allowed to post comments on this tracker with your current authentification level.

bug #10840: division by zero in font code

Submitted by:  Dennis Schridde <devurandom>
Submitted on:  Mon Jan 21 01:20:09 2008  
Category: NoneSeverity: Normal
Priority: 5 - NormalStatus: None
Assigned to: NoneOpen/Closed: Open
Release: svn/trunkOperating System: Microsoft Windows
Planned Release: None

Fri Jan 25 19:29:05 2008, comment #1:

Patch to prevent divisions by zero. The return type of these functions are not checked in any case, so don't return a non-sensical value.

(file #3672)

Per I. Mathisen <per>
Project Administrator
Mon Jan 21 01:20:09 2008, original submission:

In the fontcode a division by zero can happen when fontconfig's config is missing.

Backtrace from minidump:
WineDbg starting on minidump on pid 0b44
warzone2100.exe was running on #1 Intel Pentium 4 or AMD Athlon64 (0.27393) CPU on Windows XP (2600)
Register dump:
CS:001b SS:0023 DS:0023 ES:0023 FS:0038 GS:0000
EIP:0041de97 ESP:0022f820 EBP:0022f838 EFLAGS:00210242( - 00 - RIZ1)
EAX:000001e0 EBX:000001e0 ECX:0114a0a0 EDX:00000000
ESI:00000001 EDI:00000020
Stack dump:
0x0022f820: 00000000 00000000 00000000 006f6ad7
0x0022f830: 00000000 000001e0 0022faf8 00491989
0x0022f840: 00004e3e 00000025 0000015e 006f6ad7
0x0022f850: 03c200d0 0000280a 00002825 00000005
0x0022f860: 0000002b 00000023 0000001c 006f6953
0x0022f870: 000000ae 000000af 00000000 00000000
=>1 0x0041de97 initConsoleMessages+0x27() [src/console.c:131] in warzone2100 (0x0022f838)
2 0x00491989 addChatBox+0x117() [src/multiint.c:1364] in warzone2100 (0x0022faf8)
3 0x00493e2a startMultiOptions+0x128(bReenter=0x0) [src/multiint.c:2357] in warzone2100 (0x0022fb48)
4 0x004503f9 changeTitleMode+0xde(mode=0x8) [src/frontend.c:189] in warzone2100 (0x0022fb68)
5 0x00450e85 runMultiPlayerMenu+0x6d() [src/frontend.c:492] in warzone2100 (0x0022fb88)
6 0x004e8b62 titleLoop+0xca() [src/wrappers.c:157] in warzone2100 (0x0022fbb8)
7 0x0047d7ad runTitleLoop+0xb() [src/main.c:666] in warzone2100 (0x0022fbf8)
8 0x0047da74 mainLoop+0xaf() [src/main.c:804] in warzone2100 (0x0022fc38)
9 0x0047df55 SDL_main+0x4d5(argc=0x1, argv=0x3e5880) [src/main.c:968] in warzone2100 (0x0022fd78)
10 0x0059cfca in warzone2100 (+0x19cfca) (0x0022fda8)
11 0x0059d1b2 in warzone2100 (+0x19d1b2) (0x0022fee8)
12 0x0059cc59 in warzone2100 (+0x19cc59) (0x0022ff68)
13 0x004010b6 in warzone2100 (+0x10b6) (0x0022ffb0)
14 0x00401138 in warzone2100 (+0x1138) (0x0022ffc0)
15 0x7c816fd7 in kernel32 (+0x16fd7) (0x0022fff0)
16 0x00000000 (0x00000000)
131 maxDrop = ((pie_GetVideoBufferHeight()/iV_GetTextLineSize())/2);
WineDbg starting on pid 0b44

Dennis Schridde <devurandom>
Project Administrator


Attached Files
file #3672:  sanityText.diff added by per (2kB - text/x-patch)


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by per (Updated the item)
  • -unavailable- added by devurandom (Submitted the item)

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.


    Error: not logged in



    Follows 1 latest change.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri Jan 25 19:29:04 2008perAttached File-=>Added sanityText.diff, #3672
    Show feedback again

    Back to the top

    Powered by Savane 3.1-cleanup