6 commits

Author	SHA1	Message	Date
marauder2k7	eb33fe04af	working vhacd ... renamed ThreadPool to TorqueThreadPool to avoid conflics fixed data transmission between stages of convexDecome and trimesh creation TODO: re-add our own functions for generating sphere/cylinder/box	2024-05-12 14:43:56 +01:00
marauder2k7	2b295fb7f0	rest of virtuals removed ... virtuals removed and replaced with override where necessary on the rest of the code base, clang-tidy to the rescue.	2024-03-18 18:40:22 +00:00
Glenn Smith	47d5b6ead7	As suggested, extract strlen calls from sizes into variables so it isn't called twice	2018-03-08 20:59:40 -05:00
Glenn Smith	79c34c68db	Use strncpy instead of strcpy because again, buffer overflows	2018-03-06 02:35:33 -05:00
Ben Payne	a88339c219	Fix buffer overflows due to incorrect use of sizeof ... A snippet of example code: UTF16 pszFilter[1024]; ... convertUTF8toUTF16((UTF8 *)mData.mFilters, pszFilter, sizeof(pszFilter)); Since the conversion function is expecting the third parameter to be the length in 16-bit characters, *not* bytes, this results in the function writing outside the bounds of the output array. To make this less likely to happen in the future (I hope), I've provided a template function that infers the correct size of a static array, so it's no longer necessary to pass the size in most cases. The sized function has been renamed with an "N" suffix to hopefully encourage this use. This bug was caught due to a warning from MSVC about stack corruption occurring in codeBlock::exec(), after opening a file open dialog twice in succession. After some hunting, I found that this was due to FileDialog::Execute() passing incorrect buffer sizes to the conversion function, which resulted in the function writing a null terminator into some memory that happened to be in the stack frame of codeBlock::exec()!	2015-01-26 16:52:01 -05:00
DavidWyand-GG	7dbfe6994d	Engine directory for ticket #1	2012-09-19 11:15:01 -04:00