[OTLayout] Limit alternate-location FeatureParams to 'size' feature

This commit is contained in:
Behdad Esfahbod 2012-12-17 23:29:15 -05:00
parent efe252e600
commit 1ffd23cb47
1 changed files with 6 additions and 1 deletions

View File

@ -495,6 +495,9 @@ struct Feature
* If sanitizing "failed" for the FeatureParams subtable, try it with the
* alternative location. We would know sanitize "failed" if old value
* of the offset was non-zero, but it's zeroed now.
*
* Only do this for the 'size' feature, since at the time of the faulty
* Adobe tools, only the 'size' feature had FeatureParams defined.
*/
Offset orig_offset = featureParams;
@ -504,7 +507,9 @@ struct Feature
if (likely (!orig_offset))
return TRACE_RETURN (true);
if (featureParams == 0 && closure && closure->list_base && closure->list_base < this)
if (featureParams == 0 && closure &&
closure->tag == HB_TAG ('s','i','z','e') &&
closure->list_base && closure->list_base < this)
{
unsigned int new_offset_int = (unsigned int) orig_offset -
((char *) this - (char *) closure->list_base);