view urogue/TODO @ 276:4573b355cdc1

UltraRogue: prevent bad array accesses in call(). Marking non-magic items caused segfaults because item_color was set to NULL. item_type could also be used as an out-of-bounds index. These problems have been fixed by only using these variables when the mark argument is false, in which case they are properly initialized. A fall-through case statement was also fixed.
author John "Elwin" Edwards
date Sun, 10 Sep 2017 21:04:22 -0400
parents c495a4f288c6
children
line wrap: on
line source

UltraRogue: The Ultimate Adventure in the Dungeons of Doom
Copyright (C) 1993, 1995 Herb Chong
All rights reserved.

%W% %G%

add_pack() calls ur_free() on the linked list pointer passed
to it. Need to go through code and look for cases where we
use that pointer after calling add_pack(). Electric
fence in guard-free() mode should help.

Similary throw_away(), discard(), discard_pack() free items...
there will be more like this...

Bug in genocide scroll: don't request list, type letter, etc...

Implement potion of true sight.

Bug: apparently when creating a new familiar after a player level
change, something isn't begin reset because ur SEGVs when running
through the monster list in do_chase() right after the new
familiar is created. It appears that the familiar isn't being removed
from the monster list.