URL: https://github.com/SSSD/sssd/pull/916
Title: #916: Fix one coverity issue
pbrezina commented:
"""
* `master`
* 3c09e9dceec016c4f31452506d5ddb012f8a82d7 - sss_ssh_knownhostsproxy: fixed
Coverity issue
* 0102a253eeafee0365f3336ad6bd94606eaf484d - sss_ssh_knownhostsproxy:
relocated
URL: https://github.com/SSSD/sssd/pull/916
Title: #916: Fix one coverity issue
mzidek-rh commented:
"""
Thanks. ACK.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/916#issuecomment-550255907
___
sssd-devel mailing list --
URL: https://github.com/SSSD/sssd/pull/916
Title: #916: Fix one coverity issue
alexey-tikhonov commented:
"""
Hi @mzidek-rh,
I have fixed found issue and rewrote third patch to keep done section (still
fixing Coverity issue).
Please, take a look.
"""
See the full comment at
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
>
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
>
URL: https://github.com/SSSD/sssd/pull/916
Title: #916: Fix one coverity issue
mzidek-rh commented:
"""
I talked to Alexey offline, there are few issues. So adding changes requested.
JFTR:
- a socket file descriptor was replaced by 0 (stdin) by a mistake
- I think the 3rd patch could be left