pl = apr_palloc(p, sizeof (*pl)); pl->p = p; + pl->program = (program == NULL) ? NULL : apr_pstrdup(p, program); + pl->pid = NULL;Why is this needed now? Or was this just missed previously and is not really related to the reliable pipe usage of the error log?
This was borked. Those fields do not exist in the struct when AP_HAVE_RELIABLE_PIPED_LOGS is not defined. I removed it.
