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
_______________________________________________ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
