diff options
author | Julius Werner <jwerner@chromium.org> | 2016-08-05 20:43:47 -0700 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2016-08-08 19:02:07 +0200 |
commit | e25d3ff9bd63cb7123461b266b42af664e81025d (patch) | |
tree | 006320d8697c4dd8f464e826ad53f05645919949 /src/cpu/intel/speedstep | |
parent | 5a6955517f023c3afa79da91f7100425a17f0739 (diff) |
libpayload: lzma: Allocate scratchpad on the heap
Allocating a 15980-byte scratchpad on the stack when your default stack
size is set to 16KB is really not a great idea. We're regularly
overflowing into the end of our heap when using LZMA in libpayload, and
just happen not to notice it because the heap rarely gets filled up all
the way. Of course, since we always *have* a heap in libpayload, the
much saner solution is to just use it directly to allocate the
scratchpad rather than accidentally grow backwards into it anyway.
Change-Id: Ibe4f02057a32bd156a126302178fa6fcab637d2c
Signed-off-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: https://review.coreboot.org/16089
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/cpu/intel/speedstep')
0 files changed, 0 insertions, 0 deletions