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

gsm: handle PINs better during modem enable and simple state machine

Browse Source 
First, short-circuit the Enable process if the device requires a PIN
or PUK since for many devices the enable is going to fail anyway
until the PIN is sent.

Second, send the PIN first during the simple state machine for the
same reason; we need the device unlocked before we want to try
to enable it.  This also reworks the simple state machine to be a
bit clearer and make each state step correspond to the action it's
actually doing instead of being off-by-one visually (but not logically).
This commit is contained in:
Dan Williams
2010-03-04 19:30:54 -08:00
parent 021ca1244e
commit 407abc65c6
1 changed files with 103 additions and 64 deletions
Download Patch File Download Diff File Expand all files Collapse all files

137
src/mm-generic-gsm.c
View File

@@ -141,15 +141,7 @@ typedef struct {
guint code;
} CPinResult;
static void
pin_check_done (MMSerialPort *port,
GString *response,
GError *error,
gpointer user_data)
{
MMCallbackInfo *info = (MMCallbackInfo *) user_data;
gboolean parsed = FALSE;
static CPinResult results[] = {
static CPinResult unlock_results[] = {
{ "SIM PIN", "sim-pin", MM_MOBILE_ERROR_SIM_PIN },
{ "SIM PUK", "sim-puk", MM_MOBILE_ERROR_SIM_PUK },
{ "PH-SIM PIN", "ph-sim-pin", MM_MOBILE_ERROR_PH_SIM_PIN },
@@ -168,6 +160,35 @@ pin_check_done (MMSerialPort *port,
{ NULL, NULL, MM_MOBILE_ERROR_PHONE_FAILURE },
};
static GError *
error_for_unlock_required (const char *unlock)
{
CPinResult *iter = &unlock_results[0];
if (!unlock || !strlen (unlock))
return NULL;
/* Translate the error */
while (iter->result) {
if (!strcmp (iter->normalized, unlock))
return mm_mobile_error_for_code (iter->code);
iter++;
}
return g_error_new (MM_MOBILE_ERROR,
MM_MOBILE_ERROR_UNKNOWN,
"Unknown unlock request '%s'", unlock);
}
static void
pin_check_done (MMSerialPort *port,
GString *response,
GError *error,
gpointer user_data)
{
MMCallbackInfo *info = (MMCallbackInfo *) user_data;
gboolean parsed = FALSE;
if (error)
info->error = g_error_copy (error);
else if (response && strstr (response->str, "+CPIN: ")) {
@@ -177,7 +198,7 @@ pin_check_done (MMSerialPort *port,
mm_modem_base_set_unlock_required (MM_MODEM_BASE (info->modem), NULL);
parsed = TRUE;
} else {
CPinResult *iter = &results[0];
CPinResult *iter = &unlock_results[0];
/* Translate the error */
while (iter->result) {
@@ -563,6 +584,18 @@ enable (MMModem *modem,
{
MMGenericGsmPrivate *priv = MM_GENERIC_GSM_GET_PRIVATE (modem);
GError *error = NULL;
const char *unlock;
/* If the device needs a PIN, deal with that now */
unlock = mm_modem_base_get_unlock_required (MM_MODEM_BASE (modem));
if (unlock) {
MMCallbackInfo *info;
info = mm_callback_info_new (modem, callback, user_data);
info->error = error_for_unlock_required (unlock);
mm_callback_info_schedule (info);
return;
}
/* First, reset the previously used CID */
mm_generic_gsm_set_cid (MM_GENERIC_GSM (modem), 0);
@@ -1969,9 +2002,8 @@ mm_generic_gsm_get_port (MMGenericGsm *modem,
/* MMModemSimple interface */
typedef enum {
SIMPLE_STATE_BEGIN = 0,
SIMPLE_STATE_CHECK_PIN = 0,
SIMPLE_STATE_ENABLE,
SIMPLE_STATE_CHECK_PIN,
SIMPLE_STATE_REGISTER,
SIMPLE_STATE_SET_APN,
SIMPLE_STATE_CONNECT,
@@ -2002,68 +2034,75 @@ static void
simple_state_machine (MMModem *modem, GError *error, gpointer user_data)
{
MMCallbackInfo *info = (MMCallbackInfo *) user_data;
const char *str;
const char *str, *unlock = NULL;
SimpleState state = GPOINTER_TO_UINT (mm_callback_info_get_data (info, "simple-connect-state"));
gboolean need_pin = FALSE;
SimpleState next_state = state;
gboolean done = FALSE;
if (error) {
if (g_error_matches (error, MM_MOBILE_ERROR, MM_MOBILE_ERROR_SIM_PIN)) {
need_pin = TRUE;
state = SIMPLE_STATE_CHECK_PIN;
} else {
info->error = g_error_copy (error);
goto out;
}
}
switch (state) {
case SIMPLE_STATE_BEGIN:
state = SIMPLE_STATE_ENABLE;
case SIMPLE_STATE_CHECK_PIN:
next_state = SIMPLE_STATE_ENABLE;
/* If we need a PIN, send it now */
unlock = mm_modem_base_get_unlock_required (MM_MODEM_BASE (modem));
if (unlock) {
gboolean success = FALSE;
if (!strcmp (unlock, "sim-pin")) {
str = simple_get_string_property (info, "pin", &info->error);
if (str) {
mm_modem_gsm_card_send_pin (MM_MODEM_GSM_CARD (modem), str, simple_state_machine, info);
success = TRUE;
}
}
if (!success && !info->error)
info->error = error_for_unlock_required (unlock);
break;
}
/* Fall through if no PIN required */
case SIMPLE_STATE_ENABLE:
next_state = SIMPLE_STATE_REGISTER;
mm_modem_enable (modem, simple_state_machine, info);
break;
case SIMPLE_STATE_ENABLE:
state = SIMPLE_STATE_CHECK_PIN;
mm_generic_gsm_check_pin (MM_GENERIC_GSM (modem), simple_state_machine, info);
break;
case SIMPLE_STATE_CHECK_PIN:
if (need_pin) {
str = simple_get_string_property (info, "pin", &info->error);
if (str)
mm_modem_gsm_card_send_pin (MM_MODEM_GSM_CARD (modem), str, simple_state_machine, info);
else
info->error = g_error_copy (error);
} else {
str = simple_get_string_property (info, "network_id", &info->error);
state = SIMPLE_STATE_REGISTER;
if (!info->error)
mm_modem_gsm_network_register (MM_MODEM_GSM_NETWORK (modem), str, simple_state_machine, info);
}
break;
case SIMPLE_STATE_REGISTER:
next_state = SIMPLE_STATE_SET_APN;
str = simple_get_string_property (info, "network_id", &info->error);
if (str || info->error) {
if (str)
mm_modem_gsm_network_register (MM_MODEM_GSM_NETWORK (modem), str, simple_state_machine, info);
break;
}
/* Fall through if no explicit network registration is required */
case SIMPLE_STATE_SET_APN:
next_state = SIMPLE_STATE_CONNECT;
str = simple_get_string_property (info, "apn", &info->error);
if (str) {
state = SIMPLE_STATE_SET_APN;
if (str || info->error) {
if (str)
mm_modem_gsm_network_set_apn (MM_MODEM_GSM_NETWORK (modem), str, simple_state_machine, info);
break;
}
/* Fall through */
case SIMPLE_STATE_SET_APN:
/* Fall through if no APN or no 'apn' property error */
case SIMPLE_STATE_CONNECT:
next_state = SIMPLE_STATE_DONE;
str = simple_get_string_property (info, "number", &info->error);
state = SIMPLE_STATE_CONNECT;
if (!info->error)
mm_modem_connect (modem, str, simple_state_machine, info);
break;
case SIMPLE_STATE_CONNECT:
state = SIMPLE_STATE_DONE;
break;
case SIMPLE_STATE_DONE:
done = TRUE;
break;
}
out:
if (info->error || state == SIMPLE_STATE_DONE)
if (info->error || done)
mm_callback_info_schedule (info);
else
mm_callback_info_set_data (info, "simple-connect-state", GUINT_TO_POINTER (state), NULL);
mm_callback_info_set_data (info, "simple-connect-state", GUINT_TO_POINTER (next_state), NULL);
}
static void