kcpuproc.cpp - prevent overflow #2
Geschlossen
vor 6 Jahren geöffnet von Ghost
·
2 Kommentare
Laden…
In neuem Issue referenzieren
Hier gibt es bis jetzt noch keinen Inhalt.
Branch „%!s(<nil>)“ löschen
Das Löschen eines Branches ist permanent. Es KANN NICHT rückgängig gemacht werden. Fortfahren?
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[32]', 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?
Hi Greg,
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.
Good work!
Fixed by PR #4, commit
e816bb8048
.