From: James <james@jmp-e.com>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: starblue@users.sourceforge.net,
legousb-devel@lists.sourceforge.net, linux-usb@vger.kernel.org,
linux-kernel@vger.kernel.org, security@kernel.org,
stable@vger.kernel.org
Subject: Re: [PATCH] usb: misc: legousbtower: Fix NULL pointer deference
Date: Tue, 20 Sep 2016 10:09:56 +0100 [thread overview]
Message-ID: <20160920090955.GA25681@hostname> (raw)
In-Reply-To: <20160920083256.GA12786@kroah.com>
[-- Attachment #1: Type: text/plain, Size: 1243 bytes --]
Hi greg,
Many appologies, I did not see your patch attached/forgot it
existed when I fixed this yesterday. I created this patch based on your
feedback from my original email on September 2nd. I have tested
this patch with an emulated device and it fixes the vulnerability.
Again, applogies for the long delay and patch confusion.
Best regards,
James
On Tue, Sep 20, 2016 at 10:32:56AM +0200, Greg KH wrote:
>On Tue, Sep 20, 2016 at 10:18:21AM +0200, Oliver Neukum wrote:
>> On Tue, 2016-09-20 at 08:22 +0200, Greg KH wrote:
>> > When you sent this the last time (back on Sept 2), I sent you a
>> > "simpler" patch, the same one below, and asked you if that worked
>> > instead. I never heard back, could you test this patch and see if it
>> > resolves the issue for you?
>>
>> As far as I can tell, your patches are identical.
>
>Hah, so they are. James's original patch was not this one, sorry for
>not looking at it closer.
>
>James, any reason why you are passing off my patch as yours here? Did
>you test mine and see that this did solve the problem? You did change
>your changelog text a bit, to reflect the change between mine and yours,
>so I'm guessing that you did?
>
>thanks,
>
>greg k-h
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next prev parent reply other threads:[~2016-09-20 9:10 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-19 18:09 [PATCH] usb: misc: legousbtower: Fix NULL pointer deference James
2016-09-20 6:22 ` Greg KH
2016-09-20 8:18 ` Oliver Neukum
2016-09-20 8:32 ` Greg KH
2016-09-20 9:09 ` James [this message]
2016-09-20 11:04 ` Greg KH
2016-09-21 16:27 ` Greg KH
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160920090955.GA25681@hostname \
--to=james@jmp-e.com \
--cc=gregkh@linuxfoundation.org \
--cc=legousb-devel@lists.sourceforge.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=security@kernel.org \
--cc=stable@vger.kernel.org \
--cc=starblue@users.sourceforge.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).