ParsePartialFromString error not handled in mutator.cc

[vabr-g]

In

libprotobuf-mutator/src/mutator.cc

Line 382 in ffd86a3

message->ParsePartialFromString(std::string(any.value()));

, the Boolean result of ParsePartialFromString is not handled (by returning null).

This seems like an oversight.

But OTOH, it just allowed us to catch a bug in the proto parser library. So maybe it's on purpose?

Filing this in any case, so that the libprotobuf-mutator maintainers can decide.

[vitalybuka]

It's intentionally resilient to parsing error to avoid getting corpus obsolete if proto changed
If some required fields are missing, Mutate will try populate them.