Subj : New Defects reported by Coverity Scan for Synchronet
To   : All
From : scan-admin@coverity.com
Date : Wed Feb 18 2026 13:50:52

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.


New defect(s) Reported-by: Coverity Scan
Showing 2 of 2 defect(s)


** CID 644273:       Resource leaks  (RESOURCE_LEAK)
/js_console.cpp: 422           in js_console_set(JSContext *, JSObject *, long, int, unsigned long *)()


_____________________________________________________________________________________________
*** CID 644273:         Resource leaks  (RESOURCE_LEAK)
/js_console.cpp: 422             in js_console_set(JSContext *, JSObject *, long, int, unsigned long *)()
416     			break;
417
418     		default:
419     			return JS_TRUE;
420     	}
421
>>>     CID 644273:         Resource leaks  (RESOURCE_LEAK)
>>>     Variable "sval" going out of scope leaks the storage it points to.
422     	return JS_TRUE;
423     }
424
425     #define CON_PROP_FLAGS JSPROP_ENUMERATE
426
427     static jsSyncPropertySpec js_console_properties[] = {

** CID 644272:       Performance inefficiencies  (COPY_INSTEAD_OF_MOVE)


_____________________________________________________________________________________________
*** CID 644272:         Performance inefficiencies  (COPY_INSTEAD_OF_MOVE)
/con_hi.cpp: 61             in sbbs_t::uselect(bool, unsigned int, const char *, const char *, const unsigned char *)()
55     	if (add) {
56     		if (name == nullptr)
57     			return -1;
58     		if (ar != nullptr && !chk_ar(ar, &useron, &client))
59     			return 0;
60     		uselect_item item = { name, num };
>>>     CID 644272:         Performance inefficiencies  (COPY_INSTEAD_OF_MOVE)
>>>     "item" is copied and then passed-by-reference as parameter to STL insertion function "std::vector<sbbs_t::uselect_item, std::allocator<sbbs_t::uselect_item> >::push_back(std::vector<sbbs_t::uselect_item, std::allocator<sbbs_t::uselect_item> >::value_type const &)", when it could be moved instead.
61     		uselect_items.push_back(item);
62     		return 0;
63     	}
64
65     	if (uselect_items.size() < 1)
66     		return -1;


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://scan.coverity.com/projects/synchronet?tab=overview



---
 * Synchronet * Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

.