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 /call.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 'call.c')
-rw-r--r-- | call.c | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -84,7 +84,7 @@ void ipc_call_status(struct ipc_message_info *info) { struct ipc_call_status *call_status; - if (info == NULL || info->data == NULL || info->length < sizeof(struct ipc_call_status)) + if (info->data == NULL || info->length < sizeof(struct ipc_call_status)) return; call_status = (struct ipc_call_status *) info->data; @@ -141,7 +141,7 @@ void ipc_call_list(struct ipc_message_info *info) RIL_Call **current_calls = NULL; int i; - if (info == NULL || info->data == NULL || info->length < sizeof(unsigned char)) + if (info->data == NULL || info->length < sizeof(unsigned char)) goto error; if (info->type != IPC_TYPE_RESP) @@ -206,7 +206,7 @@ error: free(current_calls); } - if (info != NULL && info->type == IPC_TYPE_RESP) + if (info->type == IPC_TYPE_RESP) ril_request_complete(ril_request_get_token(info->aseq), RIL_E_GENERIC_FAILURE, NULL, 0); } @@ -299,7 +299,7 @@ void ipc_call_burst_dtmf(struct ipc_message_info *info) { unsigned char code; - if (info == NULL || info->data == NULL || info->length < sizeof(unsigned char)) + if (info->data == NULL || info->length < sizeof(unsigned char)) goto error; code = *((unsigned char *) info->data); |