Re: [PATCH BlueZ 1/7] tools/hciconfig: Add missing hci_close_dev

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

 



Hi,

On Wed, Aug 12, 2015 at 6:07 PM, Luiz Augusto von Dentz
<luiz.dentz@xxxxxxxxx> wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>
>
> Add hci_close_dev for commands using hci_open_dev, this is mostly to be
> consistent with other commands that do cleanup but in practice makes no
> difference since hciconfig will exit anyway.
> ---
>  tools/hciconfig.c | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
>
> diff --git a/tools/hciconfig.c b/tools/hciconfig.c
> index 0e0b790..eac96b0 100644
> --- a/tools/hciconfig.c
> +++ b/tools/hciconfig.c
> @@ -963,6 +963,8 @@ static void cmd_class(int ctl, int hdev, char *opt)
>                         printf("%s, %s\n", major_devices[cls[1] & 0x1f],
>                                 get_minor_device_name(cls[1] & 0x1f, cls[0] >> 2));
>         }
> +
> +       hci_close_dev(hdev);
>  }
>
>  static void cmd_voice(int ctl, int hdev, char *opt)
> @@ -1023,6 +1025,8 @@ static void cmd_voice(int ctl, int hdev, char *opt)
>                 }
>                 printf("\tAir Coding Format: %s\n", acf[vs & 0x03]);
>         }
> +
> +       hci_close_dev(s);
>  }
>
>  static void cmd_delkey(int ctl, int hdev, char *opt)
> @@ -1462,6 +1466,8 @@ static void cmd_inq_parms(int ctl, int hdev, char *opt)
>                 printf("\tInquiry interval: %u slots (%.2f ms), window: %u slots (%.2f ms)\n",
>                                 interval, (float)interval * 0.625, window, (float)window * 0.625);
>         }
> +
> +       hci_close_dev(s);
>  }
>
>  static void cmd_page_parms(int ctl, int hdev, char *opt)
> @@ -1533,6 +1539,8 @@ static void cmd_page_parms(int ctl, int hdev, char *opt)
>                         interval, (float)interval * 0.625,
>                         window, (float)window * 0.625);
>         }
> +
> +       hci_close_dev(s);
>  }
>
>  static void cmd_page_to(int ctl, int hdev, char *opt)
> @@ -1597,6 +1605,8 @@ static void cmd_page_to(int ctl, int hdev, char *opt)
>                 printf("\tPage timeout: %u slots (%.2f ms)\n",
>                                 timeout, (float)timeout * 0.625);
>         }
> +
> +       hci_close_dev(s);
>  }
>
>  static void cmd_afh_mode(int ctl, int hdev, char *opt)
> @@ -1630,6 +1640,8 @@ static void cmd_afh_mode(int ctl, int hdev, char *opt)
>                 print_dev_hdr(&di);
>                 printf("\tAFH mode: %s\n", mode == 1 ? "Enabled" : "Disabled");
>         }
> +
> +       hci_close_dev(dd);
>  }
>
>  static void cmd_ssp_mode(int ctl, int hdev, char *opt)
> @@ -1664,6 +1676,8 @@ static void cmd_ssp_mode(int ctl, int hdev, char *opt)
>                 printf("\tSimple Pairing mode: %s\n",
>                         mode == 1 ? "Enabled" : "Disabled");
>         }
> +
> +       hci_close_dev(dd);
>  }
>
>  static void print_rev_ericsson(int dd)
> @@ -1790,6 +1804,9 @@ static void cmd_revision(int ctl, int hdev, char *opt)
>                 printf("\tUnsupported manufacturer\n");
>                 break;
>         }
> +
> +       hci_close_dev(dd);
> +
>         return;
>  }
>
> --
> 2.4.3

Applied.


-- 
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