[SSSD] [sssd PR#916][comment] Fix one coverity issue

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

[SSSD] [sssd PR#916][comment] Fix one coverity issue

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 --

[SSSD] [sssd PR#916][comment] Fix one coverity issue

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

[SSSD] [sssd PR#916][comment] Fix one coverity issue

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 >

[SSSD] [sssd PR#916][comment] Fix one coverity issue

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 >

[SSSD] [sssd PR#916][comment] Fix one coverity issue

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