From 7163789ee1f76589fcc5b98a0ccb564879258a44 Mon Sep 17 00:00:00 2001 From: Sebastian Kemper Date: Sun, 16 Oct 2022 17:23:09 +0200 Subject: [PATCH] kamailio: bump to 5.6.2 - 4 new modules: misctest, posops, siprepo & slack - 140-redis_use_pkg-config.patch refreshed - 160-ldap-rename-private-function-ldap_connect.patch and 170-core-add-support-for-time64-libcs.patch removed (applied upstream) - 160-siputils-fix-time_t-warning-and-a-typo.patch added to fix time_t warning (sent upstream, see [1]) [1] https://github.com/kamailio/kamailio/pull/3266 Signed-off-by: Sebastian Kemper --- net/kamailio/Makefile | 12 +- .../patches/140-redis_use_pkg-config.patch | 8 +- ...rename-private-function-ldap_connect.patch | 63 ---- ...putils-fix-time_t-warning-and-a-typo.patch | 49 +++ ...70-core-add-support-for-time64-libcs.patch | 327 ------------------ 5 files changed, 65 insertions(+), 394 deletions(-) delete mode 100644 net/kamailio/patches/160-ldap-rename-private-function-ldap_connect.patch create mode 100644 net/kamailio/patches/160-siputils-fix-time_t-warning-and-a-typo.patch delete mode 100644 net/kamailio/patches/170-core-add-support-for-time64-libcs.patch diff --git a/net/kamailio/Makefile b/net/kamailio/Makefile index dd09d6a..42273dc 100644 --- a/net/kamailio/Makefile +++ b/net/kamailio/Makefile @@ -9,12 +9,12 @@ include $(TOPDIR)/rules.mk PKG_NAME:=kamailio -PKG_VERSION:=5.5.2 +PKG_VERSION:=5.6.2 PKG_RELEASE:=$(AUTORELEASE) PKG_SOURCE_URL:=https://www.kamailio.org/pub/kamailio/$(PKG_VERSION)/src PKG_SOURCE:=kamailio-$(PKG_VERSION)_src.tar.gz -PKG_HASH:=998f337d0561b5d814ebb5c59b160e0c856e7c9f3e53bcee0f3285d003d1c05c +PKG_HASH:=ea3cd5d688c34208b92072c3844c8276b693e0ca2c688168ea0357978c76b32d PKG_USE_MIPS16:=0 PKG_LICENSE:=GPL-2.0+ @@ -125,6 +125,7 @@ MODULES_AVAILABLE:= \ matrix \ maxfwd \ mediaproxy \ + misctest \ mohqueue \ mqueue \ msilo \ @@ -142,6 +143,7 @@ MODULES_AVAILABLE:= \ permissions \ pike \ pipelimit \ + posops \ prefix_route \ presence \ presence_conference \ @@ -178,10 +180,12 @@ MODULES_AVAILABLE:= \ secfilter \ sipcapture \ sipdump \ + siprepo \ sipt \ siptrace \ siputils \ sl \ + slack \ sms \ smsops \ snmpstats \ @@ -567,6 +571,7 @@ $(eval $(call BuildKamailioModule,mangler,SDP mangling,,)) $(eval $(call BuildKamailioModule,matrix,Matrix operations,,)) $(eval $(call BuildKamailioModule,maxfwd,Max-Forward processor,,)) $(eval $(call BuildKamailioModule,mediaproxy,Automatic NAT traversal,,+kamailio-mod-dialog)) +$(eval $(call BuildKamailioModule,misctest,Debugging/testing,This is a debugging/test module. It implements functions (both script\nand rpcs) that can be used to stress the memory allocator or force\nmemory leaks.\n\nWarning: This module should never be used in a production environment.,,,)) $(eval $(call BuildKamailioModule,mohqueue,Music on hold queuing system,,+kamailio-mod-rtpproxy +kamailio-mod-sl +kamailio-mod-tm)) $(eval $(call BuildKamailioModule,mqueue,Generic message queue system,,)) $(eval $(call BuildKamailioModule,msilo,SIP message silo,,+kamailio-mod-tm)) @@ -584,6 +589,7 @@ $(eval $(call BuildKamailioModule,pdt,Prefix-to-Domain translator,,)) $(eval $(call BuildKamailioModule,permissions,Permissions control,,)) $(eval $(call BuildKamailioModule,pike,Flood detector,,)) $(eval $(call BuildKamailioModule,pipelimit,Traffic shaping policies,,+kamailio-mod-sl)) +$(eval $(call BuildKamailioModule,posops,Position SIP message buffer ops,The module exports utility functions to work with the position inside the SIP\nmessage buffer. Among them are function to add or remove content at\na specific position.,,,)) $(eval $(call BuildKamailioModule,prefix_route,Execute based on prefix,,)) $(eval $(call BuildKamailioModule,presence,Presence server,,+kamailio-mod-sl +kamailio-mod-tm,)) $(eval $(call BuildKamailioModule,presence_conference,Conference events,,+kamailio-mod-presence)) @@ -620,10 +626,12 @@ $(eval $(call BuildKamailioModule,seas,Sip Express Application Server,,+kamailio $(eval $(call BuildKamailioModule,secfilter,Allow/block filters,,,)) $(eval $(call BuildKamailioModule,sipcapture,SIP capture,,)) $(eval $(call BuildKamailioModule,sipdump,Save SIP traffic,,)) +$(eval $(call BuildKamailioModule,siprepo,SIP Repo,This module can store and fetch SIP message content in an in-memory hash\ntable.,,,)) $(eval $(call BuildKamailioModule,sipt,SIP-T and SIP-I operations,,)) $(eval $(call BuildKamailioModule,siptrace,SIP trace,,)) $(eval $(call BuildKamailioModule,siputils,SIP utilities,,+kamailio-mod-sl)) $(eval $(call BuildKamailioModule,sl,Stateless replier,,)) +$(eval $(call BuildKamailioModule,slack,Slack integration,This module provides integration with Slack over webhooks.,+libcurl,,)) $(eval $(call BuildKamailioModule,sms,SIP-to-SMS IM gateway,,+kamailio-mod-tm)) $(eval $(call BuildKamailioModule,smsops,Handle SMS packets in SIP,,)) $(eval $(call BuildKamailioModule,snmpstats,SNMP interface for statistics,,+libnetsnmp)) diff --git a/net/kamailio/patches/140-redis_use_pkg-config.patch b/net/kamailio/patches/140-redis_use_pkg-config.patch index 86e7d7b..cbcaca1 100644 --- a/net/kamailio/patches/140-redis_use_pkg-config.patch +++ b/net/kamailio/patches/140-redis_use_pkg-config.patch @@ -15,7 +15,7 @@ HIREDISDEFS=-I$(LOCALBASE)/include --- a/src/modules/db_redis/Makefile +++ b/src/modules/db_redis/Makefile -@@ -5,12 +5,10 @@ include ../../Makefile.defs +@@ -5,7 +5,6 @@ include ../../Makefile.defs auto_gen= NAME=db_redis.so @@ -23,11 +23,15 @@ HIREDIS_BUILDER = $(shell \ if pkg-config --exists hiredis; then \ echo 'pkg-config hiredis'; \ +@@ -16,8 +15,6 @@ HIREDIS_CLUSTER_BUILDER = $(shell \ + echo 'pkg-config hiredis_cluster'; \ fi) --endif +-endif +- ifeq ($(HIREDIS_BUILDER),) HIREDISDEFS=-I$(LOCALBASE)/include -I$(LOCALBASE)/include/hiredis -I/usr/include/hiredis + HIREDISLIBS=-L$(LOCALBASE)/lib -lhiredis --- a/src/modules/ndb_redis/Makefile +++ b/src/modules/ndb_redis/Makefile @@ -5,12 +5,10 @@ include ../../Makefile.defs diff --git a/net/kamailio/patches/160-ldap-rename-private-function-ldap_connect.patch b/net/kamailio/patches/160-ldap-rename-private-function-ldap_connect.patch deleted file mode 100644 index 4ee9985..0000000 --- a/net/kamailio/patches/160-ldap-rename-private-function-ldap_connect.patch +++ /dev/null @@ -1,63 +0,0 @@ -From 362c6de08058948d0a70a1eeb14ec718f922b4e5 Mon Sep 17 00:00:00 2001 -From: Sebastian Kemper -Date: Wed, 15 Dec 2021 23:30:38 +0100 -Subject: [PATCH] ldap: rename private function ldap_connect - -When compiling against openldap 2.6.0 kamailio's private ldap_connect() clashes -with openldap's own. curl dealt with a similar issue earlier (see [1]). - -Simply rename the function to avoid the issue. - -In file included from ldap_api_fn.c:37: -ldap_connect.h:34:12: error: conflicting types for 'ldap_connect'; have 'int(char *)' - 34 | extern int ldap_connect(char *_ld_name); - | ^~~~~~~~~~~~ -In file included from ldap_api_fn.c:33: -/home/sk/tmp/sdk/openwrt-sdk-ath79-generic_gcc-11.2.0_musl.Linux-x86_64/staging_dir/target-mips_24kc_musl/usr/include/ldap.h:1555:1: note: previous declaration of 'ldap_connect' with type 'int(LDAP *)' {aka 'int(struct ldap *)'} - 1555 | ldap_connect( LDAP *ld ); - | ^~~~~~~~~~~~ -make[5]: *** [../../Makefile.rules:100: ldap_api_fn.o] Error 1 -make[4]: *** [Makefile:511: modules] Error 1 - -[1] https://github.com/curl/curl/commit/8bdde6b14ce3b5fd71c772a578fcbd4b6fa6df19 - -Signed-off-by: Sebastian Kemper ---- - src/modules/ldap/ldap_connect.c | 2 +- - src/modules/ldap/ldap_connect.h | 2 +- - src/modules/ldap/ldap_mod.c | 2 +- - 3 files changed, 3 insertions(+), 3 deletions(-) - ---- a/src/modules/ldap/ldap_connect.c -+++ b/src/modules/ldap/ldap_connect.c -@@ -185,7 +185,7 @@ int ldap_connect_ex(char *_ld_name, int - return 0; - } - --int ldap_connect(char *_ld_name) -+int oldap_connect(char *_ld_name) - { - return ldap_connect_ex(_ld_name, L_DBG); - } ---- a/src/modules/ldap/ldap_connect.h -+++ b/src/modules/ldap/ldap_connect.h -@@ -31,7 +31,7 @@ - #include "../../core/str.h" - #include "../../core/dprint.h" - --extern int ldap_connect(char *_ld_name); -+extern int oldap_connect(char *_ld_name); - extern int ldap_disconnect(char *_ld_name); - extern int ldap_reconnect(char *_ld_name); - extern int ldap_get_vendor_version(char **_version); ---- a/src/modules/ldap/ldap_mod.c -+++ b/src/modules/ldap/ldap_mod.c -@@ -167,7 +167,7 @@ static int child_init(int rank) - return -1; - } - -- if(ldap_connect(ld_name) != 0) { -+ if(oldap_connect(ld_name) != 0) { - LM_ERR("[%s]: failed to connect to LDAP host(s)\n", ld_name); - ldap_disconnect(ld_name); - return -1; diff --git a/net/kamailio/patches/160-siputils-fix-time_t-warning-and-a-typo.patch b/net/kamailio/patches/160-siputils-fix-time_t-warning-and-a-typo.patch new file mode 100644 index 0000000..55f20ee --- /dev/null +++ b/net/kamailio/patches/160-siputils-fix-time_t-warning-and-a-typo.patch @@ -0,0 +1,49 @@ +From f32e1d9c96f6ba9df3c27178c689ccd27eabaafc Mon Sep 17 00:00:00 2001 +From: Sebastian Kemper +Date: Sun, 16 Oct 2022 16:51:41 +0200 +Subject: [PATCH] siputils: fix time_t warning and a typo + +Fix the below warning one gets when compiling siputils for a 32 bit +target against a time64 libc (musl). + +Also fix a spelling mistake on the same line ("autdated" -> "outdated"). + +siputils.c: In function 'ki_hdr_date_check': +../../core/parser/../dprint.h:321:73: warning: format '%ld' expects argument of type 'long int', but argument 11 has type 'time_t' {aka 'long long int'} [-Wformat=] + 321 | fprintf(stderr, "%2d(%d) %s: %.*s%s%s%s" fmt, \ + | ^~~~~~~~~~~~~~~~~~~~~~~~ +../../core/parser/../dprint.h:345:25: note: in expansion of macro 'LOG_FX' + 345 | LOG_FX(facility, level, lname, prefix, _FUNC_NAME_, fmt, ## args) + | ^~~~~~ +../../core/parser/../dprint.h:351:25: note: in expansion of macro 'LOG_FL' + 351 | LOG_FL(facility, level, NULL, prefix, fmt, ## args) + | ^~~~~~ +../../core/parser/../dprint.h:354:25: note: in expansion of macro 'LOG_FP' + 354 | LOG_FP(DEFAULT_FACILITY, (level), LOC_INFO, fmt, ## args) + | ^~~~~~ +../../core/parser/../dprint.h:392:37: note: in expansion of macro 'LOG' + 392 | # define ERR(fmt, args...) LOG(L_ERR, fmt , ## args) + | ^~~ +../../core/parser/../dprint.h:418:16: note: in expansion of macro 'ERR' + 418 | #define LM_ERR ERR + | ^~~ +siputils.c:562:17: note: in expansion of macro 'LM_ERR' + 562 | LM_ERR("autdated date header value (%ld sec)\n", tnow - tmsg + tdiff); + | ^~~~~~ + +Signed-off-by: Sebastian Kemper +--- + src/modules/siputils/siputils.c | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +--- a/src/modules/siputils/siputils.c ++++ b/src/modules/siputils/siputils.c +@@ -559,7 +559,7 @@ static int ki_hdr_date_check(sip_msg_t* + } + + if (tnow > tmsg + tdiff) { +- LM_ERR("autdated date header value (%ld sec)\n", tnow - tmsg + tdiff); ++ LM_ERR("outdated date header value (%" TIME_T_FMT " sec)\n", TIME_T_CAST(tnow - tmsg + tdiff)); + return -4; + } else { + LM_ERR("Date header value OK\n"); diff --git a/net/kamailio/patches/170-core-add-support-for-time64-libcs.patch b/net/kamailio/patches/170-core-add-support-for-time64-libcs.patch deleted file mode 100644 index 79c0127..0000000 --- a/net/kamailio/patches/170-core-add-support-for-time64-libcs.patch +++ /dev/null @@ -1,327 +0,0 @@ -From 3f224cff811e970ad8279c89fffbfd800b85bb64 Mon Sep 17 00:00:00 2001 -From: Sebastian Kemper -Date: Sat, 23 Oct 2021 01:23:40 +0200 -Subject: [PATCH] core: add support for time64 libcs - -libcs are implementing changes to fix the year 2038 issue on 32 bit -platforms (see [1]). musl libc already went ahead and implemented it, -starting with musl-1.2.0 (see [2]). - -This commit adds the groundwork for always using the "lld" format when -dealing with time_t and casting the values to be formatted to (long -long). The aim is to be future proof and portable. - -Example warning when not using TIME_T_FMT while compiling for a 32 bit -target using a time64 libc: - -In file included from auth_identity.c:50: -auth_identity.c: In function 'check_date': -../../core/dprint.h:316:73: warning: format '%ld' expects argument of type 'long int', but argument 11 has type 'time_t' {aka 'long long int'} [-Wformat=] - 316 | fprintf(stderr, "%2d(%d) %s: %.*s%s%s%s" fmt, \ - | ^~~~~~~~~~~~~~~~~~~~~~~~ -../../core/dprint.h:340:25: note: in expansion of macro 'LOG_FX' - 340 | LOG_FX(facility, level, lname, prefix, _FUNC_NAME_, fmt, ## args) - | ^~~~~~ -../../core/dprint.h:346:25: note: in expansion of macro 'LOG_FL' - 346 | LOG_FL(facility, level, NULL, prefix, fmt, ## args) - | ^~~~~~ -../../core/dprint.h:349:25: note: in expansion of macro 'LOG_FP' - 349 | LOG_FP(DEFAULT_FACILITY, (level), LOC_INFO, fmt, ## args) - | ^~~~~~ -auth_identity.c:594:17: note: in expansion of macro 'LOG' - 594 | LOG(L_INFO, "AUTH_IDENTITY VERIFIER: Outdated date header value (%ld sec)\n", tnow - tmsg + glb_iauthval); - | ^~~ - -[1] https://sourceware.org/glibc/wiki/Y2038ProofnessDesign -[2] https://musl.libc.org/time64.html - -Signed-off-by: Sebastian Kemper - -auth_identity: use TIME_T_FMT - -Signed-off-by: Sebastian Kemper - -cdp: use TIME_T_FMT - -Signed-off-by: Sebastian Kemper - -db_redis: use TIME_T_FMT - -Signed-off-by: Sebastian Kemper - -ims_charging: use TIME_T_FMT - -Signed-off-by: Sebastian Kemper - -ims_dialog: use TIME_T_FMT - -Signed-off-by: Sebastian Kemper - -ims_icscf: use TIME_T_FMT - -Signed-off-by: Sebastian Kemper - -ims_registrar_pcscf: use TIME_T_FMT - -Signed-off-by: Sebastian Kemper - -ims_registrar_scscf: use TIME_T_FMT - -Signed-off-by: Sebastian Kemper - -ims_usrloc_scscf: use TIME_T_FMT - -Signed-off-by: Sebastian Kemper - -nat_traversal: use TIME_T_FMT - -The size of the time_t destination is unknown as it is depending on the -platform. So we "read into a known-size integer (either int or long -long) and then assign the value to a time_t as a second step" (quote -from [1]). - -[1] https://stackoverflow.com/questions/4171478/how-to-read-data-into-a-time-t-variable-using-scanf - -Signed-off-by: Sebastian Kemper - -xhttp_pi: use TIME_T_FMT - -Signed-off-by: Sebastian Kemper ---- - src/core/dprint.h | 2 ++ - src/modules/auth_identity/auth_identity.c | 2 +- - src/modules/cdp/routing.c | 4 ++-- - src/modules/db_redis/redis_dbase.c | 2 +- - src/modules/ims_charging/ims_ro.c | 3 ++- - src/modules/ims_dialog/dlg_handlers.c | 2 +- - src/modules/ims_icscf/scscf_list.c | 7 ++++--- - src/modules/ims_registrar_pcscf/notify.c | 4 ++-- - src/modules/ims_registrar_pcscf/save.c | 5 +++-- - src/modules/ims_registrar_scscf/registrar_notify.c | 5 +++-- - src/modules/ims_registrar_scscf/save.c | 2 +- - src/modules/ims_usrloc_scscf/impurecord.c | 7 ++++--- - src/modules/nat_traversal/nat_traversal.c | 5 ++++- - src/modules/xhttp_pi/xhttp_pi_fnc.c | 8 ++++---- - 14 files changed, 34 insertions(+), 24 deletions(-) - ---- a/src/core/dprint.h -+++ b/src/core/dprint.h -@@ -35,6 +35,8 @@ - #include "compiler_opt.h" - #include "cfg_core.h" - -+#define TIME_T_FMT "lld" -+#define TIME_T_CAST(x) ((long long)(x)) - - /** dicover the function name */ - /* C >= 99 has __func__, older gcc versions have __FUNCTION__ */ ---- a/src/modules/auth_identity/auth_identity.c -+++ b/src/modules/auth_identity/auth_identity.c -@@ -591,7 +591,7 @@ static int check_date(struct sip_msg* ms - } - - if (tnow > tmsg + glb_iauthval) { -- LOG(L_INFO, "AUTH_IDENTITY VERIFIER: Outdated date header value (%ld sec)\n", tnow - tmsg + glb_iauthval); -+ LOG(L_INFO, "AUTH_IDENTITY VERIFIER: Outdated date header value (%" TIME_T_FMT " sec)\n", TIME_T_CAST(tnow - tmsg + glb_iauthval)); - return -4; - } else - LOG(AUTH_DBG_LEVEL, "AUTH_IDENTITY VERIFIER: Date header value OK\n"); ---- a/src/modules/cdp/routing.c -+++ b/src/modules/cdp/routing.c -@@ -146,10 +146,10 @@ peer* get_first_connected_route(cdp_sess - } - - least_recent_time = peers[0]->last_selected; -- LM_DBG("peer [%.*s] was last used @ %ld\n", peers[0]->fqdn.len, peers[0]->fqdn.s, peers[0]->last_selected); -+ LM_DBG("peer [%.*s] was last used @ %" TIME_T_FMT "\n", peers[0]->fqdn.len, peers[0]->fqdn.s, TIME_T_CAST(peers[0]->last_selected)); - p = peers[0]; - for (j = 1; j < peer_count; j++) { -- LM_DBG("Peer [%.*s] was last used at [%ld]\n", peers[j]->fqdn.len, peers[j]->fqdn.s, peers[j]->last_selected); -+ LM_DBG("Peer [%.*s] was last used at [%" TIME_T_FMT "]\n", peers[j]->fqdn.len, peers[j]->fqdn.s, TIME_T_CAST(peers[j]->last_selected)); - if (peers[j]->last_selected < least_recent_time) { - least_recent_time = peers[j]->last_selected; - p = peers[j]; ---- a/src/modules/db_redis/redis_dbase.c -+++ b/src/modules/db_redis/redis_dbase.c -@@ -142,7 +142,7 @@ static int db_redis_val2str(const db_val - _str->s[_str->len] = '\0'; - break; - case DB1_DATETIME: -- LM_DBG("converting datetime value %ld to str\n", VAL_TIME(v)); -+ LM_DBG("converting datetime value %" TIME_T_FMT " to str\n", TIME_T_CAST(VAL_TIME(v))); - _str->s = (char*)pkg_malloc(_str->len); - if (!_str->s) goto memerr; - localtime_r(&(VAL_TIME(v)), &_time); ---- a/src/modules/ims_charging/ims_ro.c -+++ b/src/modules/ims_charging/ims_ro.c -@@ -856,7 +856,8 @@ void send_ccr_stop_with_param(struct ro_ - LM_DBG("Final used number of seconds for session is %ld\n", used); - } - -- LM_DBG("Call started at %ld and ended at %ld and lasted %d seconds and so far we have billed for %ld seconds\n", ro_session->start_time, stop_time, -+ LM_DBG("Call started at %" TIME_T_FMT " and ended at %" TIME_T_FMT " and lasted %d seconds and so far we have billed for %ld seconds\n", -+ TIME_T_CAST(ro_session->start_time), TIME_T_CAST(stop_time), - actual_time_seconds, ro_session->billed + used); - if (ro_session->billed + used < actual_time_seconds) { - LM_DBG("Making adjustment by adding %ld seconds\n", actual_time_seconds - (ro_session->billed + used)); ---- a/src/modules/ims_dialog/dlg_handlers.c -+++ b/src/modules/ims_dialog/dlg_handlers.c -@@ -1867,7 +1867,7 @@ void internal_print_all_dlg(struct dlg_c - - LM_DBG("----------------------------"); - LM_DBG("Dialog h_entry:h_id = [%u : %u]\n", dlg->h_entry, dlg->h_id); -- LM_DBG("Dialog age: %ld\n", act_time - dlg->init_ts); -+ LM_DBG("Dialog age: %" TIME_T_FMT "\n", TIME_T_CAST(act_time - dlg->init_ts)); - LM_DBG("Dialog call-id: %.*s\n", dlg->callid.len, dlg->callid.s); - LM_DBG("Dialog state: %d\n", dlg->state); - LM_DBG("Dialog ref counter: %d\n", dlg->ref); ---- a/src/modules/ims_icscf/scscf_list.c -+++ b/src/modules/ims_icscf/scscf_list.c -@@ -586,15 +586,16 @@ void ims_icscf_timer_routine() { - sl = l->list; - while (sl) { - -- LM_DBG("INF: Score:[%4d] Start_time [%ld] S-CSCF: <%.*s> \n", -+ LM_DBG("INF: Score:[%4d] Start_time [%" TIME_T_FMT "] S-CSCF: <%.*s> \n", - sl->score, -- sl->start_time, -+ TIME_T_CAST(sl->start_time), - sl->scscf_name.len, sl->scscf_name.s); - time_t now = time(0); - time_t time_elapsed = now - sl->start_time; - if (time_elapsed > scscf_entry_expiry) { - -- LM_DBG("Scscf entry expired: Time now %ld Start time %ld - elapsed %ld\n", now, sl->start_time, time_elapsed); -+ LM_DBG("Scscf entry expired: Time now %" TIME_T_FMT " Start time %" TIME_T_FMT " - elapsed %" TIME_T_FMT "\n", -+ TIME_T_CAST(now), TIME_T_CAST(sl->start_time), TIME_T_CAST(time_elapsed)); - delete_list = 1; //if any of the entries in this list have expired remove the whole list! - - } ---- a/src/modules/ims_registrar_pcscf/notify.c -+++ b/src/modules/ims_registrar_pcscf/notify.c -@@ -209,8 +209,8 @@ int process_contact(udomain_t * _d, int - /*TODO_LATEST - put this back */ - } else {//state is active - //update this contact -- LM_DBG("This contact: <%.*s> is in state active and is in usrloc so just updating - old expires: %li, new expires: %i which is in %i seconds\n", contact_uri.len, contact_uri.s, -- pcontact->expires, -+ LM_DBG("This contact: <%.*s> is in state active and is in usrloc so just updating - old expires: %" TIME_T_FMT ", new expires: %i which is in %i seconds\n", contact_uri.len, contact_uri.s, -+ TIME_T_CAST(pcontact->expires), - expires, - expires - local_time_now); - if (ul.update_pcontact(_d, &ci, pcontact) != 0) { ---- a/src/modules/ims_registrar_pcscf/save.c -+++ b/src/modules/ims_registrar_pcscf/save.c -@@ -223,8 +223,9 @@ static inline int update_contacts(struct - } - //TODO_LATEST replace above - } else { //update contact -- LM_DBG("Updating contact: <%.*s>, old expires: %li, new expires: %i which is in %i seconds\n", c->uri.len, c->uri.s, -- pcontact->expires-local_time_now, -+ LM_DBG("Updating contact: <%.*s>, old expires: %" TIME_T_FMT ", new expires: %i which is in %i seconds\n", -+ c->uri.len, c->uri.s, -+ TIME_T_CAST(pcontact->expires-local_time_now), - expires, - expires-local_time_now); - ci.reg_state = PCONTACT_REGISTERED; ---- a/src/modules/ims_registrar_scscf/registrar_notify.c -+++ b/src/modules/ims_registrar_scscf/registrar_notify.c -@@ -54,6 +54,7 @@ - #include "../../core/parser/parse_from.h" - #include "../../core/parser/parse_content.h" - #include "../../core/parser/parse_uri.h" -+#include "../../core/dprint.h" /* TIME_T_FMT */ - #include - - #include "../../lib/ims/useful_defs.h" -@@ -1517,8 +1518,8 @@ void create_notifications(udomain_t* _t, - subscription_state.len = 0; - if (subscription_state.s) { - -- sprintf(subscription_state.s, "%.*s%ld", subs_active.len, -- subs_active.s, s->expires - act_time); -+ sprintf(subscription_state.s, "%.*s%" TIME_T_FMT, subs_active.len, -+ subs_active.s, TIME_T_CAST(s->expires - act_time)); - subscription_state.len = strlen(subscription_state.s); - } - ---- a/src/modules/ims_registrar_scscf/save.c -+++ b/src/modules/ims_registrar_scscf/save.c -@@ -545,7 +545,7 @@ static inline int update_contacts_helper - LM_DBG("Need to update contact: <%.*s>: " - "q_value [%d]," - "sos: [%d]," -- "expires [%ld]\n", chi->uri.len, chi->uri.s, qvalue, sos, expires - time(NULL)); -+ "expires [%" TIME_T_FMT "]\n", chi->uri.len, chi->uri.s, qvalue, sos, TIME_T_CAST(expires - time(NULL))); - - LM_DBG("packing contact information\n"); - if ((ci = pack_ci(msg, chi, expires, 0)) == 0) { ---- a/src/modules/ims_usrloc_scscf/impurecord.c -+++ b/src/modules/ims_usrloc_scscf/impurecord.c -@@ -244,7 +244,7 @@ void print_impurecord(FILE* _f, impureco - header = 1; - } - fprintf(_f, "\t\twatcher uri: <%.*s> and presentity uri: <%.*s>\n", subscriber->watcher_uri.len, subscriber->watcher_uri.s, subscriber->presentity_uri.len, subscriber->presentity_uri.s); -- fprintf(_f, "\t\tExpires: %ld\n", subscriber->expires); -+ fprintf(_f, "\t\tExpires: %" TIME_T_FMT "\n", TIME_T_CAST(subscriber->expires)); - subscriber = subscriber->next; - } - -@@ -405,8 +405,9 @@ static inline void process_impurecord(im - hascontacts = 1; - } - } else { -- LM_DBG("\t\tContact #%i - %.*s, Ref [%d] (expires in %ld seconds) (State: %d)\n", -- k, ptr->c.len, ptr->c.s, ptr->ref_count, ptr->expires - act_time, ptr->state); -+ LM_DBG("\t\tContact #%i - %.*s, Ref [%d] (expires in %" TIME_T_FMT " seconds) (State: %d)\n", -+ k, ptr->c.len, ptr->c.s, ptr->ref_count, -+ TIME_T_CAST(ptr->expires - act_time), ptr->state); - mustdeleteimpu = 0; - hascontacts = 1; - } ---- a/src/modules/nat_traversal/nat_traversal.c -+++ b/src/modules/nat_traversal/nat_traversal.c -@@ -1707,6 +1707,7 @@ static void restore_keepalive_state(void - unsigned h; - str host; - FILE *f; -+ long long ll_1, ll_2; - - if(!keepalive_state_file) - return; -@@ -1724,7 +1725,9 @@ static void restore_keepalive_state(void - res = fscanf(f, STATE_FILE_HEADER); // skip header - - while(true) { -- res = fscanf(f, "%63s %63s %ld %ld", uri, socket, &rtime, &stime); -+ res = fscanf(f, "%63s %63s %" TIME_T_FMT " %" TIME_T_FMT, uri, socket, &ll_1, &ll_2); -+ rtime = ll_1; -+ stime = ll_2; - if(res == EOF) { - if(ferror(f)) - LM_ERR("error while reading keepalive state file: %s\n", ---- a/src/modules/xhttp_pi/xhttp_pi_fnc.c -+++ b/src/modules/xhttp_pi/xhttp_pi_fnc.c -@@ -3186,18 +3186,18 @@ int ph_run_pi_cmd(pi_ctx_t* ctx) - val_str.len = max_page_len - ctx->reply.body.len; - if(db_time2str(values[j].val.time_val, - val_str.s, &val_str.len)!=0){ -- LM_ERR("Unable to convert double [%ld]\n", -- values[j].val.time_val); -+ LM_ERR("Unable to convert double [%" TIME_T_FMT "]\n", -+ TIME_T_CAST(values[j].val.time_val)); - goto error; - } - p += val_str.len; - ctx->reply.body.len += val_str.len; - if(link_on) XHTTP_PI_COPY_2(p,XHTTP_PI_SQUOT_GT,val_str); - LM_DBG(" got %.*s[%d]=>" -- "[%ld][%.*s]\n", -+ "[%" TIME_T_FMT "][%.*s]\n", - command->q_keys[j]->len, - command->q_keys[j]->s, i, -- values[j].val.time_val, -+ TIME_T_CAST(values[j].val.time_val), - val_str.len, val_str.s); - break; - default: -- 2.30.2