From patchwork Wed Jan 16 01:01:16 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: apci: apicinstance: clean up smatch warning X-Patchwork-Submitter: Colin King X-Patchwork-Id: 212362 Message-Id: <1358298076-7523-1-git-send-email-colin.king@canonical.com> To: fwts-devel@lists.ubuntu.com Date: Wed, 16 Jan 2013 01:01:16 +0000 From: Colin King List-Id: Firmware Test Suite Development From: Colin Ian King smatch gets confused by the use of count to check if the madt table has been set and the code is a little bit clearer if we just check to see if the table is not set by checking if it is NULL. Signed-off-by: Colin Ian King Acked-by: Keng-Yu Lin Acked-by: Alex Hung --- src/acpi/apicinstance/apicinstance.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/acpi/apicinstance/apicinstance.c b/src/acpi/apicinstance/apicinstance.c index 245aa48..d2376de 100644 --- a/src/acpi/apicinstance/apicinstance.c +++ b/src/acpi/apicinstance/apicinstance.c @@ -43,7 +43,7 @@ static int apicinstance_test1(fwts_framework *fw) table->name, (unsigned long long)table->addr, (int)table->length); - if (count == 0) + if (first_madt_table == NULL) first_madt_table = table; else { if ((first_madt_table->length == table->length) &&