jackson-databind's renamed @JsonIgnore'd setters can deserialize via private fields
🔗 CVE IDs covered (1)
📋 Description
Summary
POJOPropertiesCollector._renameProperties() allows a property with @JsonProperty("renamed") on the getter and @JsonIgnore on the setter to be renamed rather than dropped. With MapperFeature.INFER_PROPERTY_MUTATORS enabled (default), the private backing field is retained; during deserialization BeanDeserializerFactory.addBeanProps() sees hasField()==true, builds a FieldProperty, and makes the backing field writable. An attacker supplying the renamed JSON key writes the backing field directly, bypassing the @JsonIgnore on the setter.
Impact
POJOs combining a renamed getter with an ignored setter (a read-only-over-the-wire pattern) have that field silently set from attacker input (property tampering / mass assignment). Not a general gadget; no RCE.
Affected / Patched (verified via git tag --contains)
- 2.21 line:
>= 2.21.0, < 2.21.4-> fixed in 2.21.4 (backportc3d56dd, #5968) - 3.x line:
>= 3.0.0, < 3.1.4-> fixed in 3.1.4 (#5967,e88cb17)
Severity / CWE
Maintainer: minor. Reporter: HIGH. CWE-915.
Credits
Omkhar Arasaratnam (@omkhar) - finder.
🎯 Affected products3
- maven/com.fasterxml.jackson.core:jackson-databind:>= 2.21.0, < 2.21.4
- maven/com.fasterxml.jackson.core:jackson-databind:>= 3.0.0, < 3.1.4
- maven/tools.jackson.core:jackson-databind:>= 3.0.0, < 3.1.4
🔗 References (6)
- https://github.com/FasterXML/jackson-databind/security/advisories/GHSA-9fxm-vc8v-hj55
- https://github.com/FasterXML/jackson-databind/pull/5967
- https://github.com/FasterXML/jackson-databind/pull/5968
- https://github.com/FasterXML/jackson-databind/commit/c3d56dd25d52319828147c5b9aeabf2d485c250a
- https://github.com/FasterXML/jackson-databind/commit/e88cb17006b6af4883b973058f0bb6486e5074af
- https://github.com/advisories/GHSA-9fxm-vc8v-hj55