Merge pull request #1788 from weaveworks/fix-memcache-client-miss-and-leak-on-corrupt

fix MemcacheClient.FetchReports miss & leak on corrupt report
This commit is contained in:
Matthias Radestock
2016-08-15 11:33:25 +01:00
committed by GitHub

View File

@@ -186,7 +186,8 @@ func (c *MemcacheClient) FetchReports(keys []string) (map[string]report.Report,
}
reports := map[string]report.Report{}
for i := 0; i < len(keys)-len(missing); i++ {
lenFound := len(keys) - len(missing)
for i := 0; i < lenFound; i++ {
r := <-ch
if r.report == nil {
missing = append(missing, r.key)