So in Visual Studio 2008, I got a GPF in the code that loads the toolbar information from the registry. It turns out that they have an assertion if a particular pointer is NULL, which is good. The assertion has been there for a very long time.
It would be better if they then NULL-tested the pointer before proceeding to use it in the new code below.
Of course, given that we just tripped over a NULL pointer bug in my own code, I suppose I shouldn't laugh too loud.