mirror of git://git.sysmocom.de/ofono
atmodem: Don't unregister *EPEV in *EPEV handler.
Handlers are called when looping over the unsolicited responses hash, registering or unregistering modifies that hash and confuses glib.
This commit is contained in:
parent
e8965f5f8c
commit
b450efafec
|
@ -45,6 +45,7 @@ struct sim_data {
|
||||||
GAtChat *chat;
|
GAtChat *chat;
|
||||||
unsigned int vendor;
|
unsigned int vendor;
|
||||||
guint epev_id;
|
guint epev_id;
|
||||||
|
guint epev_source;
|
||||||
};
|
};
|
||||||
|
|
||||||
static const char *crsm_prefix[] = { "+CRSM:", NULL };
|
static const char *crsm_prefix[] = { "+CRSM:", NULL };
|
||||||
|
@ -510,6 +511,18 @@ error:
|
||||||
CALLBACK_WITH_FAILURE(cb, -1, data);
|
CALLBACK_WITH_FAILURE(cb, -1, data);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static gboolean at_epev_unregister(gpointer user_data)
|
||||||
|
{
|
||||||
|
struct sim_data *sd = user_data;
|
||||||
|
|
||||||
|
sd->epev_source = 0;
|
||||||
|
|
||||||
|
g_at_chat_unregister(sd->chat, sd->epev_id);
|
||||||
|
sd->epev_id = 0;
|
||||||
|
|
||||||
|
return FALSE;
|
||||||
|
}
|
||||||
|
|
||||||
static void at_epev_notify(GAtResult *result, gpointer user_data)
|
static void at_epev_notify(GAtResult *result, gpointer user_data)
|
||||||
{
|
{
|
||||||
struct cb_data *cbd = user_data;
|
struct cb_data *cbd = user_data;
|
||||||
|
@ -517,10 +530,12 @@ static void at_epev_notify(GAtResult *result, gpointer user_data)
|
||||||
ofono_sim_lock_unlock_cb_t cb = cbd->cb;
|
ofono_sim_lock_unlock_cb_t cb = cbd->cb;
|
||||||
struct ofono_error error = { .type = OFONO_ERROR_TYPE_NO_ERROR };
|
struct ofono_error error = { .type = OFONO_ERROR_TYPE_NO_ERROR };
|
||||||
|
|
||||||
|
if (sd->epev_source)
|
||||||
|
return;
|
||||||
|
|
||||||
cb(&error, cbd->data);
|
cb(&error, cbd->data);
|
||||||
|
|
||||||
g_at_chat_unregister(sd->chat, sd->epev_id);
|
sd->epev_source = g_timeout_add(0, at_epev_unregister, sd);
|
||||||
sd->epev_id = 0;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static void at_pin_send_cb(gboolean ok, GAtResult *result,
|
static void at_pin_send_cb(gboolean ok, GAtResult *result,
|
||||||
|
@ -803,6 +818,9 @@ static void at_sim_remove(struct ofono_sim *sim)
|
||||||
|
|
||||||
ofono_sim_set_data(sim, NULL);
|
ofono_sim_set_data(sim, NULL);
|
||||||
|
|
||||||
|
if (sd->epev_source)
|
||||||
|
g_source_remove(sd->epev_source);
|
||||||
|
|
||||||
g_free(sd);
|
g_free(sd);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue