Merge pull request #2040 from l2dy/exec

liviuchircu · web-flow · commit 31fbdc99f783 · 2020-04-03T19:02:52.000+02:00
exec: fix potential use of uninitialized pointer
diff --git a/modules/exec/exec.c b/modules/exec/exec.c
@@ -170,7 +170,7 @@ int start_async_exec(struct sip_msg* msg, str* command, str* input,
 													pv_spec_t *outvar, int *fd)
 {
 	pid_t pid;
-	FILE *pin = NULL, *pout;
+	FILE *pin = NULL, *pout = NULL;
 	int val;
 
 	if ((input&&input->s&&input->len) || outvar) {

Original file line number	Diff line number	Diff line change
`@@ -170,7 +170,7 @@ int start_async_exec(struct sip_msg* msg, str* command, str* input,`
`170`	`170`	`pv_spec_t outvar, int fd)`
`171`	`171`	`{`
`172`	`172`	`pid_t pid;`
`173`		`- FILE pin = NULL, pout;`
	`173`	`+ FILE pin = NULL, pout = NULL;`
`174`	`174`	`int val;`
`175`	`175`
`176`	`176`	`if ((input&&input->s&&input->len) \|\| outvar) {`