]> granicus.if.org Git - nethack/commit
minor memory leak
authorPatR <rankin@nethack.org>
Sun, 30 Dec 2018 04:41:16 +0000 (20:41 -0800)
committerPatR <rankin@nethack.org>
Sun, 30 Dec 2018 04:41:16 +0000 (20:41 -0800)
commitadf64764f4859987b866a3bf822ce8b49653f39f
treeb4b3e42b8f74c456c455e1250cb9a4dfa7291b34
parent39b6f7f4628a073f0ea0cc34c12581fab7051a0a
minor memory leak

I ran the fuzzer with MONITOR_HEAP enabled and heaputil found a dozen
or so un-free'd allocations, all made by the same dupstr() call in
special_handling() for "symset" and "roguesymset".  (Reproducible with
a few tens of thousands of fuzzer moves, although you have to take
over from the fuzzer and make a clean exit rather than just interrupt
it or there'll be lots of other un-free'd memory.)  I haven't actually
figured out how/why it was leaking, but reorganizing the code has made
the leak go away (according to a couple of even longer fuzzer runs) so
I'm settling for that.
src/files.c
src/options.c