GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — child_process: add nullptr checks after allocs · nodejs/node@2e6e2d6

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

src
- process_wrap.cc

Expand file treeCollapse file tree

Open diff view settings

Collapse file

`‎src/process_wrap.cc‎`

Copy file name to clipboardExpand all lines: src/process_wrap.cc

+2Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -165,6 +165,7 @@ class ProcessWrap : public HandleWrap {`
`165`	`165`	`for (int i = 0; i < argc; i++) {`
`166`	`166`	`node::Utf8Value arg(env->isolate(), js_argv->Get(i));`
`167`	`167`	`options.args[i] = strdup(*arg);`
	`168`	`+ CHECK_NE(options.args[i], nullptr);`
`168`	`169`	`}`
`169`	`170`	`options.args[argc] = nullptr;`
`170`	`171`	`}`
`@@ -186,6 +187,7 @@ class ProcessWrap : public HandleWrap {`
`186`	`187`	`for (int i = 0; i < envc; i++) {`
`187`	`188`	`node::Utf8Value pair(env->isolate(), env_opt->Get(i));`
`188`	`189`	`options.env[i] = strdup(*pair);`
	`190`	`+ CHECK_NE(options.env[i], nullptr);`
`189`	`191`	`}`
`190`	`192`	`options.env[envc] = nullptr;`
`191`	`193`	`}`

0 commit comments

Comments

0 (0)

You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.