Justin Clarke Lead Author and Technical Editor Rodrigo Marcos Alvarez Dave Hartley Joseph Hemler Alexander Kornbrust Haroon Meer Gary O’Leary-Steele Alberto Revelli Marco Slaviero Dafydd Stuttard

Chapter 3 Reviewing Code for SQL Injection . . . . . . . . . . . . . . . . . . . . . . . . 95
Introduction . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 96
Reviewing Source Code for SQL Injection . . . . . . . . . . . . . . . . . . . . . . . . . . . . 96
Dangerous Coding Behaviors . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 98
Dangerous Functions . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 105
Following the Data . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 109
Following Data in PHP . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 110
Following Data in Java . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 114
Following Data in C# . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 115
Reviewing PL/SQL and T-SQL Code . . . . . . . . . . . . . . . . . . . . . . . . . . . . 117
Automated Source Code Review . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 124
Yet Another Source Code Analyzer (YASCA) . . . . . . . . . . . . . . . . . . . . . . 125
Pixy . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 126
AppCodeScan . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 127

---

 
Contents 
xi
LAPSE . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 127
Security Compass Web Application Analysis Tool (SWAAT) . . . . . . . . . . . . 128
Microsoft Source Code Analyzer for SQL Injection . . . . . . . . . . . . . . . . . . 128
Microsoft Code Analysis Tool .NET (CAT .NET) . . . . . . . . . . . . . . . . . . . . 129
Commercial Source Code Review Tools . . . . . . . . . . . . . . . . . . . . . . . . . . 129
Ounce . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 131
Source Code Analysis . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 131
CodeSecure . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 132
Summary . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 133
Solutions Fast Track . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 133
Frequently Asked Questions . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . 135