Yeah I saw this on digg...
His point, if I understand it, relates to the fact that each metafile record has a four-byte length value, the minimum length is 6, and that when you incorrectly set the length to a specific value -- 1 -- Windows starts executing the next byte in the metafile.
Apparently if you use other invalid record lengths -- 0, 2, 3, 4, 5 -- nothing bad happens. The only invalid length that causes execution of a WMF is "1."
I think his contention is that, if it was a bug, other invalid lengths should also trigger the execution of code. (e.g. a length of "0" or "2" should cause the same problem).
Steve: And so, you know, because I'm a developer when I'm not being a hacker, I wanted to understand - oh, and the other thing is, I want to write a robust testing application, you know, that always works all the time. So I wanted to know, like, okay, what bytes have to be set which way, what matters, what doesn't. Because, you know, that's the way you get something that is as solid as, you know, the code that I put out from GRC. So what I found was that, when I deliberately lied about the size of this record and set the size to one and no other value, and I gave this particular byte sequence that makes no sense for a metafile, then Windows created a thread and jumped into my code, began executing my code. Okay, Leo? This was not a mistake. This is not buggy code. This was put into Windows by someone. We are never going to know who.
I think going all X-Files and saying "This was not a mistake" is a little premature at this point... it's worthy of investigation, of trying to figure out why a "1" gets treated as a special invalid value, but you need to gather a bit more evidence before you go around making accusations like that...