Just a sample of the Echomail archive
[ << oldest | < older | list | newer > | newest >> ]
|  Message 48687  |
|  scan-admin@coverity.com to All  |
|  New Defects reported by Coverity Scan fo  |
|  12 Jan 26 13:47:57  |
 
TZUTC: 0000
MSGID: 54118.syncprog@1:103/705 2dcb04f7
PID: Synchronet 3.21b-Linux master/f717faea9 Jan 12 2026 GCC 12.2.0
TID: SBBSecho 3.34-Linux master/f717faea9 Jan 12 2026 GCC 12.2.0
BBSID: VERT
CHRS: ASCII 1
FORMAT: flowed
----==_mimepart_6964fb8d1cdba_2460bc2afbc97ad9ac59882
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Hi,
Please find the latest report on new defect(s) introduced to Synchronet found
with Coverity Scan.
2 new defect(s) introduced to Synchronet found with Coverity Scan.
6 defect(s), reported by Coverity Scan earlier, were marked fixed in the
recent build analyzed by Coverity Scan.
New defect(s) Reported-by: Coverity Scan
Showing 2 of 2 defect(s)
** CID 640971: High impact quality (Y2K38_SAFETY)
/atcodes.cpp: 620 in sbbs_t::atcode(const char *, char *, unsigned
long, int *, bool, unsigned int, JSObject *)()
________________________________________________________________
____________________________
*** CID 640971: High impact quality (Y2K38_SAFETY)
/atcodes.cpp: 620 in sbbs_t::atcode(const char *, char *, unsigned
long, int *, bool, unsigned int, JSObject *)()
614
615 if (code_match(sp, "UPTIME", ¶m)) {
616 extern volatile time_t uptime;
617 time_t up = 0;
618 if (uptime != 0 && time(&now) >= uptime)
619 up = now - uptime;
>>> CID 640971: High impact quality (Y2K38_SAFETY)
>>> A "time_t" value is stored in an integer with too few bits to
accommodate it. The expression "up" is cast to "uint".
620 return duration((uint)up, str, maxlen, param, DURATION
MINIMAL_VERBAL);
621 }
622
623 if (!strcmp(sp, "SERVED")) {
624 extern volatile uint served;
625 safe_snprintf(str, maxlen, "%u", served);
** CID 640970: Insecure data handling (INTEGER_OVERFLOW)
/atcodes.cpp: 1695 in sbbs_t::atcode(const char *, char *, unsigned
long, int *, bool, unsigned int, JSObject *)()
________________________________________________________________
____________________________
*** CID 640970: Insecure data handling (INTEGER_OVERFLOW)
/atcodes.cpp: 1695 in sbbs_t::atcode(const char *, char *,
unsigned long, int *, bool, unsigned int, JSObject *)()
1689 if (!strcmp(sp, "BYTESLEFT")) {
1690 safe_snprintf(str, maxlen, "%" PRIu64, user_available
credits(&useron));
1691 return str;
1692 }
1693
1694 if (code_match(sp, "CDTLEFT", ¶m))
>>> CID 640970: Insecure data handling (INTEGER_OVERFLOW)
>>> The cast of "user_available_credits(&this->useron)" to a signed type
could result in a negative number.
1695 return byte_count(static_cast
|
[ << oldest | < older | list | newer > | newest >> ]