diff options
| author | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2010-02-02 14:37:55 -0800 |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2010-02-16 03:17:41 -0500 |
| commit | ded180e7ebfc324b36a94931f99d0705dcd8da29 (patch) | |
| tree | 22c4f4ea4aaf6032e34b2056902ac7f7418d9a88 /drivers/acpi/processor_throttling.c | |
| parent | 0e026445fb36852d3102cb8bb24868765fe5816a (diff) | |
| download | linux-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.c | 3 |
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, |