aboutsummaryrefslogtreecommitdiffstats
path: root/epan/dissectors
diff options
context:
space:
mode:
authorMichael Mann <mmann78@netscape.net>2016-06-05 22:24:47 -0400
committerJaap Keuter <jaap.keuter@xs4all.nl>2016-06-06 06:03:58 +0000
commit57ef06e242dbbce390547d9711f810fc60829219 (patch)
treeeaadf36f96023dd59c3fd56565bf6e14c9e57431 /epan/dissectors
parent6333b98ea6f7733b150537cbba6f80f3e2d83d43 (diff)
*_stdup_printf -> strdup for "single string only" formatting.
Done for performance improvements. This could probably be done in checkAPIs.pl, but this was just a quick manual check with grepping. Change-Id: I91ff102cb528bb00fa2f65489de53890e7e46f2d Reviewed-on: https://code.wireshark.org/review/15751 Reviewed-by: Michael Mann <mmann78@netscape.net> Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl>
Diffstat (limited to 'epan/dissectors')
-rw-r--r--epan/dissectors/asn1/ldap/ldap.cnf4
-rw-r--r--epan/dissectors/asn1/ros/ros-err.cnf2
-rw-r--r--epan/dissectors/asn1/ros/ros-inv.cnf2
-rw-r--r--epan/dissectors/asn1/ros/ros-res.cnf2
-rw-r--r--epan/dissectors/file-pcapng.c12
-rw-r--r--epan/dissectors/packet-dcerpc-spoolss.c8
-rw-r--r--epan/dissectors/packet-devicenet.c2
-rw-r--r--epan/dissectors/packet-diameter_3gpp.c12
-rw-r--r--epan/dissectors/packet-dof.c24
-rw-r--r--epan/dissectors/packet-dtls.c2
-rw-r--r--epan/dissectors/packet-h450-ros.c6
-rw-r--r--epan/dissectors/packet-isakmp.c2
-rw-r--r--epan/dissectors/packet-k12.c2
-rw-r--r--epan/dissectors/packet-lbmpdmtcp.c4
-rw-r--r--epan/dissectors/packet-lbmr.c4
-rw-r--r--epan/dissectors/packet-lbtrm.c4
-rw-r--r--epan/dissectors/packet-lbtru.c4
-rw-r--r--epan/dissectors/packet-lbttcp.c4
-rw-r--r--epan/dissectors/packet-ldap.c4
-rw-r--r--epan/dissectors/packet-q932-ros.c6
-rw-r--r--epan/dissectors/packet-rpc.c2
-rw-r--r--epan/dissectors/packet-scsi.c2
-rw-r--r--epan/dissectors/packet-ssl-utils.c8
-rw-r--r--epan/dissectors/packet-ssl.c2
-rw-r--r--epan/dissectors/packet-zbee-nwk-gp.c4
-rw-r--r--epan/dissectors/packet-zbee-nwk.c4
26 files changed, 62 insertions, 70 deletions
diff --git a/epan/dissectors/asn1/ldap/ldap.cnf b/epan/dissectors/asn1/ldap/ldap.cnf
index 452c6261f1..0e0861b4ce 100644
--- a/epan/dissectors/asn1/ldap/ldap.cnf
+++ b/epan/dissectors/asn1/ldap/ldap.cnf
@@ -572,7 +572,7 @@ ldap_conv_info_t *ldap_info;
if(and_filter_string) {
proto_item_append_text(it, "%%s", and_filter_string);
- Filter_string=wmem_strdup_printf(wmem_packet_scope(), "%%s",and_filter_string);
+ Filter_string=wmem_strdup(wmem_packet_scope(), and_filter_string);
}
and_filter_string=old_and_filter_string;
@@ -596,7 +596,7 @@ ldap_conv_info_t *ldap_info;
%(DEFAULT_BODY)s
if(or_filter_string) {
proto_item_append_text(it, "%%s", or_filter_string);
- Filter_string=wmem_strdup_printf(wmem_packet_scope(), "%%s",or_filter_string);
+ Filter_string=wmem_strdup(wmem_packet_scope(), or_filter_string);
}
or_filter_string=old_or_filter_string;
diff --git a/epan/dissectors/asn1/ros/ros-err.cnf b/epan/dissectors/asn1/ros/ros-err.cnf
index 76aa331dcb..e9e9bce243 100644
--- a/epan/dissectors/asn1/ros/ros-err.cnf
+++ b/epan/dissectors/asn1/ros/ros-err.cnf
@@ -25,7 +25,7 @@
else if (actx->rose_ctx->d.code == 1)
descr = wmem_strdup_printf(wmem_packet_scope(), "ERR: %s", actx->rose_ctx->d.code_global);
} else {
- descr = wmem_strdup_printf(wmem_packet_scope(), "ERR:");
+ descr = wmem_strdup(wmem_packet_scope(), "ERR:");
}
if (actx->rose_ctx->apdu_depth >= 0)
diff --git a/epan/dissectors/asn1/ros/ros-inv.cnf b/epan/dissectors/asn1/ros/ros-inv.cnf
index 1bc9251c49..acbcaff2d2 100644
--- a/epan/dissectors/asn1/ros/ros-inv.cnf
+++ b/epan/dissectors/asn1/ros/ros-inv.cnf
@@ -25,7 +25,7 @@
else if (actx->rose_ctx->d.code == 1)
descr = wmem_strdup_printf(wmem_packet_scope(), "INV: %s", actx->rose_ctx->d.code_global);
} else {
- descr = wmem_strdup_printf(wmem_packet_scope(), "INV:");
+ descr = wmem_strdup(wmem_packet_scope(), "INV:");
}
if (actx->rose_ctx->apdu_depth >= 0)
diff --git a/epan/dissectors/asn1/ros/ros-res.cnf b/epan/dissectors/asn1/ros/ros-res.cnf
index f0900906cb..69b09f82a6 100644
--- a/epan/dissectors/asn1/ros/ros-res.cnf
+++ b/epan/dissectors/asn1/ros/ros-res.cnf
@@ -26,7 +26,7 @@
else if (actx->rose_ctx->d.code == 1)
descr = wmem_strdup_printf(wmem_packet_scope(), "RES: %s", actx->rose_ctx->d.code_global);
} else {
- descr = wmem_strdup_printf(wmem_packet_scope(), "RES:");
+ descr = wmem_strdup(wmem_packet_scope(), "RES:");
}
if (actx->rose_ctx->apdu_depth >= 0)
diff --git a/epan/dissectors/file-pcapng.c b/epan/dissectors/file-pcapng.c
index 51c7c67a25..098b076662 100644
--- a/epan/dissectors/file-pcapng.c
+++ b/epan/dissectors/file-pcapng.c
@@ -680,8 +680,7 @@ static gint dissect_options(proto_tree *tree, packet_info *pinfo,
set_address_tvb(&addr, AT_IPv4, 4, tvb, offset);
offset += 4;
- str = wmem_strdup_printf(wmem_packet_scope(), "%s",
- address_to_display(wmem_packet_scope(), &addr));
+ str = address_to_display(wmem_packet_scope(), &addr);
break;
case 0x0004:
@@ -695,8 +694,7 @@ static gint dissect_options(proto_tree *tree, packet_info *pinfo,
set_address_tvb(&addr, AT_IPv6, 16, tvb, offset);
offset += 16;
- str = wmem_strdup_printf(wmem_packet_scope(), "%s",
- address_to_display(wmem_packet_scope(), &addr));
+ str = address_to_display(wmem_packet_scope(), &addr);
break;
default:
@@ -1142,8 +1140,7 @@ static gint dissect_block(proto_tree *tree, packet_info *pinfo, tvbuff_t *tvb,
}
}
- str = wmem_strdup_printf(wmem_packet_scope(), "%s",
- address_to_display(wmem_packet_scope(), &addr));
+ str = address_to_display(wmem_packet_scope(), &addr);
break;
case 0x0002: /* IPv6 Record */
if (record_length < 17) {
@@ -1168,8 +1165,7 @@ static gint dissect_block(proto_tree *tree, packet_info *pinfo, tvbuff_t *tvb,
}
}
- str = wmem_strdup_printf(wmem_packet_scope(), "%s",
- address_to_display(wmem_packet_scope(), &addr));
+ str = address_to_display(wmem_packet_scope(), &addr);
break;
default:
diff --git a/epan/dissectors/packet-dcerpc-spoolss.c b/epan/dissectors/packet-dcerpc-spoolss.c
index 3327f8c9a9..8365817769 100644
--- a/epan/dissectors/packet-dcerpc-spoolss.c
+++ b/epan/dissectors/packet-dcerpc-spoolss.c
@@ -958,8 +958,8 @@ SpoolssSetPrinterData_q(tvbuff_t *tvb, int offset,
/* GetPrinterDataEx() stores the key/value in se_data */
if(!pinfo->fd->flags.visited){
if(!dcv->se_data){
- dcv->se_data = wmem_strdup_printf(wmem_file_scope(),
- "%s", value_name?value_name:"");
+ dcv->se_data = wmem_strdup(wmem_file_scope(),
+ value_name?value_name:"");
}
}
@@ -2448,8 +2448,8 @@ SpoolssOpenPrinterEx_q(tvbuff_t *tvb, int offset,
/* OpenPrinterEx() stores the key/value in se_data */
if(!pinfo->fd->flags.visited){
if(!dcv->se_data){
- dcv->se_data = wmem_strdup_printf(wmem_file_scope(),
- "%s", name?name:"");
+ dcv->se_data = wmem_strdup(wmem_file_scope(),
+ name?name:"");
}
}
diff --git a/epan/dissectors/packet-devicenet.c b/epan/dissectors/packet-devicenet.c
index b79af89aa9..8e6dd66b41 100644
--- a/epan/dissectors/packet-devicenet.c
+++ b/epan/dissectors/packet-devicenet.c
@@ -133,7 +133,7 @@ static gboolean uat_devicenet_record_update_cb(void* r, char** err) {
uat_devicenet_record_t* rec = (uat_devicenet_record_t *)r;
if (rec->mac_id > 63) {
- *err = g_strdup_printf("MAC ID must be between 0-63");
+ *err = g_strdup("MAC ID must be between 0-63");
return FALSE;
}
return TRUE;
diff --git a/epan/dissectors/packet-diameter_3gpp.c b/epan/dissectors/packet-diameter_3gpp.c
index e5a53232c7..d65a4ffa70 100644
--- a/epan/dissectors/packet-diameter_3gpp.c
+++ b/epan/dissectors/packet-diameter_3gpp.c
@@ -315,8 +315,7 @@ dissect_diameter_3gpp_imeisv(tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *
item = proto_tree_add_item(tree, hf_diameter_3gpp_imeisv, tvb, offset, length, ENC_UTF_8 | ENC_NA);
PROTO_ITEM_SET_GENERATED(item);
- diam_sub_dis->avp_str = wmem_strdup_printf(wmem_packet_scope(), "%s",
- tvb_get_string_enc(wmem_packet_scope(), tvb, offset, length, ENC_UTF_8 | ENC_NA));
+ diam_sub_dis->avp_str = tvb_get_string_enc(wmem_packet_scope(), tvb, offset, length, ENC_UTF_8 | ENC_NA);
}
return length;
@@ -390,8 +389,7 @@ dissect_diameter_3gpp_af_application_identifier(tvbuff_t *tvb, packet_info *pinf
item = proto_tree_add_item(tree, hf_diameter_3gpp_af_application_identifier, tvb, offset, length, ENC_UTF_8 | ENC_NA);
PROTO_ITEM_SET_GENERATED(item);
- diam_sub_dis->avp_str = wmem_strdup_printf(wmem_packet_scope(), "%s",
- tvb_get_string_enc(wmem_packet_scope(), tvb, offset, length, ENC_UTF_8 | ENC_NA));
+ diam_sub_dis->avp_str = tvb_get_string_enc(wmem_packet_scope(), tvb, offset, length, ENC_UTF_8 | ENC_NA);
}
return length;
@@ -416,8 +414,7 @@ dissect_diameter_3gpp_af_charging_identifier(tvbuff_t *tvb, packet_info *pinfo _
item = proto_tree_add_item(tree, hf_diameter_3gpp_af_charging_identifier, tvb, offset, length, ENC_UTF_8 | ENC_NA);
PROTO_ITEM_SET_GENERATED(item);
- diam_sub_dis->avp_str = wmem_strdup_printf(wmem_packet_scope(), "%s",
- tvb_get_string_enc(wmem_packet_scope(), tvb, offset, length, ENC_UTF_8 | ENC_NA));
+ diam_sub_dis->avp_str = tvb_get_string_enc(wmem_packet_scope(), tvb, offset, length, ENC_UTF_8 | ENC_NA);
}
return length;
@@ -1012,8 +1009,7 @@ dissect_diameter_3gpp_charging_rule_name(tvbuff_t *tvb, packet_info *pinfo _U_,
item = proto_tree_add_item(tree, hf_diameter_3gpp_charging_rule_name, tvb, offset, length, ENC_UTF_8 | ENC_NA);
PROTO_ITEM_SET_GENERATED(item);
- diam_sub_dis->avp_str = wmem_strdup_printf(wmem_packet_scope(), "%s",
- tvb_get_string_enc(wmem_packet_scope(), tvb, offset, length, ENC_UTF_8 | ENC_NA));
+ diam_sub_dis->avp_str = tvb_get_string_enc(wmem_packet_scope(), tvb, offset, length, ENC_UTF_8 | ENC_NA);
}
return length;
diff --git a/epan/dissectors/packet-dof.c b/epan/dissectors/packet-dof.c
index 0bb31b2fa8..e3d1fcef8b 100644
--- a/epan/dissectors/packet-dof.c
+++ b/epan/dissectors/packet-dof.c
@@ -3365,30 +3365,30 @@ static void secmode_list_update_cb(void *r, const char **err)
size = (guint32)strlen(rec->domain);
if (!VALIDHEX(rec->domain[0]) && !dof_oid_create_internal(rec->domain, &size, NULL))
{
- *err = wmem_strdup_printf(wmem_packet_scope(), "Invalid domain [must be valid OID].");
+ *err = wmem_strdup(wmem_packet_scope(), "Invalid domain [must be valid OID].");
return;
}
else if (!count_hex_bytes(rec->domain))
{
- *err = wmem_strdup_printf(wmem_packet_scope(), "Invalid domain [must be valid OID].");
+ *err = wmem_strdup(wmem_packet_scope(), "Invalid domain [must be valid OID].");
return;
}
size = (guint32)strlen(rec->identity);
if (!VALIDHEX(rec->identity[0]) && !dof_oid_create_internal(rec->identity, &size, NULL))
{
- *err = wmem_strdup_printf(wmem_packet_scope(), "Invalid identity [must be valid OID].");
+ *err = wmem_strdup(wmem_packet_scope(), "Invalid identity [must be valid OID].");
return;
}
else if (!count_hex_bytes(rec->identity))
{
- *err = wmem_strdup_printf(wmem_packet_scope(), "Invalid identity [must be valid OID].");
+ *err = wmem_strdup(wmem_packet_scope(), "Invalid identity [must be valid OID].");
return;
}
if (count_hex_bytes(rec->kek) != 32)
{
- *err = wmem_strdup_printf(wmem_packet_scope(), "Invalid KEK [must be 32 byte key].");
+ *err = wmem_strdup(wmem_packet_scope(), "Invalid KEK [must be 32 byte key].");
return;
}
}
@@ -3455,7 +3455,7 @@ static void seckey_list_update_cb(void *r, const char **err)
*err = NULL;
if (count_hex_bytes(rec->key) != 32)
{
- *err = wmem_strdup_printf(wmem_packet_scope(), "Invalid secret [must be 32 bytes].");
+ *err = wmem_strdup(wmem_packet_scope(), "Invalid secret [must be 32 bytes].");
return;
}
}
@@ -3507,13 +3507,13 @@ static void identsecret_list_update_cb(void *r, const char **err)
{
if (dof_oid_create_internal(rec->domain, &size, NULL))
{
- *err = wmem_strdup_printf(wmem_packet_scope(), "Invalid domain [must be valid OID].");
+ *err = wmem_strdup(wmem_packet_scope(), "Invalid domain [must be valid OID].");
return;
}
}
else if (!count_hex_bytes(rec->domain))
{
- *err = wmem_strdup_printf(wmem_packet_scope(), "Invalid domain [must be valid OID].");
+ *err = wmem_strdup(wmem_packet_scope(), "Invalid domain [must be valid OID].");
return;
}
@@ -3522,19 +3522,19 @@ static void identsecret_list_update_cb(void *r, const char **err)
{
if (dof_oid_create_internal(rec->identity, &size, NULL))
{
- *err = wmem_strdup_printf(wmem_packet_scope(), "Invalid identity [must be valid OID].");
+ *err = wmem_strdup(wmem_packet_scope(), "Invalid identity [must be valid OID].");
return;
}
}
else if (!count_hex_bytes(rec->identity))
{
- *err = wmem_strdup_printf(wmem_packet_scope(), "Invalid identity [must be valid OID].");
+ *err = wmem_strdup(wmem_packet_scope(), "Invalid identity [must be valid OID].");
return;
}
if (count_hex_bytes(rec->secret) != 32)
{
- *err = wmem_strdup_printf(wmem_packet_scope(), "Invalid secret [must be 32 byte key].");
+ *err = wmem_strdup(wmem_packet_scope(), "Invalid secret [must be 32 byte key].");
return;
}
}
@@ -3719,7 +3719,7 @@ static gboolean identsecret_chk_cb(void *r _U_, const char *p _U_, unsigned len
if (!num_protos)
{
- *err = wmem_strdup_printf(wmem_packet_scope(), "No protocols given");
+ *err = wmem_strdup(wmem_packet_scope(), "No protocols given");
return FALSE;
}
diff --git a/epan/dissectors/packet-dtls.c b/epan/dissectors/packet-dtls.c
index 1302c03e1e..dcb5fbd76e 100644
--- a/epan/dissectors/packet-dtls.c
+++ b/epan/dissectors/packet-dtls.c
@@ -1556,7 +1556,7 @@ static gboolean
dtlsdecrypt_uat_fld_protocol_chk_cb(void* r _U_, const char* p, guint len _U_, const void* u1 _U_, const void* u2 _U_, char** err)
{
if (!p || strlen(p) == 0u) {
- *err = g_strdup_printf("No protocol given.");
+ *err = g_strdup("No protocol given.");
return FALSE;
}
diff --git a/epan/dissectors/packet-h450-ros.c b/epan/dissectors/packet-h450-ros.c
index 1ff1f3be86..c17dbacb55 100644
--- a/epan/dissectors/packet-h450-ros.c
+++ b/epan/dissectors/packet-h450-ros.c
@@ -235,7 +235,7 @@ dissect_h450_ros_Invoke(tvbuff_t *tvb _U_, int offset _U_, asn1_ctx_t *actx _U_,
else if (actx->rose_ctx->d.code == 1)
descr = wmem_strdup_printf(wmem_packet_scope(), "INV: %s", actx->rose_ctx->d.code_global);
} else {
- descr = wmem_strdup_printf(wmem_packet_scope(), "INV:");
+ descr = wmem_strdup(wmem_packet_scope(), "INV:");
}
if (actx->rose_ctx->apdu_depth >= 0)
@@ -318,7 +318,7 @@ dissect_h450_ros_ReturnResult(tvbuff_t *tvb _U_, int offset _U_, asn1_ctx_t *act
else if (actx->rose_ctx->d.code == 1)
descr = wmem_strdup_printf(wmem_packet_scope(), "RES: %s", actx->rose_ctx->d.code_global);
} else {
- descr = wmem_strdup_printf(wmem_packet_scope(), "RES:");
+ descr = wmem_strdup(wmem_packet_scope(), "RES:");
}
if (actx->rose_ctx->apdu_depth >= 0)
@@ -388,7 +388,7 @@ dissect_h450_ros_ReturnError(tvbuff_t *tvb _U_, int offset _U_, asn1_ctx_t *actx
else if (actx->rose_ctx->d.code == 1)
descr = wmem_strdup_printf(wmem_packet_scope(), "ERR: %s", actx->rose_ctx->d.code_global);
} else {
- descr = wmem_strdup_printf(wmem_packet_scope(), "ERR:");
+ descr = wmem_strdup(wmem_packet_scope(), "ERR:");
}
if (actx->rose_ctx->apdu_depth >= 0)
diff --git a/epan/dissectors/packet-isakmp.c b/epan/dissectors/packet-isakmp.c
index d3a0213b45..a0e278e5d9 100644
--- a/epan/dissectors/packet-isakmp.c
+++ b/epan/dissectors/packet-isakmp.c
@@ -5309,7 +5309,7 @@ static gboolean ikev1_uat_data_update_cb(void* p, char** err) {
}
if (ud->key_len == 0) {
- *err = g_strdup_printf("Must have Encryption key.");
+ *err = g_strdup("Must have Encryption key.");
return FALSE;
}
diff --git a/epan/dissectors/packet-k12.c b/epan/dissectors/packet-k12.c
index 1b9fdf8025..154cef36bd 100644
--- a/epan/dissectors/packet-k12.c
+++ b/epan/dissectors/packet-k12.c
@@ -375,7 +375,7 @@ protos_chk_cb(void* r _U_, const char* p, guint len, const void* u1 _U_, const v
g_strstrip(protos[num_protos]);
if (!num_protos) {
- *err = g_strdup_printf("No protocols given");
+ *err = g_strdup("No protocols given");
wmem_free(NULL, line);
wmem_free(NULL, protos);
return FALSE;
diff --git a/epan/dissectors/packet-lbmpdmtcp.c b/epan/dissectors/packet-lbmpdmtcp.c
index e7101ef711..006b067abd 100644
--- a/epan/dissectors/packet-lbmpdmtcp.c
+++ b/epan/dissectors/packet-lbmpdmtcp.c
@@ -159,7 +159,7 @@ static gboolean lbmpdm_tcp_tag_update_cb(void * record, char * * error_string)
if (tag->name == NULL)
{
- *error_string = g_strdup_printf("Tag name can't be empty");
+ *error_string = g_strdup("Tag name can't be empty");
return FALSE;
}
else
@@ -167,7 +167,7 @@ static gboolean lbmpdm_tcp_tag_update_cb(void * record, char * * error_string)
g_strstrip(tag->name);
if (tag->name[0] == 0)
{
- *error_string = g_strdup_printf("Tag name can't be empty");
+ *error_string = g_strdup("Tag name can't be empty");
return FALSE;
}
}
diff --git a/epan/dissectors/packet-lbmr.c b/epan/dissectors/packet-lbmr.c
index e42d0bb43e..f112f7e2b6 100644
--- a/epan/dissectors/packet-lbmr.c
+++ b/epan/dissectors/packet-lbmr.c
@@ -1967,7 +1967,7 @@ static gboolean lbmr_tag_update_cb(void * record, char * * error_string)
if (tag->name == NULL)
{
- *error_string = g_strdup_printf("Tag name can't be empty");
+ *error_string = g_strdup("Tag name can't be empty");
return FALSE;
}
else
@@ -1975,7 +1975,7 @@ static gboolean lbmr_tag_update_cb(void * record, char * * error_string)
g_strstrip(tag->name);
if (tag->name[0] == 0)
{
- *error_string = g_strdup_printf("Tag name can't be empty");
+ *error_string = g_strdup("Tag name can't be empty");
return FALSE;
}
}
diff --git a/epan/dissectors/packet-lbtrm.c b/epan/dissectors/packet-lbtrm.c
index f3e1e6842d..16bbbf64e8 100644
--- a/epan/dissectors/packet-lbtrm.c
+++ b/epan/dissectors/packet-lbtrm.c
@@ -652,7 +652,7 @@ static gboolean lbtrm_tag_update_cb(void * record, char * * error_string)
if (tag->name == NULL)
{
- *error_string = g_strdup_printf("Tag name can't be empty");
+ *error_string = g_strdup("Tag name can't be empty");
return FALSE;
}
else
@@ -660,7 +660,7 @@ static gboolean lbtrm_tag_update_cb(void * record, char * * error_string)
g_strstrip(tag->name);
if (tag->name[0] == 0)
{
- *error_string = g_strdup_printf("Tag name can't be empty");
+ *error_string = g_strdup("Tag name can't be empty");
return FALSE;
}
}
diff --git a/epan/dissectors/packet-lbtru.c b/epan/dissectors/packet-lbtru.c
index 8b79178a43..91fc8bea82 100644
--- a/epan/dissectors/packet-lbtru.c
+++ b/epan/dissectors/packet-lbtru.c
@@ -701,7 +701,7 @@ static gboolean lbtru_tag_update_cb(void * record, char * * error_string)
if (tag->name == NULL)
{
- *error_string = g_strdup_printf("Tag name can't be empty");
+ *error_string = g_strdup("Tag name can't be empty");
return FALSE;
}
else
@@ -709,7 +709,7 @@ static gboolean lbtru_tag_update_cb(void * record, char * * error_string)
g_strstrip(tag->name);
if (tag->name[0] == 0)
{
- *error_string = g_strdup_printf("Tag name can't be empty");
+ *error_string = g_strdup("Tag name can't be empty");
return FALSE;
}
}
diff --git a/epan/dissectors/packet-lbttcp.c b/epan/dissectors/packet-lbttcp.c
index 615b760255..d91f19e85d 100644
--- a/epan/dissectors/packet-lbttcp.c
+++ b/epan/dissectors/packet-lbttcp.c
@@ -332,7 +332,7 @@ static gboolean lbttcp_tag_update_cb(void * record, char * * error_string)
if (tag->name == NULL)
{
- *error_string = g_strdup_printf("Tag name can't be empty");
+ *error_string = g_strdup("Tag name can't be empty");
return FALSE;
}
else
@@ -340,7 +340,7 @@ static gboolean lbttcp_tag_update_cb(void * record, char * * error_string)
g_strstrip(tag->name);
if (tag->name[0] == 0)
{
- *error_string = g_strdup_printf("Tag name can't be empty");
+ *error_string = g_strdup("Tag name can't be empty");
return FALSE;
}
}
diff --git a/epan/dissectors/packet-ldap.c b/epan/dissectors/packet-ldap.c
index 95ba5258bf..fae5f52787 100644
--- a/epan/dissectors/packet-ldap.c
+++ b/epan/dissectors/packet-ldap.c
@@ -1900,7 +1900,7 @@ dissect_ldap_T_and(gboolean implicit_tag _U_, tvbuff_t *tvb _U_, int offset _U_,
if(and_filter_string) {
proto_item_append_text(it, "%s", and_filter_string);
- Filter_string=wmem_strdup_printf(wmem_packet_scope(), "%s",and_filter_string);
+ Filter_string=wmem_strdup(wmem_packet_scope(), and_filter_string);
}
and_filter_string=old_and_filter_string;
@@ -1948,7 +1948,7 @@ dissect_ldap_T_or(gboolean implicit_tag _U_, tvbuff_t *tvb _U_, int offset _U_,
if(or_filter_string) {
proto_item_append_text(it, "%s", or_filter_string);
- Filter_string=wmem_strdup_printf(wmem_packet_scope(), "%s",or_filter_string);
+ Filter_string=wmem_strdup(wmem_packet_scope(), or_filter_string);
}
or_filter_string=old_or_filter_string;
diff --git a/epan/dissectors/packet-q932-ros.c b/epan/dissectors/packet-q932-ros.c
index 6f975104ea..d2fc2a2d92 100644
--- a/epan/dissectors/packet-q932-ros.c
+++ b/epan/dissectors/packet-q932-ros.c
@@ -296,7 +296,7 @@ dissect_q932_ros_Invoke(gboolean implicit_tag _U_, tvbuff_t *tvb _U_, int offset
else if (actx->rose_ctx->d.code == 1)
descr = wmem_strdup_printf(wmem_packet_scope(), "INV: %s", actx->rose_ctx->d.code_global);
} else {
- descr = wmem_strdup_printf(wmem_packet_scope(), "INV:");
+ descr = wmem_strdup(wmem_packet_scope(), "INV:");
}
if (actx->rose_ctx->apdu_depth >= 0)
@@ -389,7 +389,7 @@ dissect_q932_ros_ReturnResult(gboolean implicit_tag _U_, tvbuff_t *tvb _U_, int
else if (actx->rose_ctx->d.code == 1)
descr = wmem_strdup_printf(wmem_packet_scope(), "RES: %s", actx->rose_ctx->d.code_global);
} else {
- descr = wmem_strdup_printf(wmem_packet_scope(), "RES:");
+ descr = wmem_strdup(wmem_packet_scope(), "RES:");
}
if (actx->rose_ctx->apdu_depth >= 0)
@@ -469,7 +469,7 @@ dissect_q932_ros_ReturnError(gboolean implicit_tag _U_, tvbuff_t *tvb _U_, int o
else if (actx->rose_ctx->d.code == 1)
descr = wmem_strdup_printf(wmem_packet_scope(), "ERR: %s", actx->rose_ctx->d.code_global);
} else {
- descr = wmem_strdup_printf(wmem_packet_scope(), "ERR:");
+ descr = wmem_strdup(wmem_packet_scope(), "ERR:");
}
if (actx->rose_ctx->apdu_depth >= 0)
diff --git a/epan/dissectors/packet-rpc.c b/epan/dissectors/packet-rpc.c
index b7c4b2193a..5dc7168844 100644
--- a/epan/dissectors/packet-rpc.c
+++ b/epan/dissectors/packet-rpc.c
@@ -435,7 +435,7 @@ rpcstat_param(register_srt_t* srt, const char* opt_arg, char** err)
}
else
{
- *err = g_strdup_printf("<program>,<version>[,<filter>]");
+ *err = g_strdup("<program>,<version>[,<filter>]");
}
return pos;
diff --git a/epan/dissectors/packet-scsi.c b/epan/dissectors/packet-scsi.c
index 3851c5da93..7b5a526db4 100644
--- a/epan/dissectors/packet-scsi.c
+++ b/epan/dissectors/packet-scsi.c
@@ -1062,7 +1062,7 @@ scsistat_param(register_srt_t* srt, const char* opt_arg, char** err)
}
else
{
- *err = g_strdup_printf("<cmdset>[,<filter>]");
+ *err = g_strdup("<cmdset>[,<filter>]");
}
return pos;
diff --git a/epan/dissectors/packet-ssl-utils.c b/epan/dissectors/packet-ssl-utils.c
index 2ed8f89a60..4f7b000fe2 100644
--- a/epan/dissectors/packet-ssl-utils.c
+++ b/epan/dissectors/packet-ssl-utils.c
@@ -4916,7 +4916,7 @@ gboolean
ssldecrypt_uat_fld_ip_chk_cb(void* r _U_, const char* p, guint len _U_, const void* u1 _U_, const void* u2 _U_, char** err)
{
if (!p || strlen(p) == 0u) {
- *err = g_strdup_printf("No IP address given.");
+ *err = g_strdup("No IP address given.");
return FALSE;
}
@@ -4928,14 +4928,14 @@ gboolean
ssldecrypt_uat_fld_port_chk_cb(void* r _U_, const char* p, guint len _U_, const void* u1 _U_, const void* u2 _U_, char** err)
{
if (!p || strlen(p) == 0u) {
- *err = g_strdup_printf("No Port given.");
+ *err = g_strdup("No Port given.");
return FALSE;
}
if (strcmp(p, "start_tls") != 0){
const gint i = atoi(p);
if (i < 0 || i > 65535) {
- *err = g_strdup_printf("Invalid port given.");
+ *err = g_strdup("Invalid port given.");
return FALSE;
}
}
@@ -4950,7 +4950,7 @@ ssldecrypt_uat_fld_fileopen_chk_cb(void* r _U_, const char* p, guint len _U_, co
ws_statb64 st;
if (!p || strlen(p) == 0u) {
- *err = g_strdup_printf("No filename given.");
+ *err = g_strdup("No filename given.");
return FALSE;
} else {
if (ws_stat64(p, &st) != 0) {
diff --git a/epan/dissectors/packet-ssl.c b/epan/dissectors/packet-ssl.c
index 6958ab74bd..6bab404798 100644
--- a/epan/dissectors/packet-ssl.c
+++ b/epan/dissectors/packet-ssl.c
@@ -3649,7 +3649,7 @@ static gboolean
ssldecrypt_uat_fld_protocol_chk_cb(void* r _U_, const char* p, guint len _U_, const void* u1 _U_, const void* u2 _U_, char** err)
{
if (!p || strlen(p) == 0u) {
- *err = g_strdup_printf("No protocol given.");
+ *err = g_strdup("No protocol given.");
return FALSE;
}
diff --git a/epan/dissectors/packet-zbee-nwk-gp.c b/epan/dissectors/packet-zbee-nwk-gp.c
index f1d2ce35ce..14c48d213a 100644
--- a/epan/dissectors/packet-zbee-nwk-gp.c
+++ b/epan/dissectors/packet-zbee-nwk-gp.c
@@ -610,7 +610,7 @@ uat_key_record_update_cb(void *r, char **err)
uat_key_record_t *rec = (uat_key_record_t *)r;
if (rec->string == NULL) {
- *err = g_strdup_printf("Key can't be blank.");
+ *err = g_strdup("Key can't be blank.");
return FALSE;
} else {
g_strstrip(rec->string);
@@ -622,7 +622,7 @@ uat_key_record_update_cb(void *r, char **err)
return FALSE;
}
} else {
- *err = g_strdup_printf("Key can't be blank.");
+ *err = g_strdup("Key can't be blank.");
return FALSE;
}
}
diff --git a/epan/dissectors/packet-zbee-nwk.c b/epan/dissectors/packet-zbee-nwk.c
index 3816f08080..25824b75b9 100644
--- a/epan/dissectors/packet-zbee-nwk.c
+++ b/epan/dissectors/packet-zbee-nwk.c
@@ -524,7 +524,7 @@ dissect_zbee_nwk_full(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void
if ( (packet.dst == ZBEE_BCAST_ALL)
|| (packet.dst == ZBEE_BCAST_ACTIVE)
|| (packet.dst == ZBEE_BCAST_ROUTERS)){
- dst_addr = wmem_strdup_printf(pinfo->pool, "Broadcast");
+ dst_addr = wmem_strdup(pinfo->pool, "Broadcast");
}
else {
dst_addr = wmem_strdup_printf(pinfo->pool, "0x%04x", packet.dst);
@@ -549,7 +549,7 @@ dissect_zbee_nwk_full(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void
|| (packet.src == ZBEE_BCAST_ACTIVE)
|| (packet.src == ZBEE_BCAST_ROUTERS)){
/* Source Broadcast doesn't make much sense. */
- src_addr = wmem_strdup_printf(pinfo->pool, "Unexpected Source Broadcast");
+ src_addr = wmem_strdup(pinfo->pool, "Unexpected Source Broadcast");
unicast_src = FALSE;
}
else {