New Defects Reported

Subj : New Defects reported by Coverity Scan for Synchronet To : All From : scan-admin@coverity.com Date : Wed Feb 18 2026 01:50 pm ----==_mimepart_6995c3bce9f7_51bf02b0ff68619ac19798 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. 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 >::push_back(std::vector >::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 ----==_mimepart_6995c3bce9f7_51bf02b0ff68619ac19798 Content-Type: text/html; charset=us-ascii Content-Transfer-Encoding: 7bit New Defects Reported - Synchronet

Hi,

Please find the latest report on new defect(s) introduced to Synchronet found with Coverity Scan.

New Defects Found: 2
Defects Shown: Showing 2 of 2 defect(s)

Defect Details

** 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;

View Defects in Coverity Scan

Best regards,

The Coverity Scan Admin Team

----==_mimepart_6995c3bce9f7_51bf02b0ff68619ac19798-- .