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