2020-05-08 13:25:51

by Łukasz Rymanowski

[permalink] [raw]
Subject: [PATCH BlueZ] tools/btgatt-client: Add option to set BT_SECURITY_FIPS

Need for GAP/SEC/SEM/BI-10-C
---
tools/btgatt-client.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/tools/btgatt-client.c b/tools/btgatt-client.c
index 82a9e3fe0..bc762dba1 100644
--- a/tools/btgatt-client.c
+++ b/tools/btgatt-client.c
@@ -1492,8 +1492,8 @@ static void usage(void)
"\t-d, --dest <addr>\t\tSpecify the destination address\n"
"\t-t, --type [random|public] \tSpecify the LE address type\n"
"\t-m, --mtu <mtu> \t\tThe ATT MTU to use\n"
- "\t-s, --security-level <sec> \tSet security level (low|"
- "medium|high)\n"
+ "\t-s, --security-level <sec> \tSet security level (low|medium|"
+ "high|fips)\n"
"\t-v, --verbose\t\t\tEnable extra logging\n"
"\t-h, --help\t\t\tDisplay help\n");
}
@@ -1537,6 +1537,8 @@ int main(int argc, char *argv[])
sec = BT_SECURITY_MEDIUM;
else if (strcmp(optarg, "high") == 0)
sec = BT_SECURITY_HIGH;
+ else if (strcmp(optarg, "fips") == 0)
+ sec = BT_SECURITY_FIPS;
else {
fprintf(stderr, "Invalid security level\n");
return EXIT_FAILURE;
--
2.20.1


2020-05-13 11:23:33

by Szymon Janc

[permalink] [raw]
Subject: Re: [PATCH BlueZ] tools/btgatt-client: Add option to set BT_SECURITY_FIPS

Hi Łukasz,

On Friday, 8 May 2020 15:23:49 CEST Łukasz Rymanowski wrote:
> Need for GAP/SEC/SEM/BI-10-C
> ---
> tools/btgatt-client.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/tools/btgatt-client.c b/tools/btgatt-client.c
> index 82a9e3fe0..bc762dba1 100644
> --- a/tools/btgatt-client.c
> +++ b/tools/btgatt-client.c
> @@ -1492,8 +1492,8 @@ static void usage(void)
> "\t-d, --dest <addr>\t\tSpecify the destination address\n"
> "\t-t, --type [random|public] \tSpecify the LE address
type\n"
> "\t-m, --mtu <mtu> \t\tThe ATT MTU to use\n"
> - "\t-s, --security-level <sec> \tSet security level (low|"
> -
"medium|high)\n"
> + "\t-s, --security-level <sec> \tSet security level (low|
medium|"
> +
"high|fips)\n"
> "\t-v, --verbose\t\t\tEnable extra logging\n"
> "\t-h, --help\t\t\tDisplay help\n");
> }
> @@ -1537,6 +1537,8 @@ int main(int argc, char *argv[])
> sec = BT_SECURITY_MEDIUM;
> else if (strcmp(optarg, "high") == 0)
> sec = BT_SECURITY_HIGH;
> + else if (strcmp(optarg, "fips") == 0)
> + sec = BT_SECURITY_FIPS;
> else {
> fprintf(stderr, "Invalid security
level\n");
> return EXIT_FAILURE;

Patch applied, thanks.

--
pozdrawiam
Szymon Janc