From patchwork Wed Mar 11 18:59:37 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Luiz Augusto von Dentz X-Patchwork-Id: 11432559 X-Patchwork-Delegate: luiz.dentz@gmail.com Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 9B3BA14B4 for ; Wed, 11 Mar 2020 18:59:46 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 245B120739 for ; Wed, 11 Mar 2020 18:59:46 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="VA9hBLVz" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731003AbgCKS7o (ORCPT ); Wed, 11 Mar 2020 14:59:44 -0400 Received: from mail-pg1-f177.google.com ([209.85.215.177]:41213 "EHLO mail-pg1-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730944AbgCKS7n (ORCPT ); Wed, 11 Mar 2020 14:59:43 -0400 Received: by mail-pg1-f177.google.com with SMTP id b1so1684235pgm.8 for ; Wed, 11 Mar 2020 11:59:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:subject:date:message-id:in-reply-to:references:mime-version :content-transfer-encoding; bh=N3+B3zeEeavgCNkSKoeHw6qkZDgzNb8+BRA4fkGVCkY=; b=VA9hBLVzluOOyXPnJKXhyCYLblWAA2AVLv875fM/So1fXSqKxPe1eZo7ao7iNUcdlI P1e+rGGmPnFteD46qAx8Hi5YrgUDVgtUzkFFCC1xOCdyFlFcr6xXREz/950nWei1JAZB nwooYydYEQSJFUHjjj5NkMjigKZ4lLW4qWDiZv+4+MFgIyJR8AoirvNwHogpyZ4skMyE 1XAE1EDyCPOADZzxl/a2vlrJcanfzhctWNpPiGbNiLtAvwaM/S6//06Y7nCWHHX8/x7a E8mnJ7KoWrNi7+0RoG6+8uNXd5xsmnT4o1bsA7OR6j63EAUSDut1EOcgVGQZ9MqTp1F6 8bWw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=N3+B3zeEeavgCNkSKoeHw6qkZDgzNb8+BRA4fkGVCkY=; b=FyLfPv3aZhU3G3z0/3r9UOwbf3GOyPrZm+aNSEKEtePGYGLVtbbkmN2p977B3NDdW+ B+H7gxDTBQXg4wMU/sLnmhIZljj8jEtzYF6SqquiFcOPF9cso0xh92n3fUVCEePLx0+L WFUp3b8IyIEIKLuPQJhfnEj8Fx+D2RIorqv5xdl/XGfhcIyYCTWon/hdYfr84xtLuE+J cW7UXszkPQGVrX2NZt17nM+6w1DcArBixz0B+QtSx2vCt7v99dIDf3EoUkBKmCstggQw IsQ5GxS+oZaTXT2maNvdATviyjQDy5bDGAxat9n/CYHNCYX7J6AcRxCdfkFhRf/tyKaS OWkA== X-Gm-Message-State: ANhLgQ3Ynysoaxgf2V8brQVWOB8n9L/BnKl0moNZMeDqFoE64Cv47d7S u3x5LeiRLvwuO0eZ9QKNcZhIxbGBXgA= X-Google-Smtp-Source: ADFU+vtB/yMZ13p7H1EUhOjZBWyjBBB2nRRJbbns28oiCE5ojJvajjHpqq4IVFB5qBzQDzZWMKjjPg== X-Received: by 2002:a63:544:: with SMTP id 65mr4041597pgf.72.1583953180659; Wed, 11 Mar 2020 11:59:40 -0700 (PDT) Received: from localhost.localdomain (c-71-56-157-77.hsd1.or.comcast.net. [71.56.157.77]) by smtp.gmail.com with ESMTPSA id o2sm45693327pfh.26.2020.03.11.11.59.39 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 11 Mar 2020 11:59:40 -0700 (PDT) From: Luiz Augusto von Dentz To: linux-bluetooth@vger.kernel.org Subject: [PATCH v2 2/2] input: Add LEAudioSecurity setting to input.conf Date: Wed, 11 Mar 2020 11:59:37 -0700 Message-Id: <20200311185937.9032-2-luiz.dentz@gmail.com> X-Mailer: git-send-email 2.21.1 In-Reply-To: <20200311185937.9032-1-luiz.dentz@gmail.com> References: <20200311185937.9032-1-luiz.dentz@gmail.com> MIME-Version: 1.0 Sender: linux-bluetooth-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-bluetooth@vger.kernel.org From: Luiz Augusto von Dentz LEAudioSecurity can be used to enable/disable automatic upgrades of security for LE devices, by default it is enabled so existing devices that did not require security and were not bonded will automatically upgrade the security. Note: Platforms disabling this setting would require users to manually bond the device which may require changes to the user interface to always force bonding for input devices ad APIs such as Device.Connect directly will no longer work which maybe perceived as a regression. --- profiles/input/device.h | 1 + profiles/input/hog.c | 13 +++++++++++-- profiles/input/input.conf | 5 +++++ profiles/input/manager.c | 11 ++++++++++- 4 files changed, 27 insertions(+), 3 deletions(-) diff --git a/profiles/input/device.h b/profiles/input/device.h index 3044db673..5a077f92a 100644 --- a/profiles/input/device.h +++ b/profiles/input/device.h @@ -30,6 +30,7 @@ struct input_conn; void input_set_idle_timeout(int timeout); void input_enable_userspace_hid(bool state); void input_set_classic_bonded_only(bool state); +void input_set_auto_sec(bool state); int input_device_register(struct btd_service *service); void input_device_unregister(struct btd_service *service); diff --git a/profiles/input/hog.c b/profiles/input/hog.c index f0226ebbd..327a1d1c3 100644 --- a/profiles/input/hog.c +++ b/profiles/input/hog.c @@ -53,6 +53,7 @@ #include "src/shared/gatt-client.h" #include "src/plugin.h" +#include "device.h" #include "suspend.h" #include "attrib/att.h" #include "attrib/gattrib.h" @@ -67,8 +68,14 @@ struct hog_device { }; static gboolean suspend_supported = FALSE; +static bool auto_sec = true; static struct queue *devices = NULL; +void input_set_auto_sec(bool state) +{ + auto_sec = state; +} + static void hog_device_accept(struct hog_device *dev, struct gatt_db *db) { char name[248]; @@ -192,11 +199,13 @@ static int hog_accept(struct btd_service *service) if (!device_is_bonded(device, btd_device_get_bdaddr_type(device))) { struct bt_gatt_client *client; + if (!auto_sec) + return -ECONNREFUSED; + client = btd_device_get_gatt_client(device); if (!bt_gatt_client_set_security(client, - BT_ATT_SECURITY_MEDIUM)) { + BT_ATT_SECURITY_MEDIUM)) return -ECONNREFUSED; - } } /* TODO: Replace GAttrib with bt_gatt_client */ diff --git a/profiles/input/input.conf b/profiles/input/input.conf index 166aff4a4..4c70bc561 100644 --- a/profiles/input/input.conf +++ b/profiles/input/input.conf @@ -19,3 +19,8 @@ # pairing/encryption. # Defaults to false to maximize device compatibility. #ClassicBondedOnly=true + +# LE upgrade security +# Enables upgrades of security automatically if required. +# Defaults to true to maximize device compatibility. +#LEAutoSecurity=true diff --git a/profiles/input/manager.c b/profiles/input/manager.c index 5cd27b839..bf4acb4ed 100644 --- a/profiles/input/manager.c +++ b/profiles/input/manager.c @@ -96,7 +96,7 @@ static int input_init(void) config = load_config_file(CONFIGDIR "/input.conf"); if (config) { int idle_timeout; - gboolean uhid_enabled, classic_bonded_only; + gboolean uhid_enabled, classic_bonded_only, auto_sec; idle_timeout = g_key_file_get_integer(config, "General", "IdleTimeout", &err); @@ -125,6 +125,15 @@ static int input_init(void) } else g_clear_error(&err); + auto_sec = g_key_file_get_boolean(config, "General", + "LEAutoSecurity", &err); + if (!err) { + DBG("input.conf: LEAutoSecurity=%s", + auto_sec ? "true" : "false"); + input_set_auto_sec(auto_sec); + } else + g_clear_error(&err); + } btd_profile_register(&input_profile);