curiousdannii-testing / inform7-imported-bugs

0 stars 0 forks source link

[I7-1893] [Mantis 1929] Memory access error (matching topic via table) crashes the IDE #191

Closed curiousdannii-testing closed 2 years ago

curiousdannii-testing commented 2 years ago

Reported by : Eleas

Description :

As the summary says, an action intending to match a text to a table entry instead crashes Inform itself.

Steps to reproduce :

Fooing is an action applying to one topic.
Understand "foo [text]" as Fooing.  
Check Fooing: if there is a token of the topic understood in the Table of Foo, do nothing.

Table of Foo
token (text)
"blah"

There is a room.

Additional information :

imported from: [Mantis 1929] Memory access error (matching topic via table) crashes the IDE
  • status: Closed
  • resolution: Resolved
  • resolved: 2022-04-13T12:39:28+10:00
  • imported: 2022/01/10
curiousdannii-testing commented 2 years ago

557058:4c095ffd-6d6f-47ce-9e73-77c613347b86:

Comment by pchimento :
I can't reproduce this crash by following the above instructions (including the testing command.) I don't even get the Glulxe fatal error...

Nonetheless, I'll make sure VERIFY_MEMORY_ACCESS is defined when compiling Glulxe.

curiousdannii-testing commented 2 years ago

557058:4c095ffd-6d6f-47ce-9e73-77c613347b86:

Comment by zarf :
On the Mac IDE (or other Glulxe interpreters), this displays an error:

Glulxe fatal error: Memory access out of range (-6EF6FAFC)

A memory error shouldn't crash the Gnome IDE. (Possibly the Glulxe interpreter on Linux was compiled without the VERIFY_MEMORY_ACCESS flag? That flag should really be turned on for all Glulxe builds these days.)

curiousdannii-testing commented 2 years ago

557058:4c095ffd-6d6f-47ce-9e73-77c613347b86:

Comment by Eleas :
Should probably be considered Severe rather than Critical, since an author can work around it. Also, I apologize for having missed the testing command which would be something along the lines of "foo x".

curiousdannii-testing commented 2 years ago

61eedb62875fc10070240916:

Comment by Graham Nelson: occasion now doesn't arise - see Mantis 1930.