* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
@ 2008-04-23 17:34 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 21+ messages in thread
From: Mauro Carvalho Chehab @ 2008-04-23 17:34 UTC (permalink / raw)
To: Michael Krufky
Cc: video4linux-list, linux-kernel, ivtv-devel, Frank Bennett,
Alan Cox
On Sun, 20 Apr 2008 12:10:00 -0400
Michael Krufky <mkrufky@linuxtv.org> 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 <hverkuil@xs4all.nl>
> >
>
> Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
In this case, it should be reviewed-by.
I should be sending this soon to Linus.
Cheers,
Mauro
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
2008-04-23 17:34 ` Mauro Carvalho Chehab
@ 2008-04-23 18:03 ` mkrufky
-1 siblings, 0 replies; 21+ messages in thread
From: mkrufky @ 2008-04-23 18:03 UTC (permalink / raw)
To: mchehab
Cc: hverkuil, video4linux-list, alan, linux-kernel, ivtv-devel,
biercenator
Mauro Carvalho Chehab wrote:
> On Sun, 20 Apr 2008 12:10:00 -0400
> Michael Krufky <mkrufky@linuxtv.org> 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 <hverkuil@xs4all.nl>
>>>
>>>
>> Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
>>
>
> 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: <mkrufky@linuxtv.org <mailto:mkrufky@linuxtv.org>>
But I provided this:
Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
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
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
@ 2008-04-23 18:03 ` mkrufky
0 siblings, 0 replies; 21+ messages in thread
From: mkrufky @ 2008-04-23 18:03 UTC (permalink / raw)
To: mchehab; +Cc: video4linux-list, linux-kernel, ivtv-devel, biercenator, alan
Mauro Carvalho Chehab wrote:
> On Sun, 20 Apr 2008 12:10:00 -0400
> Michael Krufky <mkrufky@linuxtv.org> 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 <hverkuil@xs4all.nl>
>>>
>>>
>> Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
>>
>
> 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: <mkrufky@linuxtv.org <mailto:mkrufky@linuxtv.org>>
But I provided this:
Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
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
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
2008-04-23 18:03 ` mkrufky
@ 2008-04-24 2:24 ` Mauro Carvalho Chehab
-1 siblings, 0 replies; 21+ messages in thread
From: Mauro Carvalho Chehab @ 2008-04-24 2:24 UTC (permalink / raw)
To: mkrufky; +Cc: video4linux-list, linux-kernel, ivtv-devel, biercenator, Alan Cox
>>> Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
>>
>> 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: <mkrufky@linuxtv.org <mailto:mkrufky@linuxtv.org>>
Argh! I didn't notice that weird stuff. This should be fixed before
forwarding the patch. Sorry for the mess.
> But I provided this:
>
> Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
>
> I sign the patch, because I have handled it in my -stable queue.
SOB is the proper tag for your -stable queue. However, asking me to add
this to my tree is not what it is expected.
SOB is meant to track the patch history until it reaches the kernel. So,
the first SOB(s) is(are) from its author(s). The author(s) will send this
to a driver maintainer, that will send to a subsystem maintainer, etc,
until reach mainstream.
In the case of this patch, it was sent to the ML. Hans picked it, as the
maintainer of ivtv, and asked me to pull.
You didn't wrote the patch, not forwarded it to me, so, the tag doesn't
apply on my tree.
After the merge at mainstream, you'll send this to stable. In this case,
you'll take it from my tree, add your SOB, and forward it.
Unfortunately, it is a common mistake of people sending SOB's instead of
acked-by or reviewed-by, so several maintainers warns about the improper
usage of the tags and corrects it on their trees, or simply ignores such
improper usage.
If you prefer, I may just drop the tag from my tree, or fix the
"reviewed-by".
Cheers,
Mauro.
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
@ 2008-04-24 2:24 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 21+ messages in thread
From: Mauro Carvalho Chehab @ 2008-04-24 2:24 UTC (permalink / raw)
To: mkrufky; +Cc: Alan Cox, video4linux-list, biercenator, linux-kernel, ivtv-devel
>>> Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
>>
>> 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: <mkrufky@linuxtv.org <mailto:mkrufky@linuxtv.org>>
Argh! I didn't notice that weird stuff. This should be fixed before
forwarding the patch. Sorry for the mess.
> But I provided this:
>
> Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
>
> I sign the patch, because I have handled it in my -stable queue.
SOB is the proper tag for your -stable queue. However, asking me to add
this to my tree is not what it is expected.
SOB is meant to track the patch history until it reaches the kernel. So,
the first SOB(s) is(are) from its author(s). The author(s) will send this
to a driver maintainer, that will send to a subsystem maintainer, etc,
until reach mainstream.
In the case of this patch, it was sent to the ML. Hans picked it, as the
maintainer of ivtv, and asked me to pull.
You didn't wrote the patch, not forwarded it to me, so, the tag doesn't
apply on my tree.
After the merge at mainstream, you'll send this to stable. In this case,
you'll take it from my tree, add your SOB, and forward it.
Unfortunately, it is a common mistake of people sending SOB's instead of
acked-by or reviewed-by, so several maintainers warns about the improper
usage of the tags and corrects it on their trees, or simply ignores such
improper usage.
If you prefer, I may just drop the tag from my tree, or fix the
"reviewed-by".
Cheers,
Mauro.
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
2008-04-24 2:24 ` Mauro Carvalho Chehab
@ 2008-04-24 3:03 ` Michael Krufky
-1 siblings, 0 replies; 21+ messages in thread
From: Michael Krufky @ 2008-04-24 3:03 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: video4linux-list, linux-kernel, ivtv-devel, biercenator, Alan Cox
Mauro Carvalho Chehab wrote:
>>>> Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
>>>
>>> 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: <mkrufky@linuxtv.org <mailto:mkrufky@linuxtv.org>>
>
> Argh! I didn't notice that weird stuff. This should be fixed before
> forwarding the patch. Sorry for the mess.
>
> [snip]
>
> After the merge at mainstream, you'll send this to stable. In this
> case, you'll take it from my tree, add your SOB, and forward it.
>
> Unfortunately, it is a common mistake of people sending SOB's instead
> of acked-by or reviewed-by, so several maintainers warns about the
> improper usage of the tags and corrects it on their trees, or simply
> ignores such improper usage.
Good point.
>
> If you prefer, I may just drop the tag from my tree, or fix the
> "reviewed-by".
Perhaps it makes most sense to just drop my tag, in this case -- that's
fine with me.
Regards,
Mike
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
@ 2008-04-24 3:03 ` Michael Krufky
0 siblings, 0 replies; 21+ messages in thread
From: Michael Krufky @ 2008-04-24 3:03 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: Alan Cox, video4linux-list, biercenator, linux-kernel, ivtv-devel
Mauro Carvalho Chehab wrote:
>>>> Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
>>>
>>> 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: <mkrufky@linuxtv.org <mailto:mkrufky@linuxtv.org>>
>
> Argh! I didn't notice that weird stuff. This should be fixed before
> forwarding the patch. Sorry for the mess.
>
> [snip]
>
> After the merge at mainstream, you'll send this to stable. In this
> case, you'll take it from my tree, add your SOB, and forward it.
>
> Unfortunately, it is a common mistake of people sending SOB's instead
> of acked-by or reviewed-by, so several maintainers warns about the
> improper usage of the tags and corrects it on their trees, or simply
> ignores such improper usage.
Good point.
>
> If you prefer, I may just drop the tag from my tree, or fix the
> "reviewed-by".
Perhaps it makes most sense to just drop my tag, in this case -- that's
fine with me.
Regards,
Mike
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
2008-04-23 17:34 ` Mauro Carvalho Chehab
(?)
(?)
@ 2008-04-23 19:26 ` Frank Bennett
2008-04-24 15:47 ` Mauro Carvalho Chehab
-1 siblings, 1 reply; 21+ messages in thread
From: Frank Bennett @ 2008-04-23 19:26 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: Michael Krufky, Hans Verkuil, video4linux-list, Alan Cox,
linux-kernel, ivtv-devel
Maruo,
I don't want to make your life more complicated than necessary, but
while we're on the topic of attribution ...
The real work in identifying this issue was done by Andrew Macks, the
engineer at Skype. My role in the affair consisted of complaining,
sending along a log file, recompiling the kernel, and writing an email
message.
I relayed the initial response I received from Hans Verkuil to Andrew
(via skype chat, I do not have an email address for him), to let him
know that the problem was being addressed in the kernel, and he was
glad to hear the news. But watching things unfold, I have been feeling
slightly incomfortable that only my name might end up in the chain of
correspondence, and not his. I would just like to slip in a note here
to that effect.
Frank Bennett
On Thu, Apr 24, 2008 at 2:34 AM, Mauro Carvalho Chehab
<mchehab@infradead.org> wrote:
> On Sun, 20 Apr 2008 12:10:00 -0400
> Michael Krufky <mkrufky@linuxtv.org> 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 <hverkuil@xs4all.nl>
> > >
> >
> > Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
>
> In this case, it should be reviewed-by.
>
> I should be sending this soon to Linus.
>
> Cheers,
> Mauro
>
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
2008-04-23 19:26 ` Frank Bennett
@ 2008-04-24 15:47 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 21+ messages in thread
From: Mauro Carvalho Chehab @ 2008-04-24 15:47 UTC (permalink / raw)
To: Frank Bennett, Alan Cox
Cc: Michael Krufky, Hans Verkuil, video4linux-list, linux-kernel,
ivtv-devel
Hi Frank/Alan,
On Thu, 24 Apr 2008 04:26:46 +0900
"Frank Bennett" <biercenator@gmail.com> wrote:
> Maruo,
>
> I don't want to make your life more complicated than necessary, but
> while we're on the topic of attribution ...
>
> The real work in identifying this issue was done by Andrew Macks, the
> engineer at Skype. My role in the affair consisted of complaining,
> sending along a log file, recompiling the kernel, and writing an email
> message.
>
> I relayed the initial response I received from Hans Verkuil to Andrew
> (via skype chat, I do not have an email address for him), to let him
> know that the problem was being addressed in the kernel, and he was
> glad to hear the news. But watching things unfold, I have been feeling
> slightly incomfortable that only my name might end up in the chain of
> correspondence, and not his. I would just like to slip in a note here
> to that effect.
Maybe we can just add his name in parenthesis. Would this patch description be
ok for you, Andrew and Alan?
Fix VIDIOCGAP corruption in ivtv
From: Alan Cox <alan@redhat.com>
Frank Bennett reported that ivtv was causing skype to crash. With help
from one of their developers (Andrew Macks) 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.
Signed-off-by: Alan Cox <alan@redhat.com>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Cheers,
Mauro
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
@ 2008-04-24 15:47 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 21+ messages in thread
From: Mauro Carvalho Chehab @ 2008-04-24 15:47 UTC (permalink / raw)
To: Frank Bennett, Alan Cox
Cc: video4linux-list, Michael Krufky, linux-kernel, ivtv-devel
Hi Frank/Alan,
On Thu, 24 Apr 2008 04:26:46 +0900
"Frank Bennett" <biercenator@gmail.com> wrote:
> Maruo,
>
> I don't want to make your life more complicated than necessary, but
> while we're on the topic of attribution ...
>
> The real work in identifying this issue was done by Andrew Macks, the
> engineer at Skype. My role in the affair consisted of complaining,
> sending along a log file, recompiling the kernel, and writing an email
> message.
>
> I relayed the initial response I received from Hans Verkuil to Andrew
> (via skype chat, I do not have an email address for him), to let him
> know that the problem was being addressed in the kernel, and he was
> glad to hear the news. But watching things unfold, I have been feeling
> slightly incomfortable that only my name might end up in the chain of
> correspondence, and not his. I would just like to slip in a note here
> to that effect.
Maybe we can just add his name in parenthesis. Would this patch description be
ok for you, Andrew and Alan?
Fix VIDIOCGAP corruption in ivtv
From: Alan Cox <alan@redhat.com>
Frank Bennett reported that ivtv was causing skype to crash. With help
from one of their developers (Andrew Macks) 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.
Signed-off-by: Alan Cox <alan@redhat.com>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Cheers,
Mauro
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
2008-04-24 15:47 ` Mauro Carvalho Chehab
@ 2008-04-24 16:00 ` Alan Cox
-1 siblings, 0 replies; 21+ messages in thread
From: Alan Cox @ 2008-04-24 16:00 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: Frank Bennett, Alan Cox, video4linux-list, Michael Krufky,
linux-kernel, ivtv-devel
On Thu, Apr 24, 2008 at 12:47:08PM -0300, Mauro Carvalho Chehab wrote:
> Maybe we can just add his name in parenthesis. Would this patch description be
> ok for you, Andrew and Alan?
I have no idea if Mr Macks did the work or minds his name being used so I
can form no view on the change. Obviously please remove my sign off if you
make the change
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
@ 2008-04-24 16:00 ` Alan Cox
0 siblings, 0 replies; 21+ messages in thread
From: Alan Cox @ 2008-04-24 16:00 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: video4linux-list, linux-kernel, ivtv-devel, Michael Krufky,
Frank Bennett, Alan Cox
On Thu, Apr 24, 2008 at 12:47:08PM -0300, Mauro Carvalho Chehab wrote:
> Maybe we can just add his name in parenthesis. Would this patch description be
> ok for you, Andrew and Alan?
I have no idea if Mr Macks did the work or minds his name being used so I
can form no view on the change. Obviously please remove my sign off if you
make the change
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
2008-04-24 16:00 ` Alan Cox
@ 2008-04-24 16:19 ` Mauro Carvalho Chehab
-1 siblings, 0 replies; 21+ messages in thread
From: Mauro Carvalho Chehab @ 2008-04-24 16:19 UTC (permalink / raw)
To: Alan Cox
Cc: Frank Bennett, Alan Cox, video4linux-list, Michael Krufky,
linux-kernel, ivtv-devel
On Thu, 24 Apr 2008 12:00:43 -0400
Alan Cox <alan@redhat.com> wrote:
> On Thu, Apr 24, 2008 at 12:47:08PM -0300, Mauro Carvalho Chehab wrote:
> > Maybe we can just add his name in parenthesis. Would this patch description be
> > ok for you, Andrew and Alan?
>
> I have no idea if Mr Macks did the work or minds his name being used so I
> can form no view on the change. Obviously please remove my sign off if you
> make the change
So, I'll keep the patch as-is.
Cheers,
Mauro
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Fix VIDIOCGAP corruption in ivtv
@ 2008-04-24 16:19 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 21+ messages in thread
From: Mauro Carvalho Chehab @ 2008-04-24 16:19 UTC (permalink / raw)
To: Alan Cox
Cc: video4linux-list, linux-kernel, ivtv-devel, Michael Krufky,
Frank Bennett, Alan Cox
On Thu, 24 Apr 2008 12:00:43 -0400
Alan Cox <alan@redhat.com> wrote:
> On Thu, Apr 24, 2008 at 12:47:08PM -0300, Mauro Carvalho Chehab wrote:
> > Maybe we can just add his name in parenthesis. Would this patch description be
> > ok for you, Andrew and Alan?
>
> I have no idea if Mr Macks did the work or minds his name being used so I
> can form no view on the change. Obviously please remove my sign off if you
> make the change
So, I'll keep the patch as-is.
Cheers,
Mauro
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply [flat|nested] 21+ messages in thread