|
Description
|
[dep, 11Apr2006]
In startd's exec_method, we explicitly store the result of
restarter_set_method_context() in errno for later processing, but
immediately call fputs() before getting anywhere:
/* Set credentials. */
errno = restarter_set_method_context(mcp, &errf);
if (errno != 0) {
(void) fputs("svc.startd could not set context for method: ",
stderr);
As a result, all error processing that follows is broken.
We should be using a local variable instead of abusing errno.
|