Crash when accessing HTMLElement properties from node's vm module

[rogual]

Here's a minimal example:

package.json:

{
    "name": "crash test",
    "main": "bug.html"
}

bug.html:

<script src='bug.js'></script>

bug.js:

window.onload = function() {
  var vm = require('vm');
  var c = vm.createContext({e: document.body});
  var script = 'e.style';
  vm.runInContext(script, c);
};

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

[chaseWillden]

I think you are forgetting the new keyword.

var c = new vm.createContext({e: document.body});

See Documentation

[rogual]

I think that's a mistake in the docs — seems weird to require new there, and there's another sample on that page that doesn't have it.

Tried anyway, but it doesn't stop the crash.

[imyzf]

Modified code according to newest node.js doc, still crash.

var vm = require('vm');
var c = new vm.createContext({e: document.body});
var script = new vm.Script('e.style');
script.runInContext(c);

[rogerwang]

This is fixed in git and is available in the latest nightly build.