Please include patch from bug 606081 (missing copy_from_user in radeon drm code)
tim.gardner at canonical.com
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 treblig.org> wrote:
>> * Daniel J Blueman wrote:
>>> On 7 August 2010 18:59, Dr. David Alan Gilbert<dave at treblig.org> wrote:
>>>> 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).
>>> It's best to forward your patch
>>> [http://launchpadlibrarian.net/52825846/radeon-info-ioctl-cfu.diff] to
>>> linux-kernel at vger.kernel.org and dri-devel at lists.freedesktop.org and
>>> CC stable at vger.kernel.org .
>>> The patch will be released in 184.108.40.206, 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 vger.kernel.org . 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.
Leann - this is a good candidate for pre-stable. Its at
Note that it morphed a bit before being committed to Dave's repo.
Tim Gardner tim.gardner at canonical.com
More information about the kernel-team