[SSSD] [sssd PR#341][comment] CACHE_REQ_SR_OVERLAY: Fix coverity warning

2017-07-31 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/341 Title: #341: CACHE_REQ_SR_OVERLAY: Fix coverity warning lslebodn commented: """ master: * 9d90396775715de66e735dd55826e35f311450af """ See the full comment at https://github.com/SSSD/sssd/pull/341#issuecomment-319028108 ___

[SSSD] [sssd PR#341][comment] CACHE_REQ_SR_OVERLAY: Fix coverity warning

2017-07-31 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/341 Title: #341: CACHE_REQ_SR_OVERLAY: Fix coverity warning lslebodn commented: """ BTW gcc on el7 complained as well. """ See the full comment at https://github.com/SSSD/sssd/pull/341#issuecomment-319027152 ___

[SSSD] [sssd PR#341][comment] CACHE_REQ_SR_OVERLAY: Fix coverity warning

2017-07-31 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/341 Title: #341: CACHE_REQ_SR_OVERLAY: Fix coverity warning fidencio commented: """ Adding the "Accepted" label as per @spbnick's review. """ See the full comment at https://github.com/SSSD/sssd/pull/341#issuecomment-319000400

[SSSD] [sssd PR#341][comment] CACHE_REQ_SR_OVERLAY: Fix coverity warning

2017-07-31 Thread spbnick
URL: https://github.com/SSSD/sssd/pull/341 Title: #341: CACHE_REQ_SR_OVERLAY: Fix coverity warning spbnick commented: """ Yes, this is absolutely correct, thank you, Fabiano, ACK! """ See the full comment at https://github.com/SSSD/sssd/pull/341#issuecomment-318995775 _

[SSSD] [sssd PR#341][comment] CACHE_REQ_SR_OVERLAY: Fix coverity warning

2017-07-31 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/341 Title: #341: CACHE_REQ_SR_OVERLAY: Fix coverity warning fidencio commented: """ @spbnick, please, whenever you have time, take a look on this patch fixing an issue found by coverity. """ See the full comment at https://github.com/SSSD/sssd/pull/341#i