chiark / gitweb /
Fix array overruns in the new Filling solver pass.
authorSimon Tatham <anakin@pobox.com>
Wed, 21 Oct 2015 17:32:15 +0000 (18:32 +0100)
committerSimon Tatham <anakin@pobox.com>
Wed, 21 Oct 2015 17:32:15 +0000 (18:32 +0100)
commit1cf403ceb81482dea7107a49573d6834c5a650d1
tree04c954af41db6ac10931625bf098cf84bb22d75f
parent90af15b43ed57a6835091bb1c98227052590b3ea
Fix array overruns in the new Filling solver pass.

Probably because I wrote a couple of loops up to the maximum cell
value using the non-idiomatic <= for their termination test, I also
managed to use <= inappropriately for iterating over every cell of the
grid, leading to a couple of references just off the end of arrays.

Amusingly, it was the Emscripten front end which pointed this out to
me by actually crashing as a result! Though valgrind found it just
fine too, once I thought to run that. But it comes to something when
running your C program in Javascript detects your memory errors :-)
filling.c