> > Behdad is right. Don't test stuff that FreeType doesn't need for > itself. In other words, only test data that FreeType itself is going > to access. For example, if we need the 12th byte of structure `foo', > then we should check whether we can do that, together with some sanity > tests to find out whether this value in `foo' is reasonable. >
Okay, thanks Behdad and Werner for pointing this out. So I guess then the table size checks are probably not useful, however, the checks for `numEntries' not being non-zero and an SVG document's length not being non-zero are probably needed? Since, my code directly relies on these two. If `numEntries' is zero, the SVG table is pointless. if an SVG document's length is zero, there is no need to process it. The `version' should be zero. Should these checks stay? or should I just revert the whole commit?
_______________________________________________ Freetype-devel mailing list Freetype-devel@nongnu.org https://lists.nongnu.org/mailman/listinfo/freetype-devel