From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail.kernel.org ([198.145.29.99]:38666 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726158AbeKZSZg (ORCPT ); Mon, 26 Nov 2018 13:25:36 -0500 Date: Mon, 26 Nov 2018 08:32:20 +0100 From: Greg Kroah-Hartman To: Sakari Ailus Cc: Dave Stevenson , Hans Verkuil , mchehab@kernel.org, linux-media@vger.kernel.org, Sasha Levin Subject: Re: [PATCH v2 for v4.9 1/1] v4l: event: Add subscription to list before calling "add" operation Message-ID: <20181126073220.GC18375@kroah.com> References: <20181114093753.29081-1-sakari.ailus@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20181114093753.29081-1-sakari.ailus@linux.intel.com> Sender: linux-media-owner@vger.kernel.org List-ID: On Wed, Nov 14, 2018 at 11:37:53AM +0200, Sakari Ailus wrote: > [ upstream commit 92539d3eda2c090b382699bbb896d4b54e9bdece ] > > Patch ad608fbcf166 changed how events were subscribed to address an issue > elsewhere. As a side effect of that change, the "add" callback was called > before the event subscription was added to the list of subscribed events, > causing the first event queued by the add callback (and possibly other > events arriving soon afterwards) to be lost. > > Fix this by adding the subscription to the list before calling the "add" > callback, and clean up afterwards if that fails. > > Fixes: ad608fbcf166 ("media: v4l: event: Prevent freeing event subscriptions while accessed") > > Reported-by: Dave Stevenson > Signed-off-by: Sakari Ailus > Tested-by: Dave Stevenson > Reviewed-by: Hans Verkuil > Tested-by: Hans Verkuil > Signed-off-by: Mauro Carvalho Chehab > [Sakari Ailus: Backported to v4.9 stable] > Signed-off-by: Sakari Ailus > --- > since v1 (as requested by Sasha): > > - Add my final SoB > - Indicate specifically this is a backport > - Remove the extra cc stable Now queued up, thanks. greg k-h