This project is read-only.
1
Vote

GoogleSafeBrowsing: critical bug when removing existing keys

description

In Synchronizer.cs, the Update method crashes if keys need to be removed.
Can't perform a "keys.Remove(key)" inside the foreach statement.
We could use an old style code instead:

for (int i = keys.Count - 1; i >= 0; i--)
{
Key key = keys[i];
if (key.Type == KeyType.Deletion)
{
DataProviderManager.Provider.RemoveKey(key.Value, listType);
keys.Remove(key);
}
}

comments