diff options
| author | Rasmus Villemoes <rasmus.villemoes@prevas.dk> | 2020-11-20 11:45:35 +0100 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2021-01-16 14:49:09 -0500 |
| commit | 31ce367cd10087b532431c023e4a95513ecdee5d (patch) | |
| tree | abe69d421178391a6cffe43b6c231d92f9100ef8 /include/uuid.h | |
| parent | 2d572ede1185db2129685e8cedfb690a5e3c4d3d (diff) | |
| download | u-boot-31ce367cd10087b532431c023e4a95513ecdee5d.tar.gz u-boot-31ce367cd10087b532431c023e4a95513ecdee5d.tar.bz2 u-boot-31ce367cd10087b532431c023e4a95513ecdee5d.zip | |
lib/uuid.c: change prototype of uuid_guid_get_str()
There's no reason to require an appropriately sized output parameter
for the string, that's error-prone should the table ever grow an
element with a longer string. We can just return the const char*
pointer directly.
Update the only caller accordingly, and get rid of pointless ifdeffery
in the header so that the compiler always sees a declaration and can
thus do type-checking, whether or not PARTITION_TYPE_GUID is enabled
or not.
Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
Diffstat (limited to 'include/uuid.h')
| -rw-r--r-- | include/uuid.h | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/include/uuid.h b/include/uuid.h index 73c5a89ec7..0c653cb087 100644 --- a/include/uuid.h +++ b/include/uuid.h @@ -39,10 +39,8 @@ int uuid_str_to_bin(const char *uuid_str, unsigned char *uuid_bin, int str_format); void uuid_bin_to_str(const unsigned char *uuid_bin, char *uuid_str, int str_format); -#ifdef CONFIG_PARTITION_TYPE_GUID int uuid_guid_get_bin(const char *guid_str, unsigned char *guid_bin); -int uuid_guid_get_str(const unsigned char *guid_bin, char *guid_str); -#endif +const char *uuid_guid_get_str(const unsigned char *guid_bin); void gen_rand_uuid(unsigned char *uuid_bin); void gen_rand_uuid_str(char *uuid_str, int str_format); #endif |