kcpuproc.cpp - prevent overflow #2
닫힘
opened 6 년 전 by Ghost
·
2개의 코멘트
불러오는 중...
Reference in new issue
아직 콘텐츠가 없습니다.
'%!s(<nil>)' 브랜치 삭제
Deleting a branch is permanent. It CANNOT be undone. Continue?
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
.