Because the test to continue is *after* the loop, we can't continue
when c is the last index into lzh->son, or the code will make use of lzh->son[sizeof(lzh->son)/sizeof(lzh->son[0])] which is outside the bounds
of the array.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
From deuce@VERT to CVS commit on Wed Apr 15 10:54:55 2020
src/encode lzh.c 1.14 1.15
Update of /cvsroot/sbbs/src/encode
In directory cvs:/tmp/cvs-serv688
Modified Files:
lzh.c
Log Message:
More weirdness uncovered by Coverity.
To be fair, this is all weird.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
From deuce@VERT to CVS commit on Fri Apr 17 07:08:11 2020
src/encode lzh.c 1.15 1.16
Update of /cvsroot/sbbs/src/encode
In directory cvs:/tmp/cvs-serv1738
Modified Files:
lzh.c
Log Message:
I'm going to get the LZH to pass Coverity if it's the last thing I do!
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
Who's Online
System Info
Sysop:
deepend
Location:
Calgary, Alberta
Users:
240
Nodes:
10 (0
/
10)
Uptime:
04:03:37
Calls:
1,448
Files:
3,343
Messages:
358,553
Synchronet Oneliners
Vintagegeek@rdbbs Thu Apr 18 05:31:35 2024 Apple IIGS w WiModem or iPad Telnet in Chrome
Vintagegeek@rdbbs Fri Apr 19 05:10:04 2024 More INPUT STEPHANIE
Vintagegeek@rdbbs Sun Apr 21 16:10:32 2024 We need more Commercials
Vintagegeek@rdbbs Tue Apr 23 06:41:24 2024 Boss, the plane the plane
Ratthing@rdbbs Tue Apr 23 21:29:06 2024 hello from Texas!
Vintagegeek@rdbbs Wed Apr 24 06:18:54 2024 Greetings Lone Star State