Please include patch from bug 606081 (missing copy_from_user in radeon drm code)

Tim Gardner tim.gardner at
Mon Aug 9 23:10:23 UTC 2010

On 08/08/2010 07:37 AM, Daniel J Blueman wrote:
> On 8 August 2010 13:28, Dr. David Alan Gilbert<dave at>  wrote:
>> * Daniel J Blueman wrote:
>>> On 7 August 2010 18:59, Dr. David Alan Gilbert<dave at>  wrote:
>>>> Hi,
>>>>   Please could you pick up my patch in bug 606081
>>>> It fixes a missing copy_from_user in the radeon drm code that
>>>> causes an oops running google earth on 64bit Maverick.
>>>> It's not clear to me whether there also needs to be a separate
>>>> fix to the 32bit compat code; but the patch fixes the oops
>>>> which is a pretty obvious missing copy_from_user.
>>>> (I don't think it's actually a security issue - the read value
>>>> is only used in a comparison, not returned directly).
>>>> Dave
>>> It's best to forward your patch
>>> [] to
>>> linux-kernel at and dri-devel at and
>>> CC stable at .
>>> The patch will be released in, which will be used in Maverick.
>> Hi Daniel,
>>   Thanks for the reply.  The patch is already included in the upstream
>> bug at fdo and Dave Airlie has queued it for Linus.
>> However it would be nice to get it into the Maverick alpha/beta
>> kernels since it's breaking a bunch of 3d apps which means it is harder
>> for people to look for other bugs.
> Dave should have queued it up for 2.6.35-stable if it is a good fix
> (which is clearly is).
> I would ping him the request, CCing stable at . The first
> stable update will be in the next few weeks likely, so it'll get into
> Maverick soon after; better to minimise the Ubuntu delta with
> upstream, minimising risk and work.
> Thanks,
>    Daniel

Leann - this is a good candidate for pre-stable. Its at 
drm-core-next d8ab35575098b2d6dc10b2535aeb40545933ae56

Note that it morphed a bit before being committed to Dave's repo.

Tim Gardner tim.gardner at

More information about the kernel-team mailing list