ICU-22549 Add DateFormatSymbols fuzzer #2748
Conversation
| fuzzData.remove_prefix(sizeof(context)); | ||
| icu::DateFormatSymbols::DtContextType context_mod = | ||
| static_cast<icu::DateFormatSymbols::DtContextType>( | ||
| context % icu::DateFormatSymbols::DtContextType::DT_CONTEXT_COUNT); |
There was a problem hiding this comment.
With both the context and the width, do you want to do the "mod" thing (or only do the "mod" thing)? It seems again that it'd be useful also to check that passing a completely illegal enum value (with, of course, you can do in C++ with casting) doesn't cause anything to barf.
| dfs->getLeapMonthPatterns(count); | ||
| int32_t count2; | ||
| dfs->getZoneStrings(count, count2); | ||
| dfs->getLocalPatternChars(output); |
There was a problem hiding this comment.
Unless I'm mistaken, a DateFormatSymbols object is mutable, so you might also want to check what happens if you actually set the symbols themselves to random garbage values. Of course, the symbols object just behaves as a struct, so to really test that, you'd also have to use the bogus symbols object to format a date. (This shouldn't be a problem unless you've got really long strings, unassigned characters, illegal byte combinations, or possibly characters with unusual semantics, but it seems worth checking for all those things...)
Checklist