{"push_id":1472607013,"size":3,"distinct_size":3,"ref":"refs/heads/master","head":"45c9ca7fd430022a9c0567e1b9f535b3e3f956fd","before":"c65d55f0878a42c5a9fe775ee190d6092c527661","commits":[{"sha":"b374ee8c3dfcefe7b22060c1a2073ee07e4e1b8c","author":{"name":"cjihrig","email":"b0a62289a2635542c30e3402b36d8cbeb000fe3e@gmail.com"},"message":"src: add handle check to spawn_sync\n\nThis commit verifies that the child process handle is of the\ncorrect type before trying to close it in\nCloseHandlesAndDeleteLoop(). This catches the case where input\nvalidation failed, and the child process was never actually\nspawned.\n\nFixes: https://github.com/nodejs/node/issues/8096\nFixes: https://github.com/nodejs/node/issues/8539\nRefs: https://github.com/nodejs/node/issues/9722\nPR-URL: https://github.com/nodejs/node/pull/8312\nReviewed-By: Ben Noordhuis <info@bnoordhuis.nl>","distinct":true,"url":"https://api.github.com/repos/nodejs/node/commits/b374ee8c3dfcefe7b22060c1a2073ee07e4e1b8c"},{"sha":"fc7b0dda85c006e5830a0e34645d769e20b894d2","author":{"name":"cjihrig","email":"b0a62289a2635542c30e3402b36d8cbeb000fe3e@gmail.com"},"message":"child_process: improve input validation\n\nThis commit applies stricter input validation in\nnormalizeSpawnArguments(), which is run by all of the\nchild_process methods. Additional checks are added for spawnSync()\nspecific inputs.\n\nFixes: https://github.com/nodejs/node/issues/8096\nFixes: https://github.com/nodejs/node/issues/8539\nRefs: https://github.com/nodejs/node/issues/9722\nPR-URL: https://github.com/nodejs/node/pull/8312\nReviewed-By: Ben Noordhuis <info@bnoordhuis.nl>","distinct":true,"url":"https://api.github.com/repos/nodejs/node/commits/fc7b0dda85c006e5830a0e34645d769e20b894d2"},{"sha":"45c9ca7fd430022a9c0567e1b9f535b3e3f956fd","author":{"name":"cjihrig","email":"b0a62289a2635542c30e3402b36d8cbeb000fe3e@gmail.com"},"message":"src: remove redundant spawn/spawnSync type checks\n\nThis commit removes C++ checks from spawn() and spawnSync()\nthat are duplicates of the JavaScript type checking.\n\nFixes: https://github.com/nodejs/node/issues/8096\nFixes: https://github.com/nodejs/node/issues/8539\nRefs: https://github.com/nodejs/node/issues/9722\nPR-URL: https://github.com/nodejs/node/pull/8312\nReviewed-By: Ben Noordhuis <info@bnoordhuis.nl>","distinct":true,"url":"https://api.github.com/repos/nodejs/node/commits/45c9ca7fd430022a9c0567e1b9f535b3e3f956fd"}]}
{"actor":{"display_login":"cjihrig"}}