diff options
author | Paul Kocialkowski <contact@paulk.fr> | 2013-03-31 22:46:24 +0200 |
---|---|---|
committer | Paul Kocialkowski <contact@paulk.fr> | 2013-03-31 22:48:21 +0200 |
commit | ab6eef687bc891e3e47edd43cee7e39336d30d26 (patch) | |
tree | bb61b1c5fc844cc78ec1311dc830cdb40c787229 /sat.c | |
parent | 05951ff716f62b05da40fef1aa1f82e2fa9d2601 (diff) | |
download | hardware_ril_samsung-ril-ab6eef687bc891e3e47edd43cee7e39336d30d26.zip hardware_ril_samsung-ril-ab6eef687bc891e3e47edd43cee7e39336d30d26.tar.gz hardware_ril_samsung-ril-ab6eef687bc891e3e47edd43cee7e39336d30d26.tar.bz2 |
Avoid checking ipc_message_info pointer since it is checked in readloop
Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
Diffstat (limited to 'sat.c')
-rw-r--r-- | sat.c | 9 |
1 files changed, 4 insertions, 5 deletions
@@ -39,7 +39,7 @@ void ipc_sat_proactive_cmd_unsol(struct ipc_message_info *info) char *hexdata; int length; - if (info == NULL || info->data == NULL || info->length < 2) + if (info->data == NULL || info->length < 2) return; length = (info->length - 2); @@ -56,7 +56,7 @@ void ipc_sat_proactive_cmd_sol(struct ipc_message_info *info) { unsigned char sw1, sw2; - if (info == NULL || info->data == NULL || info->length < 2 * sizeof(unsigned char)) + if (info->data == NULL || info->length < 2 * sizeof(unsigned char)) goto error; sw1 = ((unsigned char*) info->data)[0]; @@ -147,7 +147,7 @@ void ipc_sat_envelope_cmd(struct ipc_message_info *info) char *hexdata; int size; - if (info == NULL || info->data == NULL || info->length < 2) + if (info->data == NULL || info->length < 2) goto error; size = (info->length - 2); @@ -162,7 +162,6 @@ void ipc_sat_envelope_cmd(struct ipc_message_info *info) return; error: - if (info != NULL) - ril_request_complete(ril_request_get_token(info->aseq), RIL_E_GENERIC_FAILURE, NULL, 0); + ril_request_complete(ril_request_get_token(info->aseq), RIL_E_GENERIC_FAILURE, NULL, 0); } |