summaryrefslogtreecommitdiffstats
path: root/drivers/acpi/processor_throttling.c
diff options
context:
space:
mode:
authorBartlomiej Zolnierkiewicz <bzolnier@gmail.com>2010-02-02 14:37:55 -0800
committerLen Brown <len.brown@intel.com>2010-02-16 03:17:41 -0500
commitded180e7ebfc324b36a94931f99d0705dcd8da29 (patch)
tree22c4f4ea4aaf6032e34b2056902ac7f7418d9a88 /drivers/acpi/processor_throttling.c
parent0e026445fb36852d3102cb8bb24868765fe5816a (diff)
downloadlinux-ded180e7ebfc324b36a94931f99d0705dcd8da29.tar.gz
linux-ded180e7ebfc324b36a94931f99d0705dcd8da29.tar.bz2
linux-ded180e7ebfc324b36a94931f99d0705dcd8da29.zip
ACPI: remove superfluous NULL pointer check from acpi_processor_get_throttling_info()
Dan's list contains: drivers/acpi/processor_throttling.c +1139 acpi_processor_get_throttling_info(11) warning: variable derefenced before check 'pr' acpi_processor_get_throttling_info() is never called with pr == NULL. [ bart: the potential NULL pointer dereference was finally fixed in (much later than mine) commit 5cfa245 but my patch is still valid ] Reported-by: Dan Carpenter <error27@gmail.com> Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/acpi/processor_throttling.c')
-rw-r--r--drivers/acpi/processor_throttling.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/acpi/processor_throttling.c b/drivers/acpi/processor_throttling.c
index 1c5d7a8b2fdf..649b2b9b4759 100644
--- a/drivers/acpi/processor_throttling.c
+++ b/drivers/acpi/processor_throttling.c
@@ -1133,9 +1133,6 @@ int acpi_processor_get_throttling_info(struct acpi_processor *pr)
int result = 0;
struct acpi_processor_throttling *pthrottling;
- if (!pr)
- return -EINVAL;
-
ACPI_DEBUG_PRINT((ACPI_DB_INFO,
"pblk_address[0x%08x] duty_offset[%d] duty_width[%d]\n",
pr->throttling.address,