diff options
author | Thomas Huth <thuth@redhat.com> | 2015-11-25 20:58:18 +0100 |
---|---|---|
committer | Alexey Kardashevskiy <aik@ozlabs.ru> | 2015-12-01 17:02:04 +1100 |
commit | f0d251a0775572ebce8566c16e4482455b9efd84 (patch) | |
tree | 8a54444507eeb17d4c0a351f87378db8e777b76e /clients | |
parent | edd582383f9800e61b48669f7515e5d334ef961b (diff) | |
download | SLOF-f0d251a0775572ebce8566c16e4482455b9efd84.zip SLOF-f0d251a0775572ebce8566c16e4482455b9efd84.tar.gz SLOF-f0d251a0775572ebce8566c16e4482455b9efd84.tar.bz2 |
Rework wrapper for new_nvram_partition() and fix possible bug in there
The wrapper for new_nvram_partition() is using a 12 bytes buffer to
create a zero-terminated string. However, if the string has exactly
12 characters, the final NUL-terminator is missing. new_nvram_partition()
then calls create_nvram_partition() internally which depends on proper
NUL-terminated strings. So fix this by making sure that the copied
string is always NUL-terminated - and while we're at it, also move
the copy code out of libnvram.code to save some precious bytes in the
stack space of the engine() function.
Signed-off-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
Diffstat (limited to 'clients')
0 files changed, 0 insertions, 0 deletions