3
0
Fork 0
forked from suyu/suyu

nfp: Correct erroneous sizeof expression within GetTagInfo()

The previous expression would copy sizeof(size_t) amount of bytes (8 on
a 64-bit platform) rather than the full 10 bytes comprising the uuid
member.

Given the source and destination types are the same, we can just use an
assignment here instead.
This commit is contained in:
Lioncash 2018-11-14 01:04:38 -05:00
parent 65bd03d74c
commit b4f63db04e

View file

@ -212,7 +212,7 @@ private:
IPC::ResponseBuilder rb{ctx, 2}; IPC::ResponseBuilder rb{ctx, 2};
auto amiibo = nfp_interface.GetAmiiboBuffer(); auto amiibo = nfp_interface.GetAmiiboBuffer();
TagInfo tag_info{}; TagInfo tag_info{};
std::memcpy(tag_info.uuid.data(), amiibo.uuid.data(), sizeof(tag_info.uuid.size())); tag_info.uuid = amiibo.uuid;
tag_info.uuid_length = static_cast<u8>(tag_info.uuid.size()); tag_info.uuid_length = static_cast<u8>(tag_info.uuid.size());
tag_info.protocol = 1; // TODO(ogniK): Figure out actual values tag_info.protocol = 1; // TODO(ogniK): Figure out actual values