Hi Andrei,
On Tuesday 26 of August 2014 16:42:19 Andrei Emeltchenko wrote:
> From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx>
>
> In case argc==3 we might access argv[3]. Make code more readable.
> ---
> android/client/if-hf.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/android/client/if-hf.c b/android/client/if-hf.c
> index d0e7a66..77216df 100644
> --- a/android/client/if-hf.c
> +++ b/android/client/if-hf.c
> @@ -576,7 +576,7 @@ static void at_response_c(int argc, const char **argv, enum_func *enum_func,
> static void at_response_p(int argc, const char **argv)
> {
> bthf_at_response_t response_code;
> - int error_code = 0;
> + int error_code;
>
> RETURN_IF_NULL(if_hf);
>
> @@ -588,7 +588,9 @@ static void at_response_p(int argc, const char **argv)
> response_code = str2bthf_at_response_t(argv[2]);
>
> /* error code */
> - if (argc >= 3)
> + if (argc <= 3)
> + error_code = 0;
> + else
> error_code = atoi(argv[3]);
>
> EXEC(if_hf->at_response, response_code, error_code);
>
Applied, thanks.
--
Best regards,
Szymon Janc
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
