diff --git a/libnm-util/nm-setting-gsm.c b/libnm-util/nm-setting-gsm.c index 731215f093..a8917019f7 100644 --- a/libnm-util/nm-setting-gsm.c +++ b/libnm-util/nm-setting-gsm.c @@ -80,7 +80,6 @@ typedef struct { int band; char *pin; - char *puk; } NMSettingGsmPrivate; enum { @@ -180,9 +179,8 @@ nm_setting_gsm_get_pin (NMSettingGsm *setting) const char * nm_setting_gsm_get_puk (NMSettingGsm *setting) { - g_return_val_if_fail (NM_IS_SETTING_GSM (setting), NULL); - - return NM_SETTING_GSM_GET_PRIVATE (setting)->puk; + g_warning ("Tried to set deprecated property " NM_SETTING_GSM_SETTING_NAME "/" NM_SETTING_GSM_PUK); + return NULL; } static gboolean @@ -299,7 +297,6 @@ finalize (GObject *object) g_free (priv->apn); g_free (priv->network_id); g_free (priv->pin); - g_free (priv->puk); G_OBJECT_CLASS (nm_setting_gsm_parent_class)->finalize (object); } @@ -342,9 +339,8 @@ set_property (GObject *object, guint prop_id, priv->pin = g_value_dup_string (value); break; case PROP_PUK: - g_free (priv->puk); - priv->puk = g_value_dup_string (value); - break; + g_warning ("Tried to set deprecated property " NM_SETTING_GSM_SETTING_NAME "/" NM_SETTING_GSM_PUK); + break; default: G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec); break; @@ -383,7 +379,7 @@ get_property (GObject *object, guint prop_id, g_value_set_string (value, nm_setting_gsm_get_pin (setting)); break; case PROP_PUK: - g_value_set_string (value, nm_setting_gsm_get_puk (setting)); + g_warning ("Tried to get deprecated property " NM_SETTING_GSM_SETTING_NAME "/" NM_SETTING_GSM_PUK); break; default: G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec); @@ -476,8 +472,8 @@ nm_setting_gsm_class_init (NMSettingGsmClass *setting_class) g_object_class_install_property (object_class, PROP_PUK, g_param_spec_string (NM_SETTING_GSM_PUK, - "PUK", - "PUK", + "PUK (DEPRECATED and UNUSED)", + "PUK (DEPRECATED and UNUSED)", NULL, G_PARAM_READWRITE | NM_SETTING_PARAM_SERIALIZE | NM_SETTING_PARAM_SECRET)); } diff --git a/libnm-util/nm-setting-gsm.h b/libnm-util/nm-setting-gsm.h index b6b8af32c4..3b503941f4 100644 --- a/libnm-util/nm-setting-gsm.h +++ b/libnm-util/nm-setting-gsm.h @@ -61,6 +61,8 @@ GQuark nm_setting_gsm_error_quark (void); #define NM_SETTING_GSM_NETWORK_TYPE "network-type" #define NM_SETTING_GSM_BAND "band" #define NM_SETTING_GSM_PIN "pin" + +/* DEPRECATED & UNUSED */ #define NM_SETTING_GSM_PUK "puk" enum { diff --git a/src/modem-manager/nm-modem-gsm.c b/src/modem-manager/nm-modem-gsm.c index d1949c5098..8f5b3b2db0 100644 --- a/src/modem-manager/nm-modem-gsm.c +++ b/src/modem-manager/nm-modem-gsm.c @@ -169,8 +169,6 @@ stage1_prepare_done (DBusGProxy *proxy, DBusGProxyCall *call_id, gpointer user_d if (dbus_g_error_has_name (error, MM_MODEM_ERROR_SIM_PIN)) required_secret = NM_SETTING_GSM_PIN; - else if (dbus_g_error_has_name (error, MM_MODEM_ERROR_SIM_PUK)) - required_secret = NM_SETTING_GSM_PUK; else if (dbus_g_error_has_name (error, MM_MODEM_ERROR_SIM_WRONG)) { clear_pin (device); required_secret = NM_SETTING_GSM_PIN;