Fruity Loop
by in Feature Articles on 2012-07-30They might have been chipping their code out of the living mountain with stone tools and classic ASP, but they weren't primitive. Colt's development team still required code reviews of all of the classic ASP code before it went to production. Sure, most of the time, an ASP code review ended with, "ARGH WHY WHY MY EYES THE PAIN!", but the code still got reviewed.

During one review, Colt noticed something odd about his co-worker's file-upload module. The module was meant to handle pretty small CSV files, and even had a check to ensure they weren't larger than expected. That wasn't the odd part.