summaryrefslogtreecommitdiff
path: root/upb/table.c
diff options
context:
space:
mode:
authorSakala Venkata Krishna Rohit <rohitsakala@gmail.com>2018-08-27 16:16:09 +0530
committerSakala Venkata Krishna Rohit <rohitsakala@gmail.com>2018-08-27 16:25:34 +0530
commit898f640e65879c793af392479b97dfcfb4fa2f53 (patch)
tree6657940e0605fcd4277bbe41dc5e900909bd5d98 /upb/table.c
parent38d674e86f4d5b368e37e20d31a44c2924de6998 (diff)
Bugfix on bigendianess by casting size_t to unint32_t
The reason for typecasting size_t to unint32_t is that size_t is 8 bytes and uint32_t is only 4 bytes. If not typecasted Memcpy fails to copy the *correct* four bytes in big endian platforms.
Diffstat (limited to 'upb/table.c')
-rw-r--r--upb/table.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/upb/table.c b/upb/table.c
index 61202b0..4239c6f 100644
--- a/upb/table.c
+++ b/upb/table.c
@@ -270,9 +270,10 @@ static size_t begin(const upb_table *t) {
/* A simple "subclass" of upb_table that only adds a hash function for strings. */
static upb_tabkey strcopy(lookupkey_t k2, upb_alloc *a) {
+ uint32_t len = (uint32_t) k2.str.len;
char *str = upb_malloc(a, k2.str.len + sizeof(uint32_t) + 1);
if (str == NULL) return 0;
- memcpy(str, &k2.str.len, sizeof(uint32_t));
+ memcpy(str, &len, sizeof(uint32_t));
memcpy(str + sizeof(uint32_t), k2.str.str, k2.str.len + 1);
return (uintptr_t)str;
}
generated by cgit on debian on lair
contact matthew@masot.net with questions or feedback