diff options
| author | Michael S. Tsirkin <mst@redhat.com> | 2016-02-21 12:41:55 +0200 |
|---|---|---|
| committer | Michael S. Tsirkin <mst@redhat.com> | 2016-02-23 12:55:16 +0200 |
| commit | b54ca0c3df4f21315bebdb96dc81cdf1abb9b214 (patch) | |
| tree | 148543850c38bf7b17c2835f4263cdadc8e1d096 /hw/arm/virt-acpi-build.c | |
| parent | 8eb779e4223a18db9838a49ece1bc72cfdfb7761 (diff) | |
| download | qemu-b54ca0c3df4f21315bebdb96dc81cdf1abb9b214.zip qemu-b54ca0c3df4f21315bebdb96dc81cdf1abb9b214.tar.gz qemu-b54ca0c3df4f21315bebdb96dc81cdf1abb9b214.tar.bz2 | |
bios-linker-loader: document+validate input
While guest/host ABI is documented in hw/acpi/bios-linker-loader.c,
the API was left undocumented.
This adds documentation for all API functions.
Additionally, input is validated to make sure all
pointers fall within range of provided files.
To allow this validation for checksum commands,
bios_linker_loader_add_checksum is changed to accept GArray * in place
of void *.
Reported-by: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'hw/arm/virt-acpi-build.c')
| -rw-r--r-- | hw/arm/virt-acpi-build.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c index 8cf9a21..b8b3ece 100644 --- a/hw/arm/virt-acpi-build.c +++ b/hw/arm/virt-acpi-build.c @@ -359,7 +359,8 @@ build_rsdp(GArray *rsdp_table, GArray *linker, unsigned rsdt) rsdp->checksum = 0; /* Checksum to be filled by Guest linker */ bios_linker_loader_add_checksum(linker, ACPI_BUILD_RSDP_FILE, - rsdp, rsdp, sizeof *rsdp, &rsdp->checksum); + rsdp_table, rsdp, sizeof *rsdp, + &rsdp->checksum); return rsdp_table; } |