Author martin.panter
Recipients John Jones, alex, benjamin.peterson, dhduvall, gennad, gregory.p.smith, martin.panter, neologix, pablogsal
Date 2018-01-29.03:54:28
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <>
Pablo’s code looked unfinished to me. As well as missing documentation, I suspect there may be memory leaks and poor error handling.

The two calls above the “fail:” label look like dead code. The “parse_envlist” result appears to be leaked.

I’m curious why you never call “posix_spawn_file_actions_destroy”. I saw on Open BSD <> it reclaims memory, and it seems sensible to call it on other platforms as well.

No error checking on any of the “posix_spawn_file_actions_” methods.

If “posix_spawn” fails, the “pid” variable will be returned uninitialized.
Date User Action Args
2018-01-29 03:54:29martin.pantersetrecipients: + martin.panter, gregory.p.smith, benjamin.peterson, alex, dhduvall, neologix, gennad, John Jones, pablogsal
2018-01-29 03:54:28martin.pantersetmessageid: <>
2018-01-29 03:54:28martin.panterlinkissue20104 messages
2018-01-29 03:54:28martin.pantercreate