c37b05c413
The field wasn't initialized in RAM first and later overwritten in a somewhat twisted way (that relied on the size field coming after the tag field in the struct). Change-Id: Ibe931b297df51e3c46ae163e059338781f5a27e2 Found-by: Coverity Scan Signed-off-by: Patrick Georgi <patrick@georgi-clan.de> Reviewed-on: http://review.coreboot.org/4087 Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org> Tested-by: build bot (Jenkins) |
||
|---|---|---|
| .. | ||
| accessors | ||
| cli | ||
| COPYING | ||
| ChangeLog | ||
| DISCLAIMER | ||
| Makefile | ||
| Makefile.inc | ||
| README | ||
| cbfs.c | ||
| cbfs.h | ||
| cmos_lowlevel.c | ||
| cmos_lowlevel.h | ||
| cmos_ops.c | ||
| cmos_ops.h | ||
| common.c | ||
| common.h | ||
| compute_ip_checksum.c | ||
| coreboot_tables.h | ||
| hexdump.c | ||
| hexdump.h | ||
| input_file.c | ||
| input_file.h | ||
| ip_checksum.h | ||
| layout.c | ||
| layout.h | ||
| lbtable.c | ||
| lbtable.h | ||
| nvramtool.spec | ||
| reg_expr.c | ||
| reg_expr.h | ||
| win32mmap.c | ||
README
Summary of Operation
--------------------
nvramtool is a utility for reading/writing coreboot parameters and
displaying information from the coreboot table. It is intended for x86-based
systems (both 32-bit and 64-bit) that use coreboot.
The coreboot table resides in low physical memory, and may be accessed
through the /dev/mem interface. It is created at boot time by coreboot, and
contains various system information such as the type of mainboard in use. It
specifies locations in the CMOS (nonvolatile RAM) where the coreboot
parameters are stored.
For information about coreboot, see http://www.coreboot.org/.
Ideas for Future Improvements
-----------------------------
1. Move the core functionality of this program into a shared library.
2. Consider adding options for displaying other BIOS-provided information
such as the MP table, ACPI table, PCI IRQ routing table, etc.