Segfault in runInContext

[lm1]

The following code generates segmentation fault:

require('vm').runInContext('', {});

possibly runInContext should check if given object is valid context.

[bnoordhuis]

@ry 06c9a4e, review please!

[ry]

@bnoordhuis, LGTM

[ghost]

By the way, something like this can happen at other places, too (Script.prototype.runInThisContext.call({}, ...) should end badly, too; I think all other WrappedObjects (or ObjectWraps? that superclass that is used in all similar cases) wrapping external objects should do this check...).

[bnoordhuis]

@Herby Those calls should be safe (though they won't do what you expect) but if you find any that crash node, please post a test case.