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/acpi/aml-build.c | |
| parent | 8eb779e4223a18db9838a49ece1bc72cfdfb7761 (diff) | |
| download | qemu-b54ca0c3df4f21315bebdb96dc81cdf1abb9b214.tar.gz | |
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/acpi/aml-build.c')
| -rw-r--r-- | hw/acpi/aml-build.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c index 603068b5ea..667553514e 100644 --- a/hw/acpi/aml-build.c +++ b/hw/acpi/aml-build.c @@ -1451,7 +1451,7 @@ build_header(GArray *linker, GArray *table_data, h->checksum = 0; /* Checksum to be filled in by Guest linker */ bios_linker_loader_add_checksum(linker, ACPI_BUILD_TABLE_FILE, - table_data->data, h, len, &h->checksum); + table_data, h, len, &h->checksum); } void *acpi_data_push(GArray *table_data, unsigned size) |