URL:
https://github.com/SSSD/sssd/pull/916
Title: #916: Fix one coverity issue
alexey-tikhonov commented:
"""
a socket file descriptor was replaced by 0 (stdin) by a mistake
Thank you, good catch! Fixed.
I think the 3rd patch could be left out (I like the goto version more
for the added "boiler plate" against future mistakes in case we add more code to
the function, but that will probably not happen, so I will let Alexey to decide, I am fine
either way)
But this is the main purpose of this PR (to reduce covscan complains). So I am looking for
a way to please Coverity while keeping "done" section there...
"""
See the full comment at
https://github.com/SSSD/sssd/pull/916#issuecomment-549833970