[Bug 3330] Memory write passed the end of allocated array

bugzilla-daemon at icculus.org bugzilla-daemon at icculus.org
Wed Aug 29 03:08:16 EDT 2007


http://bugzilla.icculus.org/show_bug.cgi?id=3330


info at dbwatersports.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
Attachment #1492 is|0                           |1
           obsolete|                            |




------- Comment #6 from info at dbwatersports.com  2007-08-29 03:08 EDT -------
Created an attachment (id=1493)
 --> (http://bugzilla.icculus.org/attachment.cgi?id=1493&action=view)
Diff file with a proposed fix

After making the changes, there was still a problem with getting more than
about 80 maps to show up. In q3_ui/ui_gameinfo.c, function UI_LoadArenas, there
is a local variable allocated to hold all of the files that need to be parsed.
This is defined as char dirlist[1024]. I changed this to 2048 to give a little
more breathing room. On my test environment, which had 147 maps, I was able to
get them all to show up with this change. This is a local stack variable that
is only used for this function, so it shouldn't cause any issues that I am
aware of. I have attached a new diff file with the proposed changes to both
files.


-- 
Configure bugmail: http://bugzilla.icculus.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.



More information about the quake3-bugzilla mailing list