Opened 10 years ago
Closed 7 years ago
#6748 closed defect (fixed)
TINSEL: Discworld II Save Naming Crash
| Reported by: | SF/softwarespecial | Owned by: | csnover |
|---|---|---|---|
| Priority: | blocker | Component: | Engine: Tinsel |
| Version: | Keywords: | has-save-game reproducible has-pull-request | |
| Cc: | Game: | Discworld II |
Description
Note When I load my save and then I try to save it crashed after imputing 17 characters. If I load the first save game and try to save it won't crash until about 20 characters. Not sure if this is a bug or something got corrupted on my end. I had saves before and I deleted them. It may be the crashing has something to do with that.
Ticket imported from: bugs/6748.
Attachments (4)
Change History (19)
by , 10 years ago
| Attachment: | Saving On Discworld 2 type in 17 characters it crashes.avi added |
|---|
by , 10 years ago
| Attachment: | dw2-gb.009 added |
|---|
comment:1 by , 10 years ago
comment:2 by , 10 years ago
| Summary: | Discworld 2 Save issues. → TINSEL: Discworld II Save Naming Crash |
|---|
comment:3 by , 10 years ago
Can't replicate this with the latest git master on Linux x86_64.
I suspect that the cause is a corrupted savegame as Tinsel has had issues in this area...
softwarespecial: I would suggest removing all the DW2 savegames and then try again from a clear start to see if this happens again?
comment:4 by , 10 years ago
| Resolution: | → worksforme |
|---|---|
| Status: | new → pending |
comment:6 by , 10 years ago
I removed the old saves and still get the bug. I may try to reinstall the ScummVM first do a uninstall then a reinstall. Remember the crash doesn't happen every save but on some it will crash only after imputing a certain amount of characters. Would it help if I sent you all the saves I did?
comment:7 by , 10 years ago
Can you describe the exact steps you take to reproduce this crash? (i.e., exactly how you launch scummvm, how you start the game, etc...)
by , 10 years ago
| Attachment: | Saving On Discworld 2 type in 17 characters it crashes.2.avi added |
|---|
comment:8 by , 10 years ago
I start ScummVM by double clicking the desktop Icon then I load the save game Then I do some stuff in the game then I try to save it. When I type just a few letters its fine its only if I type to many. Just try loading the ScummVM and then loading the save I attached then play some then try saving a new one and type tell you have added as many characters as it can possibly have. If it does not crash try also to save over the old save game and erase the old letters then type tell you can't any more.
comment:9 by , 9 years ago
| Resolution: | worksforme |
|---|---|
| Status: | pending → new |
comment:10 by , 7 years ago
| Priority: | normal → blocker |
|---|
Raising all identified crasher, hang, and memory violation bugs which I could not fully triage myself to blocker priority for the next release.
comment:11 by , 7 years ago
| Keywords: | has-save-game added |
|---|
comment:12 by , 7 years ago
| Owner: | set to |
|---|
comment:13 by , 7 years ago
| Keywords: | reproducible added |
|---|
To reproduce this, you need at least 8 copies of the attached save game. The game appears to run out of free objects (pFreeObject is null) and so crashes with an assertion failure in AllocObject.
comment:14 by , 7 years ago
Increasing NUM_OBJECTS fixes the crash. These save games each have 30 letters, times 8 saves, plus the objects existing already (14, apparently), ends up with two free objects available until the engine runs out. Since the save games can be up to 40 letters long, there is a need to either increase the maximum number of objects up to at least 374, or reduce the maximum length of save game names down to 26, to prevent this problem from happening again. Obviously increasing NUM_OBJECTS is trivial, I just don’t know enough about this engine at this point to know what ramifications (if any) this would have on e.g. save games.
comment:15 by , 7 years ago
| Keywords: | has-pull-request added |
|---|---|
| Resolution: | → fixed |
| Status: | new → pending |
comment:16 by , 7 years ago
| Status: | pending → closed |
|---|
Using a Daily Build I got this error when trying the same thing.