aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2016-02-28 15:31:49 -0500
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2016-03-09 17:47:35 -0500
commitbea3c377c286f4f63f67f2e061c1d8851af76b7e (patch)
treeb0d63c774230f3e71a2ff80f98fd186059f7e3c9
parentb2ca5dae31d61c4610ea3228f43f7cc66312897c (diff)
ACPI / util: remove redundant check if element is NULL
element is &package->package.elements[i] which can never be NULL so the check to see if it is NULL is redundant and can be removed. Detected with static analysis by CoverityScan Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r--drivers/acpi/utils.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/acpi/utils.c b/drivers/acpi/utils.c
index f2f9873bb5c3..f12a72428aac 100644
--- a/drivers/acpi/utils.c
+++ b/drivers/acpi/utils.c
@@ -201,10 +201,6 @@ acpi_extract_package(union acpi_object *package,
201 u8 **pointer = NULL; 201 u8 **pointer = NULL;
202 union acpi_object *element = &(package->package.elements[i]); 202 union acpi_object *element = &(package->package.elements[i]);
203 203
204 if (!element) {
205 return AE_BAD_DATA;
206 }
207
208 switch (element->type) { 204 switch (element->type) {
209 205
210 case ACPI_TYPE_INTEGER: 206 case ACPI_TYPE_INTEGER: