#8554 closed patch
Bugfix [1498158] SIMON: Itemheap overflow on 64bit systems
| Reported by: | (none) | Owned by: | fingolfin |
|---|---|---|---|
| Priority: | normal | Component: | Engine: AGOS |
| Version: | Keywords: | ||
| Cc: | Game: | Simon the Sorcerer 1 |
Description
The following patch fixes the above mentioned crash by enlarging the item heap and fixing alignment for native pointers. I simply doubled the heap size, but if memory consumption is an issue on other architectures, I can try to find a more precise lower bound.
Tested on AMD64 with Simon the Sorcerer 1 and 2.
Ticket imported from: #1522592. Ticket imported from: patches/659.
Attachments (1)
Change History (7)
by , 18 years ago
| Attachment: | simon-crash-amd64-fix.diff added |
|---|
comment:1 by , 18 years ago
| Owner: | set to |
|---|
comment:2 by , 18 years ago
Hopefully someone can test the patch with the Feeble Files too, to check whether item heap is large enough now.
It would be helpful to know if the games are completable now, if you are playing through them.
comment:3 by , 18 years ago
Well, does this patch cause any regressions? If not, then why not apply it and thus let more people test it? :-)
comment:5 by , 18 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → closed |
comment:6 by , 6 years ago
| Component: | → Engine: AGOS |
|---|---|
| Game: | → Simon the Sorcerer 1 |
Note:
See TracTickets
for help on using tickets.
Patch file against Revision 23494