On 11 Sep 2001 [EMAIL PROTECTED] wrote:
> - if (attr->child_in || attr->child_out || attr->child_err)
> + if ((attr->child_in && attr->child_in->filehand)
> + || (attr->child_out && attr->child_out->filehand)
> + || (attr->child_err && attr->child_err->filehand))
Yep.
> {
> si.dwFlags |= STARTF_USESTDHANDLES;
> - if (attr->child_in)
> + if (attr->child_in && attr->child_in->filehand)
> si.hStdInput = attr->child_in->filehand;
> - if (attr->child_out)
> + if (attr->child_out && attr->child_out->filehand)
> si.hStdOutput = attr->child_out->filehand;
> - if (attr->child_err)
> + if (attr->child_err && attr->child_err->filehand)
> si.hStdError = attr->child_err->filehand;
This shouldn't be required. The original version set all three
si.hStdfoo's unconditionally. Just checking attr->child_foo to avoid a
segfault is sufficient.
Thanks,
--Cliff
--------------------------------------------------------------
Cliff Woolley
[EMAIL PROTECTED]
Charlottesville, VA
