* [PATCH] rpmsg: char: release allocated memory
@ 2019-09-17 3:31 Navid Emamdoost
2019-11-21 18:20 ` Navid Emamdoost
0 siblings, 1 reply; 4+ messages in thread
From: Navid Emamdoost @ 2019-09-17 3:31 UTC (permalink / raw)
Cc: emamd001, smccaman, kjlu, Navid Emamdoost, Ohad Ben-Cohen,
Bjorn Andersson, linux-remoteproc, linux-kernel
In rpmsg_eptdev_write_iter, if copy_from_iter_full fails the allocated
buffer needs to be released.
Signed-off-by: Navid Emamdoost <navid.emamdoost@gmail.com>
---
drivers/rpmsg/rpmsg_char.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/rpmsg/rpmsg_char.c b/drivers/rpmsg/rpmsg_char.c
index eea5ebbb5119..c655074c07c2 100644
--- a/drivers/rpmsg/rpmsg_char.c
+++ b/drivers/rpmsg/rpmsg_char.c
@@ -227,8 +227,10 @@ static ssize_t rpmsg_eptdev_write_iter(struct kiocb *iocb,
if (!kbuf)
return -ENOMEM;
- if (!copy_from_iter_full(kbuf, len, from))
- return -EFAULT;
+ if (!copy_from_iter_full(kbuf, len, from)) {
+ ret = -EFAULT;
+ goto free_kbuf;
+ }
if (mutex_lock_interruptible(&eptdev->ept_lock)) {
ret = -ERESTARTSYS;
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] rpmsg: char: release allocated memory
2019-09-17 3:31 [PATCH] rpmsg: char: release allocated memory Navid Emamdoost
@ 2019-11-21 18:20 ` Navid Emamdoost
2019-11-21 23:35 ` Bjorn Andersson
0 siblings, 1 reply; 4+ messages in thread
From: Navid Emamdoost @ 2019-11-21 18:20 UTC (permalink / raw)
To: Ohad Ben-Cohen, Bjorn Andersson, linux-remoteproc; +Cc: Navid Emamdoost, LKML
On Mon, Sep 16, 2019 at 10:31 PM Navid Emamdoost
<navid.emamdoost@gmail.com> wrote:
>
> In rpmsg_eptdev_write_iter, if copy_from_iter_full fails the allocated
> buffer needs to be released.
>
> Signed-off-by: Navid Emamdoost <navid.emamdoost@gmail.com>
Would you please review this patch?
Thank you,
> ---
> drivers/rpmsg/rpmsg_char.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/rpmsg/rpmsg_char.c b/drivers/rpmsg/rpmsg_char.c
> index eea5ebbb5119..c655074c07c2 100644
> --- a/drivers/rpmsg/rpmsg_char.c
> +++ b/drivers/rpmsg/rpmsg_char.c
> @@ -227,8 +227,10 @@ static ssize_t rpmsg_eptdev_write_iter(struct kiocb *iocb,
> if (!kbuf)
> return -ENOMEM;
>
> - if (!copy_from_iter_full(kbuf, len, from))
> - return -EFAULT;
> + if (!copy_from_iter_full(kbuf, len, from)) {
> + ret = -EFAULT;
> + goto free_kbuf;
> + }
>
> if (mutex_lock_interruptible(&eptdev->ept_lock)) {
> ret = -ERESTARTSYS;
> --
> 2.17.1
>
--
Navid.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] rpmsg: char: release allocated memory
2019-11-21 18:20 ` Navid Emamdoost
@ 2019-11-21 23:35 ` Bjorn Andersson
2019-11-21 23:54 ` Navid Emamdoost
0 siblings, 1 reply; 4+ messages in thread
From: Bjorn Andersson @ 2019-11-21 23:35 UTC (permalink / raw)
To: Navid Emamdoost; +Cc: Ohad Ben-Cohen, linux-remoteproc, Navid Emamdoost, LKML
On Thu 21 Nov 10:20 PST 2019, Navid Emamdoost wrote:
> On Mon, Sep 16, 2019 at 10:31 PM Navid Emamdoost
> <navid.emamdoost@gmail.com> wrote:
> >
> > In rpmsg_eptdev_write_iter, if copy_from_iter_full fails the allocated
> > buffer needs to be released.
> >
> > Signed-off-by: Navid Emamdoost <navid.emamdoost@gmail.com>
>
> Would you please review this patch?
>
I'm sorry, I must have missed to reply as I applied the patch. You can
find it in the rpmsg-next branch [1] and it will be part of the pull
request for v5.5
[1] https://git.kernel.org/pub/scm/linux/kernel/git/andersson/remoteproc.git/log/?h=rpmsg-next
Regards,
Bjorn
> Thank you,
>
> > ---
> > drivers/rpmsg/rpmsg_char.c | 6 ++++--
> > 1 file changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/rpmsg/rpmsg_char.c b/drivers/rpmsg/rpmsg_char.c
> > index eea5ebbb5119..c655074c07c2 100644
> > --- a/drivers/rpmsg/rpmsg_char.c
> > +++ b/drivers/rpmsg/rpmsg_char.c
> > @@ -227,8 +227,10 @@ static ssize_t rpmsg_eptdev_write_iter(struct kiocb *iocb,
> > if (!kbuf)
> > return -ENOMEM;
> >
> > - if (!copy_from_iter_full(kbuf, len, from))
> > - return -EFAULT;
> > + if (!copy_from_iter_full(kbuf, len, from)) {
> > + ret = -EFAULT;
> > + goto free_kbuf;
> > + }
> >
> > if (mutex_lock_interruptible(&eptdev->ept_lock)) {
> > ret = -ERESTARTSYS;
> > --
> > 2.17.1
> >
>
>
> --
> Navid.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] rpmsg: char: release allocated memory
2019-11-21 23:35 ` Bjorn Andersson
@ 2019-11-21 23:54 ` Navid Emamdoost
0 siblings, 0 replies; 4+ messages in thread
From: Navid Emamdoost @ 2019-11-21 23:54 UTC (permalink / raw)
To: Bjorn Andersson; +Cc: Ohad Ben-Cohen, linux-remoteproc, Navid Emamdoost, LKML
Thanks for the update.
On Thu, Nov 21, 2019 at 5:35 PM Bjorn Andersson
<bjorn.andersson@linaro.org> wrote:
>
> On Thu 21 Nov 10:20 PST 2019, Navid Emamdoost wrote:
>
> > On Mon, Sep 16, 2019 at 10:31 PM Navid Emamdoost
> > <navid.emamdoost@gmail.com> wrote:
> > >
> > > In rpmsg_eptdev_write_iter, if copy_from_iter_full fails the allocated
> > > buffer needs to be released.
> > >
> > > Signed-off-by: Navid Emamdoost <navid.emamdoost@gmail.com>
> >
> > Would you please review this patch?
> >
>
> I'm sorry, I must have missed to reply as I applied the patch. You can
> find it in the rpmsg-next branch [1] and it will be part of the pull
> request for v5.5
>
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/andersson/remoteproc.git/log/?h=rpmsg-next
>
> Regards,
> Bjorn
>
> > Thank you,
> >
> > > ---
> > > drivers/rpmsg/rpmsg_char.c | 6 ++++--
> > > 1 file changed, 4 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/rpmsg/rpmsg_char.c b/drivers/rpmsg/rpmsg_char.c
> > > index eea5ebbb5119..c655074c07c2 100644
> > > --- a/drivers/rpmsg/rpmsg_char.c
> > > +++ b/drivers/rpmsg/rpmsg_char.c
> > > @@ -227,8 +227,10 @@ static ssize_t rpmsg_eptdev_write_iter(struct kiocb *iocb,
> > > if (!kbuf)
> > > return -ENOMEM;
> > >
> > > - if (!copy_from_iter_full(kbuf, len, from))
> > > - return -EFAULT;
> > > + if (!copy_from_iter_full(kbuf, len, from)) {
> > > + ret = -EFAULT;
> > > + goto free_kbuf;
> > > + }
> > >
> > > if (mutex_lock_interruptible(&eptdev->ept_lock)) {
> > > ret = -ERESTARTSYS;
> > > --
> > > 2.17.1
> > >
> >
> >
> > --
> > Navid.
--
Navid.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-11-21 23:54 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-17 3:31 [PATCH] rpmsg: char: release allocated memory Navid Emamdoost
2019-11-21 18:20 ` Navid Emamdoost
2019-11-21 23:35 ` Bjorn Andersson
2019-11-21 23:54 ` Navid Emamdoost
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).