On Mon, Nov 19, 2012 at 10:23:45AM +0100, Ondrej Kos wrote:
On 11/19/2012 12:05 AM, Jakub Hrozek wrote:
>We should at least print an error message and error out if waitpid()
>fails.
>
>https://fedorahosted.org/sssd/ticket/1651
>
>
>
>_______________________________________________
>sssd-devel mailing list
>sssd-devel(a)lists.fedorahosted.org
>https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
>
I noticed this warning while compiling:
src/util/server.c: In function 'become_daemon':
src/util/server.c:123:18: warning: 'ret' may be used uninitialized
in this function [-Wmaybe-uninitialized]
Could you please add the missing initialization? Ack on the patch though.
Ondra
Interesting, I don't see that warning..but gcc is right, in case the
waitpid call failed, we would return garbage and I would just
substitute one Coverity warning for another.
Thanks, a new patch is attached.