From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx2.suse.de ([195.135.220.15]:58271 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752453AbeEOKAI (ORCPT ); Tue, 15 May 2018 06:00:08 -0400 Message-ID: <1526378007.2724.2.camel@suse.com> Subject: Re: [PATCH 4.4 49/72] media: usbtv: prevent double free in error case From: Oliver Neukum To: Ben Hutchings , Hans Verkuil , Mauro Carvalho Chehab Cc: Tuba Yavuz , GregKroah-Hartman , LKML , stable@vger.kernel.org Date: Tue, 15 May 2018 11:53:27 +0200 In-Reply-To: <1526319669.9159.20.camel@codethink.co.uk> References: <20180406084305.210085169@linuxfoundation.org> <20180406084309.317246829@linuxfoundation.org> <1526319669.9159.20.camel@codethink.co.uk> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: stable-owner@vger.kernel.org List-ID: Am Montag, den 14.05.2018, 18:41 +0100 schrieb Ben Hutchings: > > Shouldn't this bump the V4L2 device's reference count, i.e.: > > v4l2_device_get(&usbtv->v4l2_dev); > You are right. Mixing refcounting sucks. I am preparing a patch. Regards Oliver