[ scummvm-Patches-790433 ] Fix Read Overrun

SourceForge.net noreply at sourceforge.net
Mon Aug 18 11:26:50 CEST 2003


Patches item #790433, was opened at 2003-08-18 11:26
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=790433&group_id=37116

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Armin Müller (arm_in)
Assigned to: Nobody/Anonymous (nobody)
Summary: Fix Read Overrun

Initial Comment:
In file "summ\verbs.cpp", function "void
Scumm::redrawV2Inventory()", line 279

memcpy() causes a read overrun when the source string
is shorter than 18 chars. This _might_ lead to a page
fault on some systems. Better replace with something
like that:

strncpy((char *) msg, (char *) _messagePtr, 18),


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=790433&group_id=37116




More information about the Scummvm-tracker mailing list