Closed tezc closed 3 years ago
Still getting some warnings on sc_threads
/home/delboni/Workspace/c/sc/thread/sc_thread.c: In function ‘sc_thread_start’:
/home/delboni/Workspace/c/sc/thread/sc_thread.c:127:9: error: ‘strncpy’ specified bound 128 equals destination size [-Werror=stringop-truncation]
127 | strncpy(thread->err, strerror(rc), sizeof(thread->err));
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/delboni/Workspace/c/sc/thread/sc_thread.c:118:9: error: ‘strncpy’ specified bound 128 equals destination size [-Werror=stringop-truncation]
118 | strncpy(thread->err, strerror(rc), sizeof(thread->err));
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[ 79%] Built target sc_option
[ 79%] Built target sc_heap
/home/delboni/Workspace/c/sc/thread/sc_thread.c: In function ‘sc_thread_join’:
/home/delboni/Workspace/c/sc/thread/sc_thread.c:147:9: error: ‘strncpy’ specified bound 128 equals destination size [-Werror=stringop-truncation]
147 | strncpy(thread->err, strerror(rc), sizeof(thread->err));
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Just add - 1
at the end of those lines:
strncpy(thread->err, strerror(rc), sizeof(thread->err) - 1);
You missed line 147 sc/thread/sc_thread.c:147
:)
wow, my gcc & clang don't complain about that. Probably I and CI have older versions, I'll add newer versions to CI.
I usually have to clean the build folder or make a different one for release and debug to have these warnings showing on haha
Thanks, is working fine now!
closes #44
Thanks a lot for issue & review
Codecov Report
97.95% <ø> (ø)
100.00% <ø> (ø)
100.00% <ø> (ø)
95.23% <100.00%> (ø)
100.00% <100.00%> (ø)
Continue to review full report at Codecov.