Versions of hoek prior to 4.2.1 and 5.0.3 are vulnerable to prototype pollution.

The merge function, and the applyToDefaults and applyToDefaultsWithShallow functions which leverage merge behind the scenes, are vulnerable to a prototype pollution attack when provided an unvalidated payload created from a JSON string containing the __proto__ property.

This can be demonstrated like so:

var Hoek = require('hoek');
var malicious_payload = '{"__proto__":{"oops":"It works !"}}';

var a = {};
console.log("Before : " + a.oops);
Hoek.merge({}, JSON.parse(malicious_payload));
console.log("After : " + a.oops);

This type of attack can be used to overwrite existing properties causing a potential denial of service.

Recommendation

Update to version 4.2.1, 5.0.3 or later.

References

• https://nvd.nist.gov/vuln/detail/CVE-2018-3728
• https://hackerone.com/reports/310439
• hapijs/hoek@32ed5c9
• https://access.redhat.com/errata/RHSA-2018:1263
• https://access.redhat.com/errata/RHSA-2018:1264
• https://snyk.io/vuln/npm:hoek:20180212
• hapijs/hoek@5aed1a8
• https://web.archive.org/web/20200227131737/https://www.securityfocus.com/bid/103108