Good information to have. I was surprised by step 2 though (rm login.keychain-db). How can you be absolutely sure it doesn't contain anything important and you won't need it later?
I'd probably opt for a more defensive action here and just rename it (like the original reset did).
I'm hoping that was just the blog version of what they did (since more succinct) but yes, I have so many "-CURRENTDATE-EXPLAINATION.ext" files for any flat-file databases I interact with (keychain, sqlite, db4, etc). It's saved me more times than I can count.
Going in to fix a service that uses sqlite and seeing 5 other times I recovered data or was making a change is always fun.
I left that bit of information out of the post but I actually had a copy of the Keychains directory before I mv-ed login-1.keychain-db to login.keychain-db.
Since that confirmed my hunch it probably gave the sense I was confident in moving instead of renaming.
I'd probably opt for a more defensive action here and just rename it (like the original reset did).