bluetooth: services: hrs: add characteristic permission config

Added a configuration of characteristic access permissions for
Bluetooth Heart Rate service.

Signed-off-by: Kamil Piszczek <[email protected]>

Author: kapi-no

subsys/bluetooth/services/Kconfig.hrs

@@ -6,6 +6,27 @@
 menuconfig BT_HRS
 	bool "Enable GATT Heart Rate service"
 
+if BT_HRS
+
+choice
+	prompt "Default permissions used for HRS characteristics"
+	default BT_HRS_DEFAULT_PERM_RW
+	help
+	  Default permissions for HRS characteristic attributes
+	  used when no permission is set for the report.
+	  Used also for boot reports.
+
+config BT_HRS_DEFAULT_PERM_RW
+	bool "Read and write allowed"
+
+config BT_HRS_DEFAULT_PERM_RW_ENCRYPT
+	bool "Require encryption for access"
+
+config BT_HRS_DEFAULT_PERM_RW_AUTHEN
+	bool "Require encryption and authentication for access"
+
+endchoice
+
 config BT_HRS_LOG_LEVEL
 	int "Heart Rate service log level"
 	depends on LOG
@@ -20,3 +41,5 @@ config BT_HRS_LOG_LEVEL
 	  2 WARNING, write LOG_WRN in addition to previous level
 	  3 INFO, write LOG_INF in addition to previous levels
 	  4 DEBUG, write LOG_DBG in addition to previous levels
+
+endif # BT_HRS

subsys/bluetooth/services/hrs.c

@@ -25,6 +25,31 @@
 #include <logging/log.h>
 LOG_MODULE_REGISTER(hrs);
 
+
+#define GATT_PERM_READ_MASK     (BT_GATT_PERM_READ | \
+				 BT_GATT_PERM_READ_ENCRYPT | \
+				 BT_GATT_PERM_READ_AUTHEN)
+#define GATT_PERM_WRITE_MASK    (BT_GATT_PERM_WRITE | \
+				 BT_GATT_PERM_WRITE_ENCRYPT | \
+				 BT_GATT_PERM_WRITE_AUTHEN)
+
+#ifndef CONFIG_BT_HRS_DEFAULT_PERM_RW_AUTHEN
+#define CONFIG_BT_HRS_DEFAULT_PERM_RW_AUTHEN 0
+#endif
+#ifndef CONFIG_BT_HRS_DEFAULT_PERM_RW_ENCRYPT
+#define CONFIG_BT_HRS_DEFAULT_PERM_RW_ENCRYPT 0
+#endif
+#ifndef CONFIG_BT_HRS_DEFAULT_PERM_RW
+#define CONFIG_BT_HRS_DEFAULT_PERM_RW 0
+#endif
+
+#define HRS_GATT_PERM_DEFAULT (						\
+	CONFIG_BT_HRS_DEFAULT_PERM_RW_AUTHEN ?				\
+	(BT_GATT_PERM_READ_AUTHEN | BT_GATT_PERM_WRITE_AUTHEN) :	\
+	CONFIG_BT_HRS_DEFAULT_PERM_RW_ENCRYPT ?				\
+	(BT_GATT_PERM_READ_ENCRYPT | BT_GATT_PERM_WRITE_ENCRYPT) :	\
+	(BT_GATT_PERM_READ | BT_GATT_PERM_WRITE))			\
+
 static uint8_t hrs_blsc;
 
 static void hrmc_ccc_cfg_changed(const struct bt_gatt_attr *attr, uint16_t value)
@@ -49,11 +74,13 @@ BT_GATT_SERVICE_DEFINE(hrs_svc,
 	BT_GATT_CHARACTERISTIC(BT_UUID_HRS_MEASUREMENT, BT_GATT_CHRC_NOTIFY,
 			       BT_GATT_PERM_NONE, NULL, NULL, NULL),
 	BT_GATT_CCC(hrmc_ccc_cfg_changed,
-		    BT_GATT_PERM_READ | BT_GATT_PERM_WRITE),
+		    HRS_GATT_PERM_DEFAULT),
 	BT_GATT_CHARACTERISTIC(BT_UUID_HRS_BODY_SENSOR, BT_GATT_CHRC_READ,
-			       BT_GATT_PERM_READ, read_blsc, NULL, NULL),
+			       HRS_GATT_PERM_DEFAULT & GATT_PERM_READ_MASK,
+			       read_blsc, NULL, NULL),
 	BT_GATT_CHARACTERISTIC(BT_UUID_HRS_CONTROL_POINT, BT_GATT_CHRC_WRITE,
-			       BT_GATT_PERM_NONE, NULL, NULL, NULL),
+			       HRS_GATT_PERM_DEFAULT & GATT_PERM_WRITE_MASK,
+			       NULL, NULL, NULL),
 );
 
 static int hrs_init(const struct device *dev)