Re: [RESEND PATCH v1 2/2] misc: fastrpc: detect privileged processes based on group ID

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

 





On 6/8/2023 12:17 AM, Krzysztof Kozlowski wrote:
On 07/06/2023 18:30, Ekansh Gupta wrote:
Get the information on privileged group IDs during rpmsg probing based
on DT property. Check if the process requesting an offload to remote
subsystem is privileged by comparing it's group ID with privileged
group ID. Initialization process attributes are updated for a
privileged process which is sent to remote process for resource
management.




+
  static const struct file_operations fastrpc_fops = {
  	.open = fastrpc_device_open,
  	.release = fastrpc_device_release,
@@ -2277,6 +2396,10 @@ static int fastrpc_rpmsg_probe(struct rpmsg_device *rpdev)
  	if (!data)
  		return -ENOMEM;
+ err = fastrpc_init_privileged_gids(rdev, "qcom,fastrpc-gids", &data->gidlist);
+	if (err)
+		dev_err(rdev, "Privileged gids init failed.\n");
+

What about error paths? No need for cleanup?

All the necessary clean-up is added as part of fastrpc_init_privileged_gids error path. There is no requirement to have any additional handling in error path other that error log. Also there is no intention to fail the probe in case gid information is not properly read.

Thanks,
Ekansh
Best regards,
Krzysztof




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux