GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

lib/bootmem.c: Remove unused setting of variable "begin" 

The variable "begin" is extracted from the structure, but 4 lines below
it's overwritten with "end - size". This causes a static build scan error
that should be fixed. Remove the initial assignment of variable "begin".

BUG=b:112253891
TEST=Build and boot grunt.

Change-Id: I0a265747e61289f045c5cac09e40478bd31e16fc
Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com>
Reviewed-on: https://review.coreboot.org/27886
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Martin Roth <martinroth@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Richard Spiegel 2018-08-06 15:26:54 -07:00 committed by Martin Roth
parent 2b68cb08a8
commit d3131e4164
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/lib/bootmem.c
View File

@ -278,7 +278,6 @@ void *bootmem_allocate_buffer(size_t size)
return NULL; return NULL;
/* region now points to the highest usable region for the given size. */ /* region now points to the highest usable region for the given size. */
begin = range_entry_base(region);
end = range_entry_end(region); end = range_entry_end(region);
if (end > max_addr) if (end > max_addr)
end = max_addr; end = max_addr;