> 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
