This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/OpenBSC@lists.osmocom.org/.
Holger Freyther holger at freyther.de
> Begin forwarded message:
>
> From: scan-admin at coverity.com
> Subject: New Defects reported by Coverity Scan for Osmocom
> Date: 18 March 2016 at 18:01:06 GMT+1
> To: holger at freyther.de
>
>
> Hi,
>
> Please find the latest report on new defect(s) introduced to Osmocom found with Coverity Scan.
>
> 2 new defect(s) introduced to Osmocom found with Coverity Scan.
>
>
> New defect(s) Reported-by: Coverity Scan
> Showing 2 of 2 defect(s)
>
> ________________________________________________________________________________________________________
> *** CID 76332: API usage errors (PW.TOO_MANY_PRINTF_ARGS)
> /source-iuh/openbsc/openbsc/src/gprs/gprs_gmm.c: 120 in ()
> 114 case IU_EVENT_RAB_ASSIGN:
> 115 rc = sgsn_ranap_rab_ass_resp(mm, (RANAP_RAB_SetupOrModifiedItemIEs_t *)data);
> 116 break;
> 117 case IU_EVENT_IU_RELEASE:
> 118 mm->iu.integrity_active = 0;
> 119 /* Clean up ue_conn_ctx here */
>>>> CID 76332: API usage errors (PW.TOO_MANY_PRINTF_ARGS)
>>>> the format string ends before this argument
> 120 LOGMMCTXP(LOGL_INFO, mm, "IU release\n", type);
harmless, but easy to fix and the compiler already warns
>
> 46 llist_for_each_entry(conn, &network->subscr_conns, entry) {
>>>> CID 76331: API usage errors (PW.PRINTF_ARG_MISMATCH)
>>>> argument is incompatible with corresponding format string conversion
> 47 DEBUGP(DIUCS, "%3d: %s", i++, subscr_name(conn->subscr));
please avoid side-effects like this in DEBUGP. It might be completely off but with recent changes it might or might not be evaluated depending on your logging settings.
holger