Before executing commands should be checked if any or chosen HCI is powered up. If not further executing of command makes no sense. --- android/pics-spp.txt | 2 +- android/pixit-spp.txt | 2 +- tools/sdptool.c | 12 ++++++++++-- 3 files changed, 12 insertions(+), 4 deletions(-) diff --git a/android/pics-spp.txt b/android/pics-spp.txt index f015236..d9bb883 100644 --- a/android/pics-spp.txt +++ b/android/pics-spp.txt @@ -1,6 +1,6 @@ SPP PICS for the PTS tool. -PTS version: 5.3 +PTS version: 6.0 * - different than PTS defaults # - not yet implemented/supported diff --git a/android/pixit-spp.txt b/android/pixit-spp.txt index a8a8202..8e2d70f 100644 --- a/android/pixit-spp.txt +++ b/android/pixit-spp.txt @@ -1,6 +1,6 @@ SPP PIXIT for the PTS tool. -PTS version: 5.3 +PTS version: 6.0 * - different than PTS defaults & - should be set to IUT Bluetooth address diff --git a/tools/sdptool.c b/tools/sdptool.c index 17561b1..9ff7a1c 100644 --- a/tools/sdptool.c +++ b/tools/sdptool.c @@ -4352,17 +4352,20 @@ static struct option main_options[] = { int main(int argc, char *argv[]) { - int i, opt; + int i, opt, err; bacpy(&interface, BDADDR_ANY); + err = hci_get_route(NULL); + while ((opt=getopt_long(argc, argv, "+i:h", main_options, NULL)) != -1) { switch(opt) { case 'i': if (!strncmp(optarg, "hci", 3)) - hci_devba(atoi(optarg + 3), &interface); + err = hci_devba(atoi(optarg + 3), &interface); else str2ba(optarg, &interface); + err = hci_get_route(&interface); break; case 'h': @@ -4374,6 +4377,11 @@ int main(int argc, char *argv[]) } } + if (err < 0) { + printf("No powered on HCI interface found\n"); + exit(0); + } + argc -= optind; argv += optind; optind = 0; -- 1.9.3 -- 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