Re: [PATCH] sparc64: Properly range check DAX completion index

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Rob Gardner <rob.gardner@xxxxxxxxxx>
Date: Sun, 1 Apr 2018 12:42:46 -0600

> On 04/01/2018 11:11 AM, Linus Torvalds wrote:
>> On Sat, Mar 31, 2018 at 9:53 PM, Rob Gardner <rob.gardner@xxxxxxxxxx>
>> wrote:
>>> Signed-off-by: Rob Gardner <rob.gardner@xxxxxxxxxx>
>>> Signed-off-by: Jonathan Helman <jonathan.helman@xxxxxxxxxx>
>>> Reported-by: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
>> That Reported-by: should be "oguard <oguard@xxxxxxxxxxxxxx>"
>>
>> I was just the messenger.
>>
>>                  Linus
> 
> 
> oguard observed "lack of size check on the copy_from_user", but that
> wasn't really a bug since 'count' actually is checked in dax_write().
> 
> But you noticed that idx could be negative and idx + nccbs could
> overflow, and this is a genuine bug that nobody else saw.

Agreed, Linus's as the reporter is correct.

Applied and queued up for -stable, thank you.
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Development]     [DCCP]     [Linux ARM Development]     [Linux]     [Photo]     [Yosemite Help]     [Linux ARM Kernel]     [Linux SCSI]     [Linux x86_64]     [Linux Hams]

  Powered by Linux