SymbX/ModemManager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

libmm-glib, modem-helpers: unref enum/flags GType classes in enum getters

Browse Source
This commit is contained in:
Thomas Haller
2018-10-04 09:43:05 +02:00
committed by Aleksander Morgado
parent a68bbbc1b9
commit d83f50794d
2 changed files with 58 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
libmm-glib/mm-common-helpers.c
+50 -10
View File
@@ -25,6 +25,26 @@
#include "mm-errors-types.h"
#include "mm-common-helpers.h"
static int
_enum_class_return_value (GEnumClass *enum_class, guint i)
{
int value;
value = enum_class->values[i].value;
g_type_class_unref (enum_class);
return value;
}
static int
_flags_class_return_value (GFlagsClass *flags_class, guint i)
{
guint value;
value = flags_class->values[i].value;
g_type_class_unref (flags_class);
return value;
}
gchar *
mm_common_build_capabilities_string (const MMModemCapability *capabilities,
guint n_capabilities)
@@ -795,9 +815,11 @@ mm_common_get_eps_ue_mode_operation_from_string (const gchar *str,
for (i = 0; enum_class->values[i].value_nick; i++) {
if (!g_ascii_strcasecmp (str, enum_class->values[i].value_nick))
return enum_class->values[i].value;
return _enum_class_return_value (enum_class, i);
}
g_type_class_unref (enum_class);
g_set_error (error,
MM_CORE_ERROR,
MM_CORE_ERROR_INVALID_ARGS,
@@ -914,9 +936,11 @@ mm_common_get_rm_protocol_from_string (const gchar *str,
for (i = 0; enum_class->values[i].value_nick; i++) {
if (!g_ascii_strcasecmp (str, enum_class->values[i].value_nick))
return enum_class->values[i].value;
return _enum_class_return_value (enum_class, i);
}
g_type_class_unref (enum_class);
g_set_error (error,
MM_CORE_ERROR,
MM_CORE_ERROR_INVALID_ARGS,
@@ -936,9 +960,11 @@ mm_common_get_ip_type_from_string (const gchar *str,
for (i = 0; flags_class->values[i].value_nick; i++) {
if (!g_ascii_strcasecmp (str, flags_class->values[i].value_nick))
return flags_class->values[i].value;
return _flags_class_return_value (flags_class, i);
}
g_type_class_unref (flags_class);
g_set_error (error,
MM_CORE_ERROR,
MM_CORE_ERROR_INVALID_ARGS,
@@ -1008,9 +1034,11 @@ mm_common_get_sms_storage_from_string (const gchar *str,
for (i = 0; enum_class->values[i].value_nick; i++) {
if (!g_ascii_strcasecmp (str, enum_class->values[i].value_nick))
return enum_class->values[i].value;
return _enum_class_return_value (enum_class, i);
}
g_type_class_unref (enum_class);
g_set_error (error,
MM_CORE_ERROR,
MM_CORE_ERROR_INVALID_ARGS,
@@ -1030,9 +1058,11 @@ mm_common_get_sms_cdma_teleservice_id_from_string (const gchar *str,
for (i = 0; enum_class->values[i].value_nick; i++) {
if (!g_ascii_strcasecmp (str, enum_class->values[i].value_nick))
return enum_class->values[i].value;
return _enum_class_return_value (enum_class, i);
}
g_type_class_unref (enum_class);
g_set_error (error,
MM_CORE_ERROR,
MM_CORE_ERROR_INVALID_ARGS,
@@ -1052,9 +1082,11 @@ mm_common_get_sms_cdma_service_category_from_string (const gchar *str,
for (i = 0; enum_class->values[i].value_nick; i++) {
if (!g_ascii_strcasecmp (str, enum_class->values[i].value_nick))
return enum_class->values[i].value;
return _enum_class_return_value (enum_class, i);
}
g_type_class_unref (enum_class);
g_set_error (error,
MM_CORE_ERROR,
MM_CORE_ERROR_INVALID_ARGS,
@@ -1074,9 +1106,11 @@ mm_common_get_call_direction_from_string (const gchar *str,
for (i = 0; enum_class->values[i].value_nick; i++) {
if (!g_ascii_strcasecmp (str, enum_class->values[i].value_nick))
return enum_class->values[i].value;
return _enum_class_return_value (enum_class, i);
}
g_type_class_unref (enum_class);
g_set_error (error,
MM_CORE_ERROR,
MM_CORE_ERROR_INVALID_ARGS,
@@ -1096,9 +1130,11 @@ mm_common_get_call_state_from_string (const gchar *str,
for (i = 0; enum_class->values[i].value_nick; i++) {
if (!g_ascii_strcasecmp (str, enum_class->values[i].value_nick))
return enum_class->values[i].value;
return _enum_class_return_value (enum_class, i);
}
g_type_class_unref (enum_class);
g_set_error (error,
MM_CORE_ERROR,
MM_CORE_ERROR_INVALID_ARGS,
@@ -1118,9 +1154,11 @@ mm_common_get_call_state_reason_from_string (const gchar *str,
for (i = 0; enum_class->values[i].value_nick; i++) {
if (!g_ascii_strcasecmp (str, enum_class->values[i].value_nick))
return enum_class->values[i].value;
return _enum_class_return_value (enum_class, i);
}
g_type_class_unref (enum_class);
g_set_error (error,
MM_CORE_ERROR,
MM_CORE_ERROR_INVALID_ARGS,
@@ -1190,9 +1228,11 @@ mm_common_get_oma_session_type_from_string (const gchar *str,
for (i = 0; enum_class->values[i].value_nick; i++) {
if (!g_ascii_strcasecmp (str, enum_class->values[i].value_nick))
return enum_class->values[i].value;
return _enum_class_return_value (enum_class, i);
}
g_type_class_unref (enum_class);
g_set_error (error,
MM_CORE_ERROR,
MM_CORE_ERROR_INVALID_ARGS,
src/mm-modem-helpers.c
+8 -2
View File
@@ -652,15 +652,21 @@ mm_flow_control_from_string (const gchar *str,
GError **error)
{
GFlagsClass *flags_class;
guint value;
guint i;
flags_class = G_FLAGS_CLASS (g_type_class_ref (MM_TYPE_FLOW_CONTROL));
for (i = 0; flags_class->values[i].value_nick; i++) {
if (!g_ascii_strcasecmp (str, flags_class->values[i].value_nick))
return flags_class->values[i].value;
if (!g_ascii_strcasecmp (str, flags_class->values[i].value_nick)) {
value = flags_class->values[i].value;
g_type_class_unref (flags_class);
return value;
}
}
g_type_class_unref (flags_class);
g_set_error (error,
MM_CORE_ERROR,
MM_CORE_ERROR_INVALID_ARGS,