Refactor: pacemakerd: Remove more nesting in start_child()

No meaningful code changes. A few minor things like using long long
instead of unsigned long for uid_t/gid_t, comparing explicitly against
0, using strerror() for system errno, and using parentheses around
comparisons.

Signed-off-by: Reid Wahl <nrwahl@protonmail.com>