Check the return value from utf8_to_ucs2 instead of whether or not uv is NULL. That's more correct and doesn't depend on the error behavior of utf8_to_ucs2. In practice, we'll never see this though since we pass utf8_to_ucs2 a well formed string.

[tsoome]

stand/efi/boot1/boot1.c

105

If we check the return value below, we do not need to set NULL here, except if the compiler will trigger may be used uninitialized warning...

[imp]

stand/efi/boot1/boot1.c

105

We do, because we want utf8_to_ucs2 to allocate space.