MT#62023 htable: don't crash on missing htable definition

Change-Id: Ie31517cc78ee0bb2c5541cb5410981d466e6ae88

debian/patches/series

@@ -33,6 +33,7 @@ sipwise/sca-add-pai_avp-parameter.patch
 ## upstream 5.8
 upstream/db_sqlite-fix-incompatible-pointer-types-warning.patch
 upstream/db_unixodbc-fix-incompatible-pointer-types-warning.patch
+upstream/htable-do-not-print-item-name-before-is-computed.patch
 ## upstream master
 upstream/rtpengine_rework_rtpp_flags.patch
 upstream/cfgutils-expose-optional-second-string-key-for-lock.patch

debian/patches/upstream/htable-do-not-print-item-name-before-is-computed.patch

@@ -0,0 +1,35 @@
+From: Daniel-Constantin Mierla <miconda@gmail.com>
+Date: Thu, 11 Jul 2024 19:38:24 +0200
+Subject: [PATCH] htable: do not print item name before is computed
+
+(cherry picked from commit e2193b9cd7f59377c54e0424c0d4a1628a5a60de)
+---
+ src/modules/htable/ht_var.c | 8 ++++----
+ 1 file changed, 4 insertions(+), 4 deletions(-)
+
+diff --git a/src/modules/htable/ht_var.c b/src/modules/htable/ht_var.c
+index e79c786..4d7707f 100644
+--- a/src/modules/htable/ht_var.c
++++ b/src/modules/htable/ht_var.c
+@@ -38,8 +38,8 @@ int pv_get_ht_cell(struct sip_msg *msg, pv_param_t *param, pv_value_t *res)
+ 	if(hpv->ht == NULL) {
+ 		hpv->ht = ht_get_table(&hpv->htname);
+ 		if(hpv->ht == NULL) {
+-			LM_ERR("htable not found for getting $sht(%.*s=>%.*s)\n",
+-					hpv->htname.len, hpv->htname.s, htname.len, htname.s);
++			LM_ERR("htable not found for getting $sht(%.*s=>...)\n",
++					hpv->htname.len, hpv->htname.s);
+ 			return pv_get_null(msg, param, res);
+ 		}
+ 	}
+@@ -77,8 +77,8 @@ int pv_set_ht_cell(
+ 	if(hpv->ht == NULL)
+ 		hpv->ht = ht_get_table(&hpv->htname);
+ 	if(hpv->ht == NULL) {
+-		LM_ERR("htable not found for setting $sht(%.*s=>%.*s)\n",
+-				hpv->htname.len, hpv->htname.s, htname.len, htname.s);
++		LM_ERR("htable not found for setting $sht(%.*s=>...)\n",
++				hpv->htname.len, hpv->htname.s);
+ 		return -1;
+ 	}
+