Dungeon Crawl Stone Soup Tracker - DCSS
Viewing Issue Advanced Details
7186 Bug Report crash have not tried 2013-06-09 05:06 2013-06-11 07:32
Grandiloquent Gentleman Local  
neil MacOSX  
normal Tiles  
resolved 0.13 ancient branch  
0.13-a0-1489-g11bc701 done  
none    
none 0.13 ancient branch  
0007186: Ozos 'fridging a spore whose explosion kills causes crash.
I was using Ozo's fridge against a group of centaurs and a spore.
I cast ozo's fridge. Some centaurs die, the spore explodes killing a centaur and then the game crashes.
txt file icon crash-Prince of Darkness-20130608-215823.txt [^] (47,320 bytes) 2013-06-09 05:06 [Show Content]
? file icon 0001-Fix-an-Ozocubu-s-refrigeration-crash.patch [^] (1,301 bytes) 2013-06-09 20:34 [Show Content]
Issue History
2013-06-09 05:06 Grandiloquent Gentleman New Issue
2013-06-09 05:06 Grandiloquent Gentleman File Added: crash-Prince of Darkness-20130608-215823.txt
2013-06-09 20:34 blackcustard File Added: 0001-Fix-an-Ozocubu-s-refrigeration-crash.patch
2013-06-09 20:37 blackcustard Note Added: 0023131
2013-06-09 20:37 blackcustard Note Edited: 0023131
2013-06-09 20:38 blackcustard Note Edited: 0023131
2013-06-10 00:59 neil Note Added: 0023133
2013-06-10 00:59 neil Status new => resolved
2013-06-10 00:59 neil Fixed in Branch => 0.13 development branch
2013-06-10 00:59 neil Resolution open => done
2013-06-10 00:59 neil Assigned To => neil
2013-06-11 07:32 Grandiloquent Gentleman Note Added: 0023152

Notes
(0023131)
blackcustard   
2013-06-09 20:37   
(edited on: 2013-06-09 20:38)
This bug was exactly what it looked like. Crawl was building a list of monsters to hit and forgetting to make sure they were still alive when it got to them.

Drain life should exhibit the same failure, but it's hard to set up a test case for that. Olgreb's doesn't do damage. Nevertheless, this patch applies to both of those spells as well.

Other spells should be fine, these are the only three that use cast_los_attack_spell.

A small patch is attached.

(0023133)
neil   
2013-06-10 00:59   
blackcustard's patch is applied in trunk (0.13-a0-1548-gbec542c), thanks!
(0023152)
Grandiloquent Gentleman   
2013-06-11 07:32   
Thanks, although the ignite poison might also do this if you ignite a poisonous corpse underneath a spore which kills a potential target, maybe.