nethack-gwaa-kiwi/nethack
1
0
Fork 0
Code Pull Requests Activity

Fix vision: remove NO_MACRO_CPATH

Browse Source 
The non-macro q_path() function wrappers in vision.c pass the first
two arguments in the wrong order to the _q*_path() functions, swapping
rows and columns. This causes the Bresenham line-of-sight code to use
column values as row indices into viz_clear_rows[ROWNO], producing
out-of-bounds access and infinite loops.

NO_MACRO_CPATH selects these broken function wrappers. It was only
defined for Amiga (in config1.h), so the bug never triggered on other
platforms. Remove the define to use the correct macro versions.
This commit is contained in:
Ingo Paschke
2026-03-23 20:46:08 +01:00
parent 8d33cafb3e
commit e07eb1a550
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
include/config1.h
View File

@@ -95,7 +95,7 @@
#undef UNIX
#define DLB
#define HACKDIR "NetHack:"
#define NO_MACRO_CPATH
#endif
/*