Re: [PATCH] sim: Query the status of SC facility lock
Hi Slava, On 07/06/2016 11:02 AM, Slava Monich wrote: Resending after adjusting code style. --- src/sim.c | 72 +++ 1 file changed, 35 insertions(+), 37 deletions(-) Applied, thanks. Regards, -Denis ___ ofono mailing list ofono@ofono.org https://lists.ofono.org/mailman/listinfo/ofono
[PATCH] sim: Query the status of SC facility lock
Resending after adjusting code style. --- src/sim.c | 72 +++ 1 file changed, 35 insertions(+), 37 deletions(-) diff --git a/src/sim.c b/src/sim.c index aedc617..e2da78d 100644 --- a/src/sim.c +++ b/src/sim.c @@ -2449,58 +2449,50 @@ static void sim_free_state(struct ofono_sim *sim) sim_free_main_state(sim); } -static void sim_query_fac_imsilock_cb(const struct ofono_error *error, - ofono_bool_t status, - void *data) +static void sim_set_locked_pin(struct ofono_sim *sim, + enum ofono_sim_password_type type, gboolean locked) { - struct ofono_sim *sim = data; - DBusConnection *conn = ofono_dbus_get_connection(); - const char *path = __ofono_atom_get_path(sim->atom); char **locked_pins; - if (error->type != OFONO_ERROR_TYPE_NO_ERROR) { - ofono_error("Querying Facility Lock for IMSI Lock failed"); + if (sim->locked_pins[type] == locked) return; - } - - sim->locked_pins[OFONO_SIM_PASSWORD_PHSIM_PIN] = status; + sim->locked_pins[type] = locked; locked_pins = get_locked_pins(sim); - ofono_dbus_signal_array_property_changed(conn, - path, - OFONO_SIM_MANAGER_INTERFACE, - "LockedPins", DBUS_TYPE_STRING, - _pins); + ofono_dbus_signal_array_property_changed(ofono_dbus_get_connection(), + __ofono_atom_get_path(sim->atom), + OFONO_SIM_MANAGER_INTERFACE, "LockedPins", + DBUS_TYPE_STRING, _pins); g_strfreev(locked_pins); } -static void sim_query_fac_networklock_cb(const struct ofono_error *error, - ofono_bool_t status, - void *data) +static void sim_query_fac_imsilock_cb(const struct ofono_error *error, + ofono_bool_t status, void *data) { - struct ofono_sim *sim = data; - DBusConnection *conn = ofono_dbus_get_connection(); - const char *path = __ofono_atom_get_path(sim->atom); - char **locked_pins; - - if (error->type != OFONO_ERROR_TYPE_NO_ERROR) { - ofono_error("Querying Facility Lock for Network Lock failed"); + if (error->type != OFONO_ERROR_TYPE_NO_ERROR) return; - } - sim->locked_pins[OFONO_SIM_PASSWORD_PHNET_PIN] = status; + sim_set_locked_pin(data, OFONO_SIM_PASSWORD_PHSIM_PIN, status); +} - locked_pins = get_locked_pins(sim); +static void sim_query_fac_networklock_cb(const struct ofono_error *error, + ofono_bool_t status, void *data) +{ + if (error->type != OFONO_ERROR_TYPE_NO_ERROR) + return; - ofono_dbus_signal_array_property_changed(conn, - path, - OFONO_SIM_MANAGER_INTERFACE, - "LockedPins", DBUS_TYPE_STRING, - _pins); + sim_set_locked_pin(data, OFONO_SIM_PASSWORD_PHNET_PIN, status); +} - g_strfreev(locked_pins); +static void sim_query_fac_pinlock_cb(const struct ofono_error *error, + ofono_bool_t status, void *data) +{ + if (error->type != OFONO_ERROR_TYPE_NO_ERROR) + return; + + sim_set_locked_pin(data, OFONO_SIM_PASSWORD_SIM_PIN, status); } void ofono_sim_inserted_notify(struct ofono_sim *sim, ofono_bool_t inserted) @@ -2529,14 +2521,20 @@ void ofono_sim_inserted_notify(struct ofono_sim *sim, ofono_bool_t inserted) call_state_watches(sim); if (inserted) { - sim->driver->query_facility_lock(sim, + if (sim->driver->query_facility_lock) { + sim->driver->query_facility_lock(sim, OFONO_SIM_PASSWORD_PHSIM_PIN, sim_query_fac_imsilock_cb, sim); - sim->driver->query_facility_lock(sim, + sim->driver->query_facility_lock(sim, OFONO_SIM_PASSWORD_PHNET_PIN, sim_query_fac_networklock_cb, sim); + sim->driver->query_facility_lock(sim, + OFONO_SIM_PASSWORD_SIM_PIN, + sim_query_fac_pinlock_cb, sim); + } + sim_initialize(sim); } else { sim->pin_type = OFONO_SIM_PASSWORD_NONE; -- 1.9.1 ___ ofono mailing list ofono@ofono.org https://lists.ofono.org/mailman/listinfo/ofono
Re: [PATCH] sim: Query the status of SC facility lock
Hi Slava, On 07/06/2016 03:34 AM, Slava Monich wrote: Otherwise "pin" entry in LockedPins may be lost after ofono is restarted. --- src/sim.c | 80 +-- 1 file changed, 37 insertions(+), 43 deletions(-) diff --git a/src/sim.c b/src/sim.c index aedc617..634acfc 100644 --- a/src/sim.c +++ b/src/sim.c @@ -2449,58 +2449,47 @@ static void sim_free_state(struct ofono_sim *sim) sim_free_main_state(sim); } -static void sim_query_fac_imsilock_cb(const struct ofono_error *error, - ofono_bool_t status, - void *data) +static void sim_set_locked_pin(struct ofono_sim *sim, + enum ofono_sim_password_type type, gboolean locked) { - struct ofono_sim *sim = data; - DBusConnection *conn = ofono_dbus_get_connection(); - const char *path = __ofono_atom_get_path(sim->atom); - char **locked_pins; + if (sim->locked_pins[type] != locked) { + char **locked_pins; I'd prefer: if (sim->locked_pins[type] == locked) return; .. do stuff .. - if (error->type != OFONO_ERROR_TYPE_NO_ERROR) { - ofono_error("Querying Facility Lock for IMSI Lock failed"); - return; - } + sim->locked_pins[type] = locked; + locked_pins = get_locked_pins(sim); - sim->locked_pins[OFONO_SIM_PASSWORD_PHSIM_PIN] = status; + ofono_dbus_signal_array_property_changed( + ofono_dbus_get_connection(), + __ofono_atom_get_path(sim->atom), + OFONO_SIM_MANAGER_INTERFACE, "LockedPins", + DBUS_TYPE_STRING, _pins); - locked_pins = get_locked_pins(sim); - - ofono_dbus_signal_array_property_changed(conn, - path, - OFONO_SIM_MANAGER_INTERFACE, - "LockedPins", DBUS_TYPE_STRING, - _pins); + g_strfreev(locked_pins); + } +} - g_strfreev(locked_pins); +static void sim_query_fac_imsilock_cb(const struct ofono_error *error, + ofono_bool_t status, void *data) +{ + if (error->type == OFONO_ERROR_TYPE_NO_ERROR) { + sim_set_locked_pin(data, OFONO_SIM_PASSWORD_PHSIM_PIN, status); + } This is not in-line with the Linux Kernel coding style. Also, I'd prefer: if (error->type != OFONO_ERROR_TYPE_NO_ERROR) return; sim_set_locked_pin... } static void sim_query_fac_networklock_cb(const struct ofono_error *error, - ofono_bool_t status, - void *data) + ofono_bool_t status, void *data) { - struct ofono_sim *sim = data; - DBusConnection *conn = ofono_dbus_get_connection(); - const char *path = __ofono_atom_get_path(sim->atom); - char **locked_pins; - - if (error->type != OFONO_ERROR_TYPE_NO_ERROR) { - ofono_error("Querying Facility Lock for Network Lock failed"); - return; + if (error->type == OFONO_ERROR_TYPE_NO_ERROR) { + sim_set_locked_pin(data, OFONO_SIM_PASSWORD_PHNET_PIN, status); } +} as above - sim->locked_pins[OFONO_SIM_PASSWORD_PHNET_PIN] = status; - - locked_pins = get_locked_pins(sim); - - ofono_dbus_signal_array_property_changed(conn, - path, - OFONO_SIM_MANAGER_INTERFACE, - "LockedPins", DBUS_TYPE_STRING, - _pins); - - g_strfreev(locked_pins); +static void sim_query_fac_pinlock_cb(const struct ofono_error *error, + ofono_bool_t status, void *data) +{ + if (error->type == OFONO_ERROR_TYPE_NO_ERROR) { + sim_set_locked_pin(data, OFONO_SIM_PASSWORD_SIM_PIN, status); + } as above } void ofono_sim_inserted_notify(struct ofono_sim *sim, ofono_bool_t inserted) @@ -2529,14 +2518,19 @@ void ofono_sim_inserted_notify(struct ofono_sim *sim, ofono_bool_t inserted) call_state_watches(sim); if (inserted) { - sim->driver->query_facility_lock(sim, + if (sim->driver->query_facility_lock) { + sim->driver->query_facility_lock(sim, OFONO_SIM_PASSWORD_PHSIM_PIN, sim_query_fac_imsilock_cb, sim); - sim->driver->query_facility_lock(sim, + sim->driver->query_facility_lock(sim, OFONO_SIM_PASSWORD_PHNET_PIN, sim_query_fac_networklock_cb, sim); + sim->driver->query_facility_lock(sim, +
[PATCH] sim: Query the status of SC facility lock
Otherwise "pin" entry in LockedPins may be lost after ofono is restarted. --- src/sim.c | 80 +-- 1 file changed, 37 insertions(+), 43 deletions(-) diff --git a/src/sim.c b/src/sim.c index aedc617..634acfc 100644 --- a/src/sim.c +++ b/src/sim.c @@ -2449,58 +2449,47 @@ static void sim_free_state(struct ofono_sim *sim) sim_free_main_state(sim); } -static void sim_query_fac_imsilock_cb(const struct ofono_error *error, - ofono_bool_t status, - void *data) +static void sim_set_locked_pin(struct ofono_sim *sim, + enum ofono_sim_password_type type, gboolean locked) { - struct ofono_sim *sim = data; - DBusConnection *conn = ofono_dbus_get_connection(); - const char *path = __ofono_atom_get_path(sim->atom); - char **locked_pins; + if (sim->locked_pins[type] != locked) { + char **locked_pins; - if (error->type != OFONO_ERROR_TYPE_NO_ERROR) { - ofono_error("Querying Facility Lock for IMSI Lock failed"); - return; - } + sim->locked_pins[type] = locked; + locked_pins = get_locked_pins(sim); - sim->locked_pins[OFONO_SIM_PASSWORD_PHSIM_PIN] = status; + ofono_dbus_signal_array_property_changed( + ofono_dbus_get_connection(), + __ofono_atom_get_path(sim->atom), + OFONO_SIM_MANAGER_INTERFACE, "LockedPins", + DBUS_TYPE_STRING, _pins); - locked_pins = get_locked_pins(sim); - - ofono_dbus_signal_array_property_changed(conn, - path, - OFONO_SIM_MANAGER_INTERFACE, - "LockedPins", DBUS_TYPE_STRING, - _pins); + g_strfreev(locked_pins); + } +} - g_strfreev(locked_pins); +static void sim_query_fac_imsilock_cb(const struct ofono_error *error, + ofono_bool_t status, void *data) +{ + if (error->type == OFONO_ERROR_TYPE_NO_ERROR) { + sim_set_locked_pin(data, OFONO_SIM_PASSWORD_PHSIM_PIN, status); + } } static void sim_query_fac_networklock_cb(const struct ofono_error *error, - ofono_bool_t status, - void *data) + ofono_bool_t status, void *data) { - struct ofono_sim *sim = data; - DBusConnection *conn = ofono_dbus_get_connection(); - const char *path = __ofono_atom_get_path(sim->atom); - char **locked_pins; - - if (error->type != OFONO_ERROR_TYPE_NO_ERROR) { - ofono_error("Querying Facility Lock for Network Lock failed"); - return; + if (error->type == OFONO_ERROR_TYPE_NO_ERROR) { + sim_set_locked_pin(data, OFONO_SIM_PASSWORD_PHNET_PIN, status); } +} - sim->locked_pins[OFONO_SIM_PASSWORD_PHNET_PIN] = status; - - locked_pins = get_locked_pins(sim); - - ofono_dbus_signal_array_property_changed(conn, - path, - OFONO_SIM_MANAGER_INTERFACE, - "LockedPins", DBUS_TYPE_STRING, - _pins); - - g_strfreev(locked_pins); +static void sim_query_fac_pinlock_cb(const struct ofono_error *error, + ofono_bool_t status, void *data) +{ + if (error->type == OFONO_ERROR_TYPE_NO_ERROR) { + sim_set_locked_pin(data, OFONO_SIM_PASSWORD_SIM_PIN, status); + } } void ofono_sim_inserted_notify(struct ofono_sim *sim, ofono_bool_t inserted) @@ -2529,14 +2518,19 @@ void ofono_sim_inserted_notify(struct ofono_sim *sim, ofono_bool_t inserted) call_state_watches(sim); if (inserted) { - sim->driver->query_facility_lock(sim, + if (sim->driver->query_facility_lock) { + sim->driver->query_facility_lock(sim, OFONO_SIM_PASSWORD_PHSIM_PIN, sim_query_fac_imsilock_cb, sim); - sim->driver->query_facility_lock(sim, + sim->driver->query_facility_lock(sim, OFONO_SIM_PASSWORD_PHNET_PIN, sim_query_fac_networklock_cb, sim); + sim->driver->query_facility_lock(sim, + OFONO_SIM_PASSWORD_SIM_PIN, + sim_query_fac_pinlock_cb, sim); + } sim_initialize(sim); } else { sim->pin_type = OFONO_SIM_PASSWORD_NONE; -- 1.9.1 ___ ofono mailing list