david.venhoff/QEMU-Nyx-fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tests/bios-tables-test: Compiler warning fix

Browse Source 
gcc 7.1.1 in fedora 26 moans about the:
   tables = g_new0(uint32_t, tables_nr)

because it can't convince itself that tables_nr is positive.
This is fallout from g_assert_cmpint no longer necessarily being
no-return;  replace it with a plain g_assert.

Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Reviewed-by: Marcel Apfelbaum <marcel@redhat.com>
This commit is contained in:
Dr. David Alan Gilbert 2017-07-20 17:35:36 +01:00 committed by Michael S. Tsirkin
parent 2f6b38d171
commit 91c38e08af
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tests/bios-tables-test.c
View File

@ -108,7 +108,7 @@ static void test_acpi_rsdt_table(test_data *data)
/* compute the table entries in rsdt */ /* compute the table entries in rsdt */
tables_nr = (rsdt_table->length - sizeof(AcpiRsdtDescriptorRev1)) / tables_nr = (rsdt_table->length - sizeof(AcpiRsdtDescriptorRev1)) /
sizeof(uint32_t); sizeof(uint32_t);
g_assert_cmpint(tables_nr, >, 0); g_assert(tables_nr > 0);
/* get the addresses of the tables pointed by rsdt */ /* get the addresses of the tables pointed by rsdt */
tables = g_new0(uint32_t, tables_nr); tables = g_new0(uint32_t, tables_nr);