summaryrefslogtreecommitdiffstats
path: root/src/interface
diff options
context:
space:
mode:
authorMichael Brown2015-04-24 15:34:32 +0200
committerMichael Brown2015-04-24 15:41:32 +0200
commit9aa8090d069eb0b36769f33544faf0e7e429e844 (patch)
tree02dd0bdc5afca6e625f2f6c73fd85244ab020093 /src/interface
parent[test] Include IPv6 support when performing settings self-tests (diff)
downloadipxe-9aa8090d069eb0b36769f33544faf0e7e429e844.tar.gz
ipxe-9aa8090d069eb0b36769f33544faf0e7e429e844.tar.xz
ipxe-9aa8090d069eb0b36769f33544faf0e7e429e844.zip
[base16] Add buffer size parameter to base16_encode() and base16_decode()
The current API for Base16 (and Base64) encoding requires the caller to always provide sufficient buffer space. This prevents the use of the generic encoding/decoding functionality in some situations, such as in formatting the hex setting types. Implement a generic hex_encode() (based on the existing format_hex_setting()), implement base16_encode() and base16_decode() in terms of the more generic hex_encode() and hex_decode(), and update all callers to provide the additional buffer length parameter. Signed-off-by: Michael Brown <mcb30@ipxe.org>
Diffstat (limited to 'src/interface')
-rw-r--r--src/interface/efi/efi_debug.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/interface/efi/efi_debug.c b/src/interface/efi/efi_debug.c
index e426c795..47380395 100644
--- a/src/interface/efi/efi_debug.c
+++ b/src/interface/efi/efi_debug.c
@@ -330,7 +330,7 @@ const char * efi_devpath_text ( EFI_DEVICE_PATH_PROTOCOL *path ) {
max_len = ( ( sizeof ( text ) - 1 /* NUL */ ) / 2 /* "xx" */ );
if ( len > max_len )
len = max_len;
- base16_encode ( start, len, text );
+ base16_encode ( start, len, text, sizeof ( text ) );
return text;
}