I’ve run cppcheck on kcpuload, here is the error:
[kcpuproc.cpp:176]: (error) Width 32 given in format string (no. 1) is larger than destination buffer ‘tagbuffer‘, use %31s to prevent overflowing it.
Instead, I’d like to enlarge the ‘tagbuffer’ array by 1, but I have no ideas why the guy has set the string to 32 characters.
What do think about?
yes, increasing the tagbugger to 33 bytes is the correct solution.
Changing the sequence of reading in the fscanf may lead to unexpected results, with bytes after 32 being shifted by one. Extending tagbuffer to allow for the extra null terminator will preserve functionality and avoid mistakes.
Fixed by PR #4, commit e816bb8048.
No due date set.
This issue currently doesn't have any dependencies.
Deleting a branch is permanent. It CANNOT be undone. Continue?