aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/mm-broadband-modem-mbim.c13
-rw-r--r--src/mm-charsets.c49
-rw-r--r--src/mm-charsets.h4
-rw-r--r--src/mm-sms-part-3gpp.c17
-rw-r--r--src/tests/test-charsets.c15
5 files changed, 45 insertions, 53 deletions
diff --git a/src/mm-broadband-modem-mbim.c b/src/mm-broadband-modem-mbim.c
index ba37cdb7..513017a8 100644
--- a/src/mm-broadband-modem-mbim.c
+++ b/src/mm-broadband-modem-mbim.c
@@ -4771,20 +4771,17 @@ ussd_encode (const gchar *command,
g_autoptr(GByteArray) array = NULL;
if (mm_charset_can_convert_to (command, MM_MODEM_CHARSET_GSM)) {
- guint8 *gsm;
- guint8 *packed;
- guint32 len = 0;
- guint32 packed_len = 0;
+ g_autoptr(GByteArray) gsm = NULL;
+ guint8 *packed;
+ guint32 packed_len = 0;
*scheme = MM_MODEM_GSM_USSD_SCHEME_7BIT;
- gsm = mm_charset_utf8_to_unpacked_gsm (command, FALSE, &len, error);
+ gsm = mm_modem_charset_bytearray_from_utf8 (command, MM_MODEM_CHARSET_GSM, FALSE, error);
if (!gsm) {
g_prefix_error (error, "Failed to encode USSD command in GSM7 charset: ");
return NULL;
}
- packed = mm_charset_gsm_pack (gsm, len, 0, &packed_len);
- g_free (gsm);
-
+ packed = mm_charset_gsm_pack (gsm->data, gsm->len, 0, &packed_len);
array = g_byte_array_new_take (packed, packed_len);
} else {
g_autoptr(GError) inner_error = NULL;
diff --git a/src/mm-charsets.c b/src/mm-charsets.c
index caf2abb5..ec5bb404 100644
--- a/src/mm-charsets.c
+++ b/src/mm-charsets.c
@@ -444,11 +444,11 @@ mm_charset_gsm_unpacked_to_utf8 (const guint8 *gsm,
return g_byte_array_free (g_steal_pointer (&utf8), FALSE);
}
-guint8 *
-mm_charset_utf8_to_unpacked_gsm (const gchar *utf8,
- gboolean translit,
- guint32 *out_len,
- GError **error)
+static guint8 *
+charset_utf8_to_unpacked_gsm (const gchar *utf8,
+ gboolean translit,
+ guint32 *out_len,
+ GError **error)
{
g_autoptr(GByteArray) gsm = NULL;
const gchar *c;
@@ -903,7 +903,8 @@ mm_utf8_take_and_convert_to_charset (gchar *str,
break;
case MM_MODEM_CHARSET_GSM:
- encoded = (gchar *) mm_charset_utf8_to_unpacked_gsm (str, FALSE, NULL, NULL);
+ /* This is WRONG! GSM may have embedded NULs (character @)! */
+ encoded = mm_modem_charset_str_from_utf8 (str, MM_MODEM_CHARSET_GSM, FALSE, NULL);
g_free (str);
break;
@@ -968,19 +969,16 @@ mm_utf8_take_and_convert_to_charset (gchar *str,
/* Main conversion functions */
static guint8 *
-charset_iconv_from_utf8 (const gchar *utf8,
- MMModemCharset charset,
- gboolean translit,
- guint *out_size,
- GError **error)
+charset_iconv_from_utf8 (const gchar *utf8,
+ const CharsetSettings *settings,
+ gboolean translit,
+ guint *out_size,
+ GError **error)
{
g_autoptr(GError) inner_error = NULL;
- const CharsetSettings *settings;
gsize bytes_written = 0;
g_autofree guint8 *encoded = NULL;
- settings = lookup_charset_settings (charset);
-
encoded = (guint8 *) g_convert (utf8, -1,
settings->iconv_name, "UTF-8",
NULL, &bytes_written, &inner_error);
@@ -1015,8 +1013,11 @@ mm_modem_charset_bytearray_from_utf8 (const gchar *utf8,
gboolean translit,
GError **error)
{
- guint8 *encoded = NULL;
- guint encoded_size = 0;
+ const CharsetSettings *settings;
+ guint8 *encoded = NULL;
+ guint encoded_size = 0;
+
+ settings = lookup_charset_settings (charset);
if (charset == MM_MODEM_CHARSET_UNKNOWN) {
g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_INVALID_ARGS,
@@ -1026,7 +1027,7 @@ mm_modem_charset_bytearray_from_utf8 (const gchar *utf8,
switch (charset) {
case MM_MODEM_CHARSET_GSM:
- encoded = mm_charset_utf8_to_unpacked_gsm (utf8, translit, &encoded_size, error);
+ encoded = charset_utf8_to_unpacked_gsm (utf8, translit, &encoded_size, error);
break;
case MM_MODEM_CHARSET_IRA:
case MM_MODEM_CHARSET_8859_1:
@@ -1035,7 +1036,7 @@ mm_modem_charset_bytearray_from_utf8 (const gchar *utf8,
case MM_MODEM_CHARSET_PCCP437:
case MM_MODEM_CHARSET_PCDN:
case MM_MODEM_CHARSET_UTF16:
- encoded = charset_iconv_from_utf8 (utf8, charset, translit, &encoded_size, error);
+ encoded = charset_iconv_from_utf8 (utf8, settings, translit, &encoded_size, error);
break;
case MM_MODEM_CHARSET_UNKNOWN:
default:
@@ -1095,15 +1096,12 @@ mm_modem_charset_str_from_utf8 (const gchar *utf8,
static gchar *
charset_iconv_to_utf8 (const guint8 *data,
guint32 len,
- MMModemCharset charset,
+ const CharsetSettings *settings,
gboolean translit,
GError **error)
{
- g_autoptr(GError) inner_error = NULL;
- g_autofree gchar *utf8 = NULL;
- const CharsetSettings *settings;
-
- settings = lookup_charset_settings (charset);
+ g_autoptr(GError) inner_error = NULL;
+ g_autofree gchar *utf8 = NULL;
utf8 = g_convert ((const gchar *) data, len,
"UTF-8",
@@ -1144,6 +1142,7 @@ mm_modem_charset_bytearray_to_utf8 (GByteArray *bytearray,
}
settings = lookup_charset_settings (charset);
+
switch (charset) {
case MM_MODEM_CHARSET_GSM:
utf8 = (gchar *) mm_charset_gsm_unpacked_to_utf8 (bytearray->data,
@@ -1160,7 +1159,7 @@ mm_modem_charset_bytearray_to_utf8 (GByteArray *bytearray,
case MM_MODEM_CHARSET_UTF16:
utf8 = charset_iconv_to_utf8 (bytearray->data,
bytearray->len,
- charset,
+ settings,
translit,
error);
break;
diff --git a/src/mm-charsets.h b/src/mm-charsets.h
index 4d032f38..1f635aa1 100644
--- a/src/mm-charsets.h
+++ b/src/mm-charsets.h
@@ -57,10 +57,6 @@ gchar *mm_modem_charset_byte_array_to_utf8 (GByteArray *array,
gchar *mm_modem_charset_hex_to_utf8 (const gchar *src,
MMModemCharset charset);
-guint8 *mm_charset_utf8_to_unpacked_gsm (const gchar *utf8,
- gboolean translit,
- guint32 *out_len,
- GError **error);
guint8 *mm_charset_gsm_unpacked_to_utf8 (const guint8 *gsm,
guint32 len,
gboolean translit,
diff --git a/src/mm-sms-part-3gpp.c b/src/mm-sms-part-3gpp.c
index 7547d029..4bd22626 100644
--- a/src/mm-sms-part-3gpp.c
+++ b/src/mm-sms-part-3gpp.c
@@ -984,15 +984,15 @@ mm_sms_part_3gpp_get_submit_pdu (MMSmsPart *part,
}
if (encoding == MM_SMS_ENCODING_GSM7) {
- guint8 *unpacked, *packed;
- guint32 unlen = 0, packlen = 0;
+ g_autoptr(GByteArray) unpacked = NULL;
+ g_autofree guint8 *packed = NULL;
+ guint32 packlen = 0;
- unpacked = mm_charset_utf8_to_unpacked_gsm (mm_sms_part_get_text (part), FALSE, &unlen, error);
+ unpacked = mm_modem_charset_bytearray_from_utf8 (mm_sms_part_get_text (part), MM_MODEM_CHARSET_GSM, FALSE, error);
if (!unpacked)
goto error;
- if (unlen == 0) {
- g_free (unpacked);
+ if (unpacked->len == 0) {
g_set_error_literal (error,
MM_MESSAGE_ERROR,
MM_MESSAGE_ERROR_INVALID_PDU_PARAMETER,
@@ -1003,15 +1003,13 @@ mm_sms_part_3gpp_get_submit_pdu (MMSmsPart *part,
/* Set real data length, in septets
* If we had UDH, add 7 septets
*/
- *udl_ptr = mm_sms_part_get_concat_sequence (part) ? (7 + unlen) : unlen;
+ *udl_ptr = mm_sms_part_get_concat_sequence (part) ? (7 + unpacked->len) : unpacked->len;
mm_obj_dbg (log_object, " user data length is %u septets (%s UDH)",
*udl_ptr,
mm_sms_part_get_concat_sequence (part) ? "with" : "without");
- packed = mm_charset_gsm_pack (unpacked, unlen, shift, &packlen);
- g_free (unpacked);
+ packed = mm_charset_gsm_pack (unpacked->data, unpacked->len, shift, &packlen);
if (!packed || packlen == 0) {
- g_free (packed);
g_set_error_literal (error,
MM_MESSAGE_ERROR,
MM_MESSAGE_ERROR_INVALID_PDU_PARAMETER,
@@ -1020,7 +1018,6 @@ mm_sms_part_3gpp_get_submit_pdu (MMSmsPart *part,
}
memcpy (&pdu[offset], packed, packlen);
- g_free (packed);
offset += packlen;
} else if (encoding == MM_SMS_ENCODING_UCS2) {
g_autoptr(GByteArray) array = NULL;
diff --git a/src/tests/test-charsets.c b/src/tests/test-charsets.c
index c2e08591..4df104ed 100644
--- a/src/tests/test-charsets.c
+++ b/src/tests/test-charsets.c
@@ -23,25 +23,23 @@
static void
common_test_gsm7 (const gchar *in_utf8)
{
- guint32 unpacked_gsm_len = 0;
guint32 packed_gsm_len = 0;
guint32 unpacked_gsm_len_2 = 0;
- g_autofree guint8 *unpacked_gsm = NULL;
+ g_autoptr(GByteArray) unpacked_gsm = NULL;
g_autofree guint8 *packed_gsm = NULL;
g_autofree guint8 *unpacked_gsm_2 = NULL;
g_autofree gchar *built_utf8 = NULL;
g_autoptr(GError) error = NULL;
/* Convert to GSM */
- unpacked_gsm = mm_charset_utf8_to_unpacked_gsm (in_utf8, FALSE, &unpacked_gsm_len, &error);
+ unpacked_gsm = mm_modem_charset_bytearray_from_utf8 (in_utf8, MM_MODEM_CHARSET_GSM, FALSE, &error);
g_assert_nonnull (unpacked_gsm);
g_assert_no_error (error);
- g_assert_cmpuint (unpacked_gsm_len, >, 0);
/* Pack */
- packed_gsm = mm_charset_gsm_pack (unpacked_gsm, unpacked_gsm_len, 0, &packed_gsm_len);
+ packed_gsm = mm_charset_gsm_pack (unpacked_gsm->data, unpacked_gsm->len, 0, &packed_gsm_len);
g_assert_nonnull (packed_gsm);
- g_assert_cmpuint (packed_gsm_len, <=, unpacked_gsm_len);
+ g_assert_cmpuint (packed_gsm_len, <=, unpacked_gsm->len);
#if 0
{
@@ -356,6 +354,10 @@ test_take_convert_ucs2_bad_ascii2 (void)
static void
test_take_convert_gsm_utf8 (void)
{
+ /* NOTE: this is wrong, charset GSM may contain embedded NULs so we cannot convert
+ * from a plain UTF-8 string to a NUL-terminated string in GSM, as there is no
+ * such, thing. */
+#if 0
gchar *src, *converted, *utf8;
src = g_strdup ("T-Mobile");
@@ -364,6 +366,7 @@ test_take_convert_gsm_utf8 (void)
utf8 = mm_utf8_take_and_convert_to_charset (converted, MM_MODEM_CHARSET_GSM);
g_assert_cmpstr (utf8, ==, "T-Mobile");
g_free (utf8);
+#endif
}
struct charset_can_convert_to_test_s {
generated by cgit v1.2.3-70-g09d2 (git 2.46.0) at 2025-06-07 20:51:16 +0000