Re: [PATCH BlueZ] Handle HCI emulator setup errors

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Kuba,

On Thu, Oct 1, 2015 at 3:18 PM, Kuba Pawlak <kubax.t.pawlak@xxxxxxxxx> wrote:
> Missing return is causing confusing output from sco-tester:
>
>   Read Index List callback
>     Status: 0x00
>   Failed to setup HCI emulation
> eSCO CVSD - Success - pre setup failed
>   New hciemu instance created
> eSCO CVSD - Success - done
> ---
>  tools/sco-tester.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/tools/sco-tester.c b/tools/sco-tester.c
> index d55d51a3155faacb43742271c2e7ea6754b9f42c..651fbe01d14e59b4580704b3319196620d9d54e5 100644
> --- a/tools/sco-tester.c
> +++ b/tools/sco-tester.c
> @@ -160,6 +160,7 @@ static void read_index_list_callback(uint8_t status, uint16_t length,
>         if (!data->hciemu) {
>                 tester_warn("Failed to setup HCI emulation");
>                 tester_pre_setup_failed();
> +               return;
>         }
>
>         tester_print("New hciemu instance created");
> --
> 1.9.3

Applied, thanks.


-- 
Luiz Augusto von Dentz
--
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



[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux