I was in a code review this week, and we came across the mistake of locking the type of a static class for thread sychronization (probably because it was documented in MSDN to do it this way).
http://msdn.microsoft.com/library/default.asp?url=/archive/en-us/dnaraskdr/html/askgui06032003.asp