From: Dan Carpenter <dan.carpenter@oracle.com>
To: Johan Hovold <johan@kernel.org>
Cc: Jared Kangas <kangas.jd@gmail.com>,
vaibhav.sr@gmail.com, elder@kernel.org,
gregkh@linuxfoundation.org, greybus-dev@lists.linaro.org,
linux-kernel@vger.kernel.org, stable@vger.kernel.org,
linux-staging@lists.linux.dev, mgreer@animalcreek.com
Subject: Re: [PATCH v2] staging: greybus: audio: fix loop cursor use after iteration
Date: Fri, 10 Jun 2022 11:06:27 +0300 [thread overview]
Message-ID: <20220610080627.GA2146@kadam> (raw)
In-Reply-To: <YqL6A3pVC8LOqE4d@hovoldconsulting.com>
On Fri, Jun 10, 2022 at 10:00:03AM +0200, Johan Hovold wrote:
> > Fixes: 510e340efe0c ("staging: greybus: audio: Add helper APIs for dynamic audio modules")
> > Cc: stable@vger.kernel.org
> > Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
> > Reviewed-by: Johan Hovold <johan@kernel.org>
> > Signed-off-by: Jared Kangas <kangas.jd@gmail.com>
> > ---
> >
> > Changes since v1:
> > * Removed safe list iteration as suggested by Johan Hovold <johan@kernel.org>
> > * Updated patch changelog to explain the list iteration change
> > * Added tags to changelog based on feedback (Cc:, Fixes:, Reviewed-by:)
>
> Apparently Greg applied this to staging-next before we had a change to
> look at it. You should have received a notification from Greg when he
> did so.
>
> https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/commit/?h=staging-next&id=80c968a04a381dc0e690960c60ffd6b6aee7e157
>
> It seems unlikely that this would cause any issues in real life, but
> there's still a chance it will be picked up by the stable team despite
> the lack of a CC stable tag.
If you want you can always email the stable team to pick up the patch.
regards,
dan carpenter
next prev parent reply other threads:[~2022-06-10 8:06 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-09 21:45 [PATCH v2] staging: greybus: audio: fix loop cursor use after iteration Jared Kangas
2022-06-10 7:03 ` Greg KH
2022-06-10 8:00 ` Johan Hovold
2022-06-10 8:06 ` Dan Carpenter [this message]
2022-06-10 8:16 ` [greybus-dev] " Johan Hovold
2022-06-10 14:56 ` Jared Kangas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220610080627.GA2146@kadam \
--to=dan.carpenter@oracle.com \
--cc=elder@kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=greybus-dev@lists.linaro.org \
--cc=johan@kernel.org \
--cc=kangas.jd@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=mgreer@animalcreek.com \
--cc=stable@vger.kernel.org \
--cc=vaibhav.sr@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.