joi: object().rename() with a template target can set the validated object's prototype
Affected releases · joi
- 18.x18.0.0 – 18.2.3Fixed in 18.2.4
- 17.x17.1.1 – 17.13.4Fixed in 17.13.5
On 18.x, the newest line:
npm install joi@18.2.4Every release on a timeline
Affected
18.0.0 – 18.2.3
10 releases
Fixed
18.2.4 – 18.3.1
8 releases, latest
Affected
17.1.1 – 17.13.4
38 releases
Fixed
17.13.5 – 17.13.9
5 releases
What happens
Impact
Applications are affected only if a schema renames keys with a regular-expression source and a Joi.expression() / Joi.x() target that interpolates the pattern's own match data, combined with { multiple: true }, for example .rename(/^x-(.+)$/, Joi.x('{#1}'), { multiple: true }). Because the target is rendered from the matched input key, an attacker who controls input keys can send x-__proto__ with an object value and make the rename target render as __proto__, which sets the prototype of the object joi returns instead of creating a key on it. The global Object.prototype is not modified, so the effect is confined to the object returned by that one validate() call.
Schemas using a static string rename target are not affected, and neither are schemas left on the default { multiple: false }.
Patches
Versions 17.13.5 and 18.2.4 have been released to address the issue.
Workarounds
Replace the template rename target with a static string target.
Keep the template but make the capture unable to produce
__proto__, using a negative lookahead:.rename(/^x-(?!__proto__$)(.+)$/, Joi.x('{#1}'), { multiple: true })Drop { multiple: true } from the rename, which stops the rename before the assignment.