mirror of
https://github.com/kremalicious/metamask-extension.git
synced 2024-11-29 23:58:06 +01:00
1175b4bfa7
This PR makes ~all named intrinsics in all of our JavaScript processes non-modifiable. A named intrinsic is any property specified by the ECMAScript specification that exists on `globalThis` when the JavaScript process starts. We say that a property is non-modifiable if it is non-configurable and non-writable. We make exceptions for properties that meet any of the following criteria: 1. Properties that are non-configurable by the time `lockdown-run.js` is executed are not modified, because they can't be. 2. Properties that have accessor properties (`get` or `set`) are made non-configurable, but their writability cannot be modified, and is therefore left unchanged. It's unclear how many of the named intrinsics this applies to, if any, but it's good defensive programming, regardless.
115 lines
4.1 KiB
JavaScript
115 lines
4.1 KiB
JavaScript
// Freezes all intrinsics
|
|
try {
|
|
// eslint-disable-next-line no-undef,import/unambiguous
|
|
lockdown({
|
|
consoleTaming: 'unsafe',
|
|
errorTaming: 'unsafe',
|
|
mathTaming: 'unsafe',
|
|
dateTaming: 'unsafe',
|
|
overrideTaming: 'severe',
|
|
});
|
|
} catch (error) {
|
|
// If the `lockdown` call throws an exception, it interferes with the
|
|
// contentscript injection on some versions of Firefox. The error is
|
|
// caught and logged here so that the contentscript still gets injected.
|
|
// This affects Firefox v56 and Waterfox Classic.
|
|
console.error('Lockdown failed:', error);
|
|
if (globalThis.sentry && globalThis.sentry.captureException) {
|
|
globalThis.sentry.captureException(
|
|
new Error(`Lockdown failed: ${error.message}`),
|
|
);
|
|
}
|
|
}
|
|
|
|
// Make all "object" and "function" own properties of globalThis
|
|
// non-configurable and non-writable, when possible.
|
|
// We call the a property that is non-configurable and non-writable,
|
|
// "non-modifiable".
|
|
try {
|
|
/**
|
|
* `lockdown` only hardens the properties enumerated by the
|
|
* universalPropertyNames constant specified in 'ses/src/whitelist'. This
|
|
* function makes all function and object properties on the start compartment
|
|
* global non-configurable and non-writable, unless they are already
|
|
* non-configurable.
|
|
*
|
|
* It is critical that this function runs at the right time during
|
|
* initialization, which should always be immediately after `lockdown` has been
|
|
* called. At the time of writing, the modifications this function makes to the
|
|
* runtime environment appear to be non-breaking, but that could change with
|
|
* the addition of dependencies, or the order of our scripts in our HTML files.
|
|
* Exercise caution.
|
|
*
|
|
* See inline comments for implementation details.
|
|
*
|
|
* We write this function in IIFE format to avoid polluting global scope.
|
|
*/
|
|
(function protectIntrinsics() {
|
|
const namedIntrinsics = Reflect.ownKeys(new Compartment().globalThis);
|
|
|
|
// These named intrinsics are not automatically hardened by `lockdown`
|
|
const shouldHardenManually = new Set(['eval', 'Function']);
|
|
|
|
const globalProperties = new Set([
|
|
// universalPropertyNames is a constant added by lockdown to global scope
|
|
// at the time of writing, it is initialized in 'ses/src/whitelist'.
|
|
// These properties tend to be non-enumerable.
|
|
...namedIntrinsics,
|
|
|
|
// TODO: Also include the named platform globals
|
|
// This grabs every enumerable property on globalThis.
|
|
// ...Object.keys(globalThis),
|
|
]);
|
|
|
|
globalProperties.forEach((propertyName) => {
|
|
const descriptor = Reflect.getOwnPropertyDescriptor(
|
|
globalThis,
|
|
propertyName,
|
|
);
|
|
|
|
if (descriptor) {
|
|
if (descriptor.configurable) {
|
|
// If the property on globalThis is configurable, make it
|
|
// non-configurable. If it has no accessor properties, also make it
|
|
// non-writable.
|
|
if (hasAccessor(descriptor)) {
|
|
Object.defineProperty(globalThis, propertyName, {
|
|
configurable: false,
|
|
});
|
|
} else {
|
|
Object.defineProperty(globalThis, propertyName, {
|
|
configurable: false,
|
|
writable: false,
|
|
});
|
|
}
|
|
}
|
|
|
|
if (shouldHardenManually.has(propertyName)) {
|
|
harden(globalThis[propertyName]);
|
|
}
|
|
}
|
|
});
|
|
|
|
/**
|
|
* Checks whether the given propertyName descriptor has any accessors, i.e. the
|
|
* properties `get` or `set`.
|
|
*
|
|
* We want to make globals non-writable, and we can't set the `writable`
|
|
* property and accessor properties at the same time.
|
|
*
|
|
* @param {Object} descriptor - The propertyName descriptor to check.
|
|
* @returns {boolean} Whether the propertyName descriptor has any accessors.
|
|
*/
|
|
function hasAccessor(descriptor) {
|
|
return 'set' in descriptor || 'get' in descriptor;
|
|
}
|
|
})();
|
|
} catch (error) {
|
|
console.error('Protecting intrinsics failed:', error);
|
|
if (globalThis.sentry && globalThis.sentry.captureException) {
|
|
globalThis.sentry.captureException(
|
|
new Error(`Protecting intrinsics failed: ${error.message}`),
|
|
);
|
|
}
|
|
}
|