**crypto_mod.c:343:12** ``` make[3]: 'libsrdb1.so.1.0' is up to date. LD (gcc) [M db_sqlite.so] db_sqlite.so CC (gcc) [M log_systemd.so] journal_send.o CC (gcc) [M log_systemd.so] log_systemd_mod.o LD (gcc) [M log_systemd.so] log_systemd.so CC (gcc) [M systemdops.so] systemdops_mod.o LD (gcc) [M systemdops.so] systemdops.so CC (gcc) [M crypto.so] crypto_uuid.o CC (gcc) [M crypto.so] crypto_aes.o CC (gcc) [M crypto.so] crypto_evcb.o CC (gcc) [M crypto.so] crypto_mod.o crypto_mod.c:343:12: warning: 'ki_crypto_hmac_sha256' defined but not used [-Wunused-function] static int ki_crypto_hmac_sha256(sip_msg_t* msg, str *ins, str *keys, str *dpv) ^~~~~~~~~~~~~~~~~~~~~ CC (gcc) [M crypto.so] api.o LD (gcc) [M crypto.so] crypto.so CC (gcc) [M tls.so] tls_util.o CC (gcc) [M tls.so] tls_domain.o CC (gcc) [M tls.so] tls_locking.o ``` Maybe not related to Kamailio sources ``` LD (gcc) [M usrloc.so] usrloc.so CC (gcc) [M xhttp.so] xhttp_mod.o CC (gcc) [M xhttp.so] xhttp_trans.o In file included from ../../core/str.h:24, from ../../core/pvar.h:30, from xhttp_trans.c:21: In function 'memset', inlined from 'xhttp_tr_eval_xhttpurl' at xhttp_trans.c:69:5: /usr/include/fortify/string.h:72:9: warning: '__builtin_memset' offset [9, 16] from the object at '_httpurl_str' is out of the bounds of referenced subobject 's' with type 'char *' at offset 0 [-Warray-bounds] 72 | return __builtin_memset(__d, __c, __n); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ LD (gcc) [M xhttp.so] xhttp.so CC (gcc) [M xhttp_prom.so] prom.o CC (gcc) [M xhttp_prom.so] prom_metric.o ```
Thanks for the report. About the crypto ki_crypto_hmac_sha256(..) function, it seems to be indeed not used in the module. It was introduced from @vitalikvoip in a pull request, maybe he can share some details. Otherwise it might be removed, if its not used.
About the other error, it seems that the fortify detects some warning in the xhttp_tr_eval_xhttpurl(..) function, seemed valid. I have pushed a commit to fix it.
Closed #2619.
ki_crypto_hmac_sha256() supposed to be exported to kemi, the patch adding it was missing that. I just pushed pushed a commit for it. It looks all the problems were addressed, I am closing this issue, reopen if other problems were introduced.