osmith has uploaded this change for review. (
https://gerrit.osmocom.org/c/onomondo-ipa/+/42860?usp=email )
Change subject: esipa: fix printf fmt str for ssize_t
......................................................................
esipa: fix printf fmt str for ssize_t
Fix for:
In file included from …/src/ipa/libipa/esipa.c:14:
…/src/ipa/libipa/esipa.c: In function ‘ipa_esipa_msg_to_eim_enc’:
…/src/ipa/libipa/esipa.h:16:33: error: format ‘%d’ expects argument of type
‘int’, but argument 7 has type ‘ssize_t’ {aka ‘long int’} [-Werror=form
at=]
16 | IPA_LOGP(SESIPA, level, "%s: " fmt, func, ## args)
| ^~~~~~
…/include/onomondo/ipa/log.h:18:53: note: in definition of macro ‘IPA_LOGP’
18 | ipa_logp(subsys, level, __FILE__, __LINE__, fmt, ## args)
| ^~~
…/src/ipa/libipa/esipa.c:129:17: note: in expansion of macro ‘IPA_LOGP_ESIPA’
129 | IPA_LOGP_ESIPA(function_name, LERROR, "cannot encode
eIM request! rc = %d\n", rc.encoded);
| ^~~~~~~~~~~~~~
Change-Id: I5b210ae5acff362cf845f6a37073d2bffe69c76e
---
M src/ipa/libipa/esipa.c
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/onomondo-ipa refs/changes/60/42860/1
diff --git a/src/ipa/libipa/esipa.c b/src/ipa/libipa/esipa.c
index a0ec4de..d88da5d 100644
--- a/src/ipa/libipa/esipa.c
+++ b/src/ipa/libipa/esipa.c
@@ -126,7 +126,7 @@
rc = der_encode(&asn_DEF_EsipaMessageFromIpaToEim, msg_to_eim,
ipa_asn1c_consume_bytes_cb, &buf_encoded);
if (rc.encoded <= 0) {
- IPA_LOGP_ESIPA(function_name, LERROR, "cannot encode eIM
request! rc = %d\n", rc.encoded);
+ IPA_LOGP_ESIPA(function_name, LERROR, "cannot encode eIM
request! rc = %zd\n", rc.encoded);
IPA_FREE(buf_encoded);
return NULL;
}
--
To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/42860?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: onomondo-ipa
Gerrit-Branch: master
Gerrit-Change-Id: I5b210ae5acff362cf845f6a37073d2bffe69c76e
Gerrit-Change-Number: 42860
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <[email protected]>
