Re: [PATCH] misc: fastrpc: fix memory leak

From: Markus Elfring
Date: Wed Apr 29 2020 - 16:40:20 EST


> if misc_register() fails, previously allocated data is left without freeing,
> this could result in memory leak.

How do you think about a wording variant like the following?

Subject:
[PATCH v2] misc: fastrpc: Fix an incomplete memory release in fastrpc_rpmsg_probe()

Change description:
A few fastrpc data were not released after a call of the
function âmisc_registerâ failed.
Thus add a call of the function âkfreeâ.


> So fix it!

Would you like to replace such an information by the tag âFixesâ?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?id=96c9a7802af7d500a582d89a8b864584fe878c1b#n183

Regards,
Markus