diff options
| author | est31 <MTest31@outlook.com> | 2016-03-28 14:39:43 +0200 |
|---|---|---|
| committer | est31 <MTest31@outlook.com> | 2016-03-30 16:26:05 +0200 |
| commit | 0aac1b74037ba0f6b0526e81ad0ea7367e31c224 (patch) | |
| tree | 68fdb96cb3121615a98d250092ad5a6d3c5e6e76 /src/unittest/test_schematic.cpp | |
| parent | 0115da1d6348a08388238d8728daaa048a603522 (diff) | |
| download | minetest-0aac1b74037ba0f6b0526e81ad0ea7367e31c224.tar.gz minetest-0aac1b74037ba0f6b0526e81ad0ea7367e31c224.tar.bz2 minetest-0aac1b74037ba0f6b0526e81ad0ea7367e31c224.zip | |
mg_schematic: fix leak in lua API, and small cleanup
* Fix leak like behaviour if you load multiple schematics in a loop.
* Cleanup check in for, fixing theoretical out of bounds read if
Schematic::deserializeFromMts reduced the number of elements
in m_nodenames. A != check may need an overflow of the counter
before it hits, if origsize is larger than m_nodenames.size().
* Fix function name passed to errorstream: it was wrong. Also use
__FUNCTION__ instead of manually using the method name at other
places in the function.
* Don't shadow the name member in the loop.
Diffstat (limited to 'src/unittest/test_schematic.cpp')
0 files changed, 0 insertions, 0 deletions