oldSubscription is never null

[annevk]

Presumably oldSubscription is null the first time you create a push subscription, but this isn't actually stated anywhere. Instead there's an assumption there's always a push subscription holding the old keys presumably resulting in a crash.