[subset] Fix undefined behaviour.

Don't memcpy if there's nothing to copy.
This commit is contained in:
Garret Rieger 2022-06-11 01:27:33 +00:00 committed by Behdad Esfahbod
parent 311413f16b
commit 1e34852f07
1 changed files with 2 additions and 1 deletions

View File

@ -61,7 +61,8 @@ HB_INTERNAL bool postV2Tail::serialize (hb_serialize_context_t *c,
HBUINT8 *o = c->allocate_size<HBUINT8> (HBUINT8::static_size * (s.length + 1));
if (unlikely (!o)) return_trace (false);
if (!c->check_assign (o[0], s.length, HB_SERIALIZE_ERROR_INT_OVERFLOW)) return_trace (false);
memcpy (o+1, s.arrayZ, HBUINT8::static_size * s.length);
if (s.length)
memcpy (o+1, s.arrayZ, HBUINT8::static_size * s.length);
}
return_trace (true);