Received-SPF: pass (google.com: domain of selinux-refpolicy+bounces-26-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <2f12322b-4f04-4b77-b539-9b5b6c7985e3@ieee.org>
Date: Tue, 4 Jun 2024 09:59:12 -0400
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v1] Sepolicy changes for bluez to access uhid
To: Amisha Jain <quic_amisjain@quicinc.com>, selinux-refpolicy@vger.kernel.org
Cc: quic_mohamull@quicinc.com, quic_hbandi@quicinc.com,
 quic_anubhavg@quicinc.com
References: <20240529130814.13423-1-quic_amisjain@quicinc.com>
Content-Language: en-US
From: Chris PeBenito <pebenito@ieee.org>
In-Reply-To: <20240529130814.13423-1-quic_amisjain@quicinc.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

On 5/29/2024 9:08 AM, Amisha Jain wrote:
> Resolve selinux premission for HID
> 
> Below avc denials that are fixed with this patch -
> 
> avc:  denied  { read write } for  pid=656 comm="bluetoothd" name="uhid" dev="devtmpfs" ino=841 scontext=system_u:system_r:bluetooth_t:s0-s15:c0.c1023 tcontext=system_u:object_r:uhid_device_t:s0 tclass=chr_file permissive=0
> 
> Signed-off-by: Amisha Jain <quic_amisjain@quicinc.com>
> ---
>   policy/modules/services/bluetooth.te | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/policy/modules/services/bluetooth.te b/policy/modules/services/bluetooth.te
> index f23a979de..b2d6a9685 100644
> --- a/policy/modules/services/bluetooth.te
> +++ b/policy/modules/services/bluetooth.te
> @@ -104,6 +104,7 @@ dev_rw_generic_usb_dev(bluetooth_t)
>   dev_read_urand(bluetooth_t)
>   dev_rw_input_dev(bluetooth_t)
>   dev_rw_wireless(bluetooth_t)
> +uhid_device_rw(bluetooth_t)

This interface doesn't exist in refpolicy.  It would need to be in the 
devces module, probably named "device_rw_uhid".

-- 
Chris PeBenito