这两天在上课时被同学拿了一段代码问我,这段代码有什么问题,我看了一会说:Connection和PreparedStatement都没关。他说不止这方面的问题,还有sql注入的问题,我就坚决的说使用了占位符不存在sql注入的问题,但是他提出了一种情况,在我看来也很有道理的情况。
pstmt = conn.prepareStatement("delete from user where user.id=?");
pstmt.setString(1, "w");
他认为如果把代码写成这样就有注入问题了
pstmt = conn.prepareStatement("delete from user where user.id=?"); pstmt.setString(1, "w‘ or ‘2‘=‘2");
当时我看了只能告诉他一定不存在注入问题,因为在我的想法中我一直记得的是用占位符能解决注入问题,至于怎么解决的就不知道了,看了上面的代码也很有道理,感觉setString后的sql语句应该是
delete from user where user.id=‘w‘ or ‘2‘=‘2‘;
回到宿舍我专门写了程序测试一下,事实证明并不想我们想的这样,的确使用占位符不存在注入问题,所以解释是在执行的时候把一些字符给转义了,但这个转义的过程是在什么地方转义的呢,把上面的sql语句在mysql控制台上运行一下,查看一下数据看到所有数据都被删除完,那只能解释成在java程序中转义的,于是我就去看java的源代码,发现在java源码中PreparedStatement只是一个接口,而且是没有子类的接口,我就很纳闷,没实现怎么用的?所以一定有实现的地方,去网上查了一下,jdk直提供接口,而具体实现是由数据库厂商实现的,我们用的就是数据库厂商实现的类。于是我就又去查mysql的jar包源码,发现有个PreparedStatement实现了jdk中的PreparedStatement了。里面的setString方法如下实现:
public void setString(int parameterIndex, String x) throws SQLException { // if the passed string is null, then set this column to null if (x == null) { setNull(parameterIndex, Types.CHAR); } else { StringBuffer buf = new StringBuffer((int) (x.length() * 1.1)); buf.append(‘\‘‘); int stringLength = x.length(); // // Note: buf.append(char) is _faster_ than // appending in blocks, because the block // append requires a System.arraycopy().... // go figure... // for (int i = 0; i < stringLength; ++i) { char c = x.charAt(i); switch (c) { case 0: /* Must be escaped for ‘mysql‘ */ buf.append(‘\\‘); buf.append(‘0‘); break; case ‘\n‘: /* Must be escaped for logs */ buf.append(‘\\‘); buf.append(‘n‘); break; case ‘\r‘: buf.append(‘\\‘); buf.append(‘r‘); break; case ‘\\‘: buf.append(‘\\‘); buf.append(‘\\‘); break; case ‘\‘‘: buf.append(‘\\‘); buf.append(‘\‘‘); break; case ‘"‘: /* Better safe than sorry */ if (this.usingAnsiMode) { buf.append(‘\\‘); } buf.append(‘"‘); break; case ‘\032‘: /* This gives problems on Win32 */ buf.append(‘\\‘); buf.append(‘Z‘); break; default: buf.append(c); } } buf.append(‘\‘‘); String parameterAsString = buf.toString(); byte[] parameterAsBytes = null; if (!this.isLoadDataQuery) { parameterAsBytes = StringUtils.getBytes(parameterAsString, this.charConverter, this.charEncoding, this.connection .getServerCharacterEncoding(), this.connection .parserKnowsUnicode()); } else { // Send with platform character encoding parameterAsBytes = parameterAsString.getBytes(); } setInternal(parameterIndex, parameterAsBytes); } }
到此就告一段落,可以发现在setString时最外面的单引号被转义了,也就是说setString后的sql语句是这样的
delete from user where user.id=\‘w‘ or ‘2‘=‘2\‘;
而且仔细看会发现在setString中是一个字符一个字符的解析,该转义的都已经转义,正如他一句注释中写的Better safe than sorry.所以最终,占位符确实不存在注入问题