diff options
| author | Adrian Bunk <bunk@stusta.de> | 2006-03-11 16:12:00 +0100 |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2006-03-31 08:02:39 +0200 |
| commit | 6665bda76461308868bd1e52caf627f4cb29ed32 (patch) | |
| tree | 7d4690bc13d631759c3c8f59f72f66b515a02968 /drivers/acpi/video.c | |
| parent | [ACPI] fix possible acpi thermal leak in failure path (diff) | |
| download | linux-6665bda76461308868bd1e52caf627f4cb29ed32.tar.xz linux-6665bda76461308868bd1e52caf627f4cb29ed32.zip | |
[ACPI] drivers/acpi/video.c: fix error path NULL pointer dereference
The Coverity checker spotted this bug in
acpi_video_device_lcd_query_levels().
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/acpi/video.c')
| -rw-r--r-- | drivers/acpi/video.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c index bd4887518373..724149d72d91 100644 --- a/drivers/acpi/video.c +++ b/drivers/acpi/video.c @@ -323,7 +323,7 @@ acpi_video_device_lcd_query_levels(struct acpi_video_device *device, if (!ACPI_SUCCESS(status)) return_VALUE(status); obj = (union acpi_object *)buffer.pointer; - if (!obj && (obj->type != ACPI_TYPE_PACKAGE)) { + if (!obj || (obj->type != ACPI_TYPE_PACKAGE)) { ACPI_DEBUG_PRINT((ACPI_DB_ERROR, "Invalid _BCL data\n")); status = -EFAULT; goto err; |