Thread (1 message) 1 message, 1 author, 2021-06-16

Re: [PATCH v4 4/4] remoteproc: core: Cleanup device in case of failure

From: Greg KH <hidden>
Date: 2021-06-16 05:57:06
Also in: linux-arm-msm, linux-remoteproc, lkml, stable

On Tue, Jun 15, 2021 at 01:21:11PM -0700, Siddharth Gupta wrote:
On 6/15/2021 12:06 PM, Greg KH wrote:
quoted
On Tue, Jun 15, 2021 at 12:03:44PM -0700, Siddharth Gupta wrote:
quoted
When a failure occurs in rproc_add() it returns an error, but does
not cleanup after itself. This change adds the failure path in such
cases.

Signed-off-by: Siddharth Gupta <redacted>
Cc: stable@vger.kernel.org
---
  drivers/remoteproc/remoteproc_core.c | 15 ++++++++++++---
  1 file changed, 12 insertions(+), 3 deletions(-)
Why is this needed for stable kernels?  And again, a Fixes: tag?
Patch 2 and patch 3 are leading up to fix rproc_add()
in case of a failure. This means we'll have errors with
use after free unless we call device_del() or cdev_del(),
also the sysfs and devtempfs nodes will also not be
removed.
Then please explain that better in the changelogs.  At it is, no one
knows this.

greg k-h

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help