mirror of git://git.sysmocom.de/ofono
sim: Fix missing return in the atmodem driver
In some circumstances it was possible to call the callback with both failure and success
This commit is contained in:
parent
55d47dcac1
commit
94c104b25f
|
@ -524,17 +524,18 @@ static void at_pin_retries_query(struct ofono_sim *sim,
|
||||||
ofono_sim_pin_retries_cb_t cb, void *data)
|
ofono_sim_pin_retries_cb_t cb, void *data)
|
||||||
{
|
{
|
||||||
struct sim_data *sd = ofono_sim_get_data(sim);
|
struct sim_data *sd = ofono_sim_get_data(sim);
|
||||||
|
struct cb_data *cbd;
|
||||||
int retries[OFONO_SIM_PASSWORD_INVALID];
|
int retries[OFONO_SIM_PASSWORD_INVALID];
|
||||||
int i;
|
int i;
|
||||||
|
|
||||||
DBG("");
|
DBG("");
|
||||||
|
|
||||||
if (sd->vendor == OFONO_VENDOR_HUAWEI) {
|
switch (sd->vendor) {
|
||||||
struct cb_data *cbd = cb_data_new(cb, data);
|
case OFONO_VENDOR_HUAWEI:
|
||||||
|
cbd = cb_data_new(cb, data);
|
||||||
|
|
||||||
if (cbd == NULL) {
|
if (cbd == NULL) {
|
||||||
CALLBACK_WITH_FAILURE(cb, NULL, data);
|
CALLBACK_WITH_FAILURE(cb, NULL, data);
|
||||||
|
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -545,12 +546,14 @@ static void at_pin_retries_query(struct ofono_sim *sim,
|
||||||
g_free(cbd);
|
g_free(cbd);
|
||||||
|
|
||||||
CALLBACK_WITH_FAILURE(cb, NULL, data);
|
CALLBACK_WITH_FAILURE(cb, NULL, data);
|
||||||
|
break;
|
||||||
|
|
||||||
|
default:
|
||||||
|
for(i = 0; i < OFONO_SIM_PASSWORD_INVALID; i++)
|
||||||
|
retries[i] = -1;
|
||||||
|
|
||||||
|
CALLBACK_WITH_SUCCESS(cb, retries, data);
|
||||||
}
|
}
|
||||||
|
|
||||||
for(i = 0; i < OFONO_SIM_PASSWORD_INVALID; i++)
|
|
||||||
retries[i] = -1;
|
|
||||||
|
|
||||||
CALLBACK_WITH_SUCCESS(cb, retries, data);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static void at_cpin_cb(gboolean ok, GAtResult *result, gpointer user_data)
|
static void at_cpin_cb(gboolean ok, GAtResult *result, gpointer user_data)
|
||||||
|
|
Loading…
Reference in New Issue