efi_loader: buffer size for load options
authorHeinrich Schuchardt <xypron.glpk@gmx.de>
Fri, 31 Aug 2018 19:31:33 +0000 (21:31 +0200)
committerAlexander Graf <agraf@suse.de>
Sun, 23 Sep 2018 19:55:29 +0000 (21:55 +0200)
The number of bytes in an utf-8 string is an upper limit for the number of
words in the equivalent utf-16 string. In so far the inumbant coding works
correctly. For non-ASCII characters the utf-16 string is shorter. With the
patch only the necessary buffer size is allocated for the load options.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Signed-off-by: Alexander Graf <agraf@suse.de>
cmd/bootefi.c

index e169f5edc0d3d734652ea1c8f54ab47355f0b611..a0a8a7cdac0e5bda19880ca71277621869890d51 100644 (file)
@@ -116,18 +116,20 @@ static void set_load_options(struct efi_loaded_image *loaded_image_info,
 {
        size_t size;
        const char *env = env_get(env_var);
+       u16 *pos;
 
        loaded_image_info->load_options = NULL;
        loaded_image_info->load_options_size = 0;
        if (!env)
                return;
-       size = strlen(env) + 1;
+       size = utf8_utf16_strlen(env) + 1;
        loaded_image_info->load_options = calloc(size, sizeof(u16));
        if (!loaded_image_info->load_options) {
                printf("ERROR: Out of memory\n");
                return;
        }
-       utf8_to_utf16(loaded_image_info->load_options, (u8 *)env, size);
+       pos = loaded_image_info->load_options;
+       utf8_utf16_strcpy(&pos, env);
        loaded_image_info->load_options_size = size * 2;
 }