drm/radeon: fix use after free in ATRM bios reading code.
authorDave Airlie <airlied@redhat.com>
Thu, 2 Feb 2012 15:25:16 +0000 (15:25 +0000)
committerDave Airlie <airlied@redhat.com>
Thu, 2 Feb 2012 15:25:16 +0000 (15:25 +0000)
Fixes:
https://bugs.freedesktop.org/show_bug.cgi?id=45503

Reported-and-Debugged-by: mlambda@gmail.com
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/radeon/radeon_atpx_handler.c

index 13ac63b..98724fc 100644 (file)
@@ -59,8 +59,9 @@ static int radeon_atrm_call(acpi_handle atrm_handle, uint8_t *bios,
 
        obj = (union acpi_object *)buffer.pointer;
        memcpy(bios+offset, obj->buffer.pointer, obj->buffer.length);
+       len = obj->buffer.length;
        kfree(buffer.pointer);
-       return obj->buffer.length;
+       return len;
 }
 
 bool radeon_atrm_supported(struct pci_dev *pdev)