Repository navigation
NullPointerException when oneOf changes to anyOf in response schema #381
Description
Activity
Experienced the same issue:
While comparing two schemas with maven plugin, one file had absent
oneOf, other had it present, so it happily went through
if (CollectionUtils.isNotEmpty(leftComposedSchema.getOneOf()) || CollectionUtils.isNotEmpty(rightComposedSchema.getOneOf())) {And failed with NPE in
org.openapitools.openapidiff.core.compare.schemadiffresult.ComposedSchemaDiffResult#getMapping:
for (Schema<?> schema : composedSchema.getOneOf()) {Unexpected error: Cannot invoke "java.util.List.iterator()" because the return value of "io.swagger.v3.oas.models.media.ComposedSchema.getOneOf()" is null
Iterator here has ZERO null-protection. It's funny that issue haven't been fixed for such a long time.
I believe that null check and replacement with an empty list should do the trick, since after this diff will be emptyList vs nonEmptyList
Thanks for reporting this.
I think we should scope #381 to the runtime crash only (NPE on
oneOf -> anyOf), and handleanyOfsemantics separately.The
anyOf/allOfbehavior has already been discussed and tracked:- Open enhancement: Unify Handling of allOf/anyOf and oneOf in Diff logic #772 (unify handling of
allOf/anyOfvsoneOf)
Unify Handling of allOf/anyOf and oneOf in Diff logic #772 - Related test/consistency work: DrSatyr/issue212 #774
DrSatyr/issue212 #774 - Related composed-schema improvements (recursive refs): Issue 422 #771
Issue 422 #771 - Original
anyOfbug report: [bug] support of breaking changes for "anyOf" #212 (closed)
[bug] support of breaking changes for "anyOf" #212
For this issue, the proposed fix is specifically to prevent the NPE in
ComposedSchemaDiffResult#getMappingwhengetOneOf()is null (e.g., right side isanyOf).
That resolves the crash without changing currentanyOfmerge semantics.- Open enhancement: Unify Handling of allOf/anyOf and oneOf in Diff logic #772 (unify handling of
- added a commit that references this issue
on Feb 23, 2026 - added a commit that references this issue
on Mar 17, 2026
I observed a NullPointerException when a response schema changes from using
oneOftoanyOf. Here is a test that reproduces the issue.The error is as follows:
The direct reason seems to be that in
ComposedSchemaDiffResult#diffthere is this check:and later, in
getMapping(rightComposedSchema)it's assumed that both left and right containoneOf, which is incorrect as the check here is an OR instead of an AND.Changing
||to&&fixes this particular issue and the tests pass, however I'm not certain that this is the correct fix as I don't understand the logic enough to be able to tell if I'm not breaking anything else.Additionally, the way that
anyOfis handled seems to be merging all schema alternatives into one, which includes merging theirrequiredoptions, which seems incorrect. If a schema is anyOf(A, B), where A has required property propA and B has required property propB, after the merge the result would be a schema that requires both propA and propB instead of propA OR propB.Do you think that the
||to&&is a valid fix for the issue, or should the anyOf handling be changed somehow?