Received: by 2002:a25:8b91:0:0:0:0:0 with SMTP id j17csp4195116ybl; Tue, 21 Jan 2020 14:54:31 -0800 (PST) X-Google-Smtp-Source: APXvYqxte48zx7iGJ5ka8lOgHj9a71iyfbXEGVnJlL7AFp/h6DjoD7n+e+2ivNg4ktH7zGW8eXVJ X-Received: by 2002:a05:6830:1607:: with SMTP id g7mr5297681otr.320.1579647271605; Tue, 21 Jan 2020 14:54:31 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1579647271; cv=none; d=google.com; s=arc-20160816; b=HvlS7OPzlAKKXP513lDGSC9EVbvCwqONJNXRB5w+s2C9QOldsO5jDrcnelDMOqtmso LY5S/1eeI6ztIaDDqprVRIhNcSeC6B4NzwEOSEUKV/we4BIekaDstchRv5fKfQ405R16 XpEpmYTuGfuLygTS2KB65YLmUpwN/7IkQqxrVcqUW59NibtxAKuKvQP3HzeiRpjk85cx fovaCEbRbaMZuTdEJ9jdAtsS9SPgMOY1aHgXUooYN5S74FitlSGgZDu7TWnFXSrOWDtm /eN7NGgXxX4gUOV/csw5zem9yB9Cj+p6WJLDVNGy2u9erMQWrH6MlmpDTvvmsoBUQHwP 7QWw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=P7AjEb1vJESNVYGfeCUxnog2kOXe2LSJN4ECRhlNJ8s=; b=TvN3dbFoCF3sXAv/3gHrMNc3LsMI4g6blWlcv1O/+XTVYcv96KPnbnXksm9fHxoVNi LrBMaAU4UXIIS31vk1CW7D1PcY6TuAJBx4zv7WEnKhHgVpKlEuqVHiWz2HcsNIEb/Vve rT4wsnoSKdwIWUAEkXCeId/s6b6zIZONm3IFqncAkPybqQVPUrJQ/8jv1L2lol7YCK8j kLWT5/AB3uLTR2M4JDKFXEQKc6Js3Xm8DUBEWr3tmV+N7tIcdTtZiygImTqPnznU3Fzw NghMb8jobo8UjUV5l36IALpk5cP88owfNIa4fafTbwrUfFouDr4U/8Q8MwmwDA5I69G/ ET4g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass (test mode) header.i=@ideasonboard.com header.s=mail header.b=nq4aFWpu; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id r4si23645012otd.50.2020.01.21.14.54.19; Tue, 21 Jan 2020 14:54:31 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass (test mode) header.i=@ideasonboard.com header.s=mail header.b=nq4aFWpu; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726889AbgAUWxY (ORCPT + 99 others); Tue, 21 Jan 2020 17:53:24 -0500 Received: from perceval.ideasonboard.com ([213.167.242.64]:33276 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725876AbgAUWxY (ORCPT ); Tue, 21 Jan 2020 17:53:24 -0500 Received: from pendragon.ideasonboard.com (81-175-216-236.bb.dnainternet.fi [81.175.216.236]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id C1C9052A; Tue, 21 Jan 2020 23:53:21 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1579647201; bh=+o9lAebGcfIPvu6BwlMSFfOT4pdXPhZxgEAsy4SNm2E=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=nq4aFWpuAjtBfYJ8ciGIrsSYLe2oU+g5nTBkSi/7WHs22iD/iSDNauv5gn19he62S qUPdeLcyTH/uZojPqjxZV4KNVbGBMwNbZR0HdjB1ciQLxMs/lcjLxBj+hh2Ydq9pT3 p9wFPeq8MIE1hZVJlaSX4Ez83OSqRgw1rjRGVeSI= Date: Wed, 22 Jan 2020 00:53:05 +0200 From: Laurent Pinchart To: Will Deacon Cc: Andrey Konovalov , Greg Kroah-Hartman , linux-media@vger.kernel.org, LKML , Andrew Morton , Mauro Carvalho Chehab , Dmitry Vyukov , Kostya Serebryany , stable Subject: Re: [PATCH RESEND RESEND] media: uvc: Avoid cyclic entity chains due to malformed USB descriptors Message-ID: <20200121225305.GL5003@pendragon.ideasonboard.com> References: <20191108154838.21487-1-will@kernel.org> <20191108155503.GB15731@pendragon.ideasonboard.com> <20191216121651.GA12947@willie-the-truck> <20191218114137.GA15505@willie-the-truck> <20191218122324.GB17086@kroah.com> <20191218165153.GC17876@pendragon.ideasonboard.com> <20200121190142.GB13592@willie-the-truck> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20200121190142.GB13592@willie-the-truck> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Will, On Tue, Jan 21, 2020 at 07:01:42PM +0000, Will Deacon wrote: > On Wed, Dec 18, 2019 at 06:51:53PM +0200, Laurent Pinchart wrote: > > On Wed, Dec 18, 2019 at 01:46:00PM +0100, Andrey Konovalov wrote: > > > On Wed, Dec 18, 2019 at 1:23 PM Greg Kroah-Hartman wrote: > > > > On Wed, Dec 18, 2019 at 11:41:38AM +0000, Will Deacon wrote: > > > >> On Mon, Dec 16, 2019 at 02:17:52PM +0100, Andrey Konovalov wrote: > > > >>> On Mon, Dec 16, 2019 at 1:16 PM Will Deacon wrote: > > > >>>> On Fri, Nov 08, 2019 at 05:55:03PM +0200, Laurent Pinchart wrote: > > > >>>>> Thank you for the patch. > > > >>>>> > > > >>>>> I'm sorry for the delay, and will have to ask you to be a bit more > > > >>>>> patient I'm afraid. I will leave tomorrow for a week without computer > > > >>>>> access and will only be able to go through my backlog when I will be > > > >>>>> back on the 17th. > > > >>>> > > > >>>> Gentle reminder on this, now you've been back a month ;) > > > >>> > > > >>> I think we now have a reproducer for this issue that syzbot just reported: > > > >>> > > > >>> https://syzkaller.appspot.com/bug?extid=0a5c96772a9b26f2a876 > > > >>> > > > >>> You can try you patch on it :) > > > >> > > > >> Oh wow, I *really* like the raw USB gadget thingy you have to reproduce > > > >> these! I also really like that this patch fixes the issue. Logs below. > > > > > > Thanks! An easier way to test the patch would be to issue a syz test > > > command, but I'm glad you managed to set up raw gadget manually and it > > > worked for you. > > > > > > > > > > > Ok, that's a good poke for me to go review that raw gadget code to see > > > > if it can be merged upstream :) > > > > > > Looking forward to it! =) > > > > Reviewed-by: Laurent Pinchart > > > > and merged in my tree. I'm so sorry for the way too long delay. > > Please can you send this upstream and/or put it in linux-next? I can't see > it anywhere at the moment :( I've now sent the pull request. Seems I failed the schedule from A to Z with this patch. I'm extremely sorry :-( -- Regards, Laurent Pinchart