From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx3.redhat.com (mx3.redhat.com [172.16.48.32]) by int-mx1.corp.redhat.com (8.13.1/8.13.1) with ESMTP id m3NI4VC5000956 for ; Wed, 23 Apr 2008 14:04:31 -0400 Received: from mail.hauppauge.com (mail.hauppauge.com [167.206.143.4]) by mx3.redhat.com (8.13.8/8.13.8) with ESMTP id m3NI4Isu008596 for ; Wed, 23 Apr 2008 14:04:20 -0400 Message-ID: <480F7A0E.2080202@linuxtv.org> From: mkrufky@linuxtv.org To: mchehab@infradead.org Date: Wed, 23 Apr 2008 14:03:58 -0400 MIME-Version: 1.0 in-reply-to: <20080423143454.0d50b209@gaivota> Content-Type: text/plain; charset="iso-8859-1" Cc: video4linux-list@redhat.com, linux-kernel@vger.kernel.org, ivtv-devel@ivtvdriver.org, biercenator@gmail.com, alan@lxorguk.ukuu.org.uk Subject: Re: [PATCH] Fix VIDIOCGAP corruption in ivtv List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: video4linux-list-bounces@redhat.com Errors-To: video4linux-list-bounces@redhat.com List-ID: Mauro Carvalho Chehab wrote: > On Sun, 20 Apr 2008 12:10:00 -0400 > Michael Krufky wrote: > > >> Hans Verkuil wrote: >> >>> On Sunday 20 April 2008 13:27:36 Alan Cox wrote: >>> >>> >>>> Frank Bennett reported that ivtv was causing skype to crash. With >>>> help from one of their developers he showed it was a kernel problem. >>>> VIDIOCGCAP copies a name into a fixed length buffer - ivtv uses names >>>> that are too long and does not truncate them so corrupts a few bytes >>>> of the app data area. >>>> >>>> Possibly the names also want trimming but for now this should fix the >>>> corruption case. >>>> >>>> >>> Ouch, nasty one. >>> >>> Mauro, can you apply this patch to the v4l-dvb master? >>> >>> Mike, this one should obviously go into a 2.6.25 dot-release, and I >>> think also to a 2.6.24 dot-release. >>> >>> Frank, thank you for reporting this! >>> >>> Hans >>> >>> Signed-off-by: Hans Verkuil >>> >>> >> Signed-off-by: Michael Krufky >> > > In this case, it should be reviewed-by. > > I should be sending this soon to Linus. Mauro, I think it's wrong that you alter signatures. You did this: Reviewed-by: > But I provided this: Signed-off-by: Michael Krufky I sign the patch, because I have handled it in my -stable queue. You put my email address without my name! A signature is sacred and nobody should fudge it. Please fix it before you send to -git. -Mike -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list