summaryrefslogtreecommitdiff
path: root/Kbuild
diff options
context:
space:
mode:
authorMark Tomlinson <mark.tomlinson@alliedtelesis.co.nz>2018-08-29 10:51:14 +1200
committerTom Rini <trini@konsulko.com>2018-09-10 21:19:32 -0400
commit8961c8ad252b8af887439e4e5c6c1bc0c912f2de (patch)
tree168dceaa4be9885b4415a925acb78911308a48c3 /Kbuild
parent5d754197a745f0496935dbdcca775d8e3e7ffce8 (diff)
tools: mkimage: Ensure munmap unmaps the same length that was mapped
The set_header call in kwbimage.c adds a checksum to the end of the image in addition to setting up the header. It 'helpfully' updates the st_size to match the fact that the file is now longer. However, mkimage uses this length in the munmap call. This can lead to unmapping an extra page, of perhaps required data. When this happens, a SEGV can occur. To prevent this from happening, the munmap call now uses the same length that was passed to mmap. This could also have been fixed by not changing the length in kwbimage.c, however changing it in the main file means that other plugins will also not fall for the same trap. Signed-off-by: Mark Tomlinson <mark.tomlinson@alliedtelesis.co.nz> Signed-off-by: Chris Packham <judge.packham@gmail.com> [cp: resolve checkpatch complaints] Tested-by: Chris Packham <judge.packham@gmail.com>
Diffstat (limited to 'Kbuild')
0 files changed, 0 insertions, 0 deletions