diff options
| author |   Dan Williams <dcbw@redhat.com> | 2012-05-31 17:23:08 -0500 |
|---|---|---|
| committer | Dan Williams <dcbw@redhat.com> | 2012-05-31 17:23:08 -0500 |
| commit | bd28fdfbfb567ac71cb5682254ea869d4aadb46e (patch) | |
| tree | 428990c0821c08416ed7bd2422aac5ce24f1e601 | |
| parent | 03fb11bd018868a793bf74a66d94685c2c533d78 (diff) | |
cdma: fix QCDM registration state checking
Broken by b22b2d99db57e4cec8e6c3074dd20acd6845cb62 which changed the
return types of the qcdm_result_get_*() function, but I forgot to
update the usage of these functions in mm-generic-cdma.c, so they
were assuming failure when in fact the HDR subsystem state request
was successful.
| -rw-r--r-- | src/mm-generic-cdma.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/mm-generic-cdma.c b/src/mm-generic-cdma.c index 5397ce95..a72232f1 100644 --- a/src/mm-generic-cdma.c +++ b/src/mm-generic-cdma.c @@ -1879,9 +1879,9 @@ reg_hdrstate_cb (MMQcdmSerialPort *port, guint8 almp_state = QCDM_CMD_HDR_SUBSYS_STATE_INFO_ALMP_STATE_INACTIVE; guint8 hybrid_mode = 0; - if ( qcdm_result_get_u8 (result, QCDM_CMD_HDR_SUBSYS_STATE_INFO_ITEM_SESSION_STATE, &session_state) - && qcdm_result_get_u8 (result, QCDM_CMD_HDR_SUBSYS_STATE_INFO_ITEM_ALMP_STATE, &almp_state) - && qcdm_result_get_u8 (result, QCDM_CMD_HDR_SUBSYS_STATE_INFO_ITEM_HDR_HYBRID_MODE, &hybrid_mode)) { + if ( qcdm_result_get_u8 (result, QCDM_CMD_HDR_SUBSYS_STATE_INFO_ITEM_SESSION_STATE, &session_state) == 0 + && qcdm_result_get_u8 (result, QCDM_CMD_HDR_SUBSYS_STATE_INFO_ITEM_ALMP_STATE, &almp_state) == 0 + && qcdm_result_get_u8 (result, QCDM_CMD_HDR_SUBSYS_STATE_INFO_ITEM_HDR_HYBRID_MODE, &hybrid_mode) == 0) { /* EVDO state is registered if the HDR subsystem is registered, and * we're in hybrid mode, and the Call Manager system mode is |