Hi Sebastian,
On Fri, Jan 10, 2014 at 6:31 PM, Sebastian Chlad
<sebastianx.chlad@xxxxxxxxx> wrote:
> Bluetooth-player change-folder cmd approves any argument however it
> can be and then should a valid path only. Failing in providing
> a valid path crashes bluetooth-player thus argument should be
> checked if it's a valid path.
> ---
> tools/bluetooth-player.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/tools/bluetooth-player.c b/tools/bluetooth-player.c
> index 2afdd17..f10d9be 100644
> --- a/tools/bluetooth-player.c
> +++ b/tools/bluetooth-player.c
> @@ -738,6 +738,11 @@ static void cmd_change_folder(int argc, char *argv[])
> return;
> }
>
> + if (dbus_validate_path(argv[1], NULL) == FALSE) {
> + rl_printf("Not a valid path\n");
> + return;
> + }
> +
> if (check_default_player() == FALSE)
> return;
>
> --
> 1.8.3.2
Pushed, 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
