Code Reviews and Other Verification Activities: USELESS Unless Acted Upon IMMEDIATELY
Performing code reviews and other types of verification activities such as security testing is USELESS unless findings from these activities are acted upon IMMEDIATELY by the development team. Failure to act in a timely manner makes attempting fixes much more difficult than they otherwise would have been if action had been taken immediately, or simply not possible.
How can fixing what development teams may at first glance see as mere "bugs" become so much harder or not possible? As time goes on and as development actively continues, code where findings were originally found becomes unrecognizable as it is branched and refactored, and as new (unexamined) code (with potential new findings) is added to branches and the baseline.
In these cases, a new code review or verification becomes necessary. As one might expect however, performing a new code review or verification activity out of necessity in these cases further sets back budgets and timelines. Worse, in the meanwhile, your application AND the data AND systems that it relies on and that it talks to REMAIN VULNERBLE TO ATTACK.
How can one guard against reports from security teams going stale? There are two specific actions that management and development teams can take:
- ADD code reviews and other types of verification activities as a REGULAR activity during YOUR development life cycle, and
- TRACK and TRIAGE findings from these activities with the SAME URGENCY as failed baseline builds and user-reported bugs.
Doing the above will have meant that the management team will have provided the necessary funding AND MANDATE for the security team by allowing the SDLC to be opened up to the addition of a new activity. Since, performing one-time ("drive-by") code reviews and other types of verification activities outside of the SDLC guarantees the reports will sit. Doing the above will also have meant that the development team will have figured out the mechanics of all necessary actions to take to work towards the end of securing their application.
For an example of how to add code reviews and other types of verification activities to an SDLC, see: http://www.owasp.org/index.php/Agile_Software_Development:_Don%27t_Forget_EVIL_User_Stories
For more information about code reviews and other types of verification activities, see: http://www.owasp.org/index.php/ASVS