SQL 쿼리 형식은 보안에 민감할 수 있습니다.

형식이 지정된 쿼리는 유지보수 및 디버깅이 어려울 수 있으며, 신뢰할 수 없는 값을 쿼리와 결합할 때 SQL Injection 위험을 증가시킬 수 있습니다. 하지만, 이 규칙은 {rule:javasecurity:s3649} 과는 달리 SQL Injection 을 감지하지 않으며, 복잡하고 형식이 지정된 쿼리를 강조하는 것이 목표입니다.

스스로에게 자문하세요

쿼리의 일부는 신뢰할 수 없는 값(사용자 입력과 같은)에서 가져옵니다.
쿼리가 코드의 다른 부분에서 반복 또는 중복됩니다.
애플리케이션은 서로 다른 유형의 관계형 데이터베이스를 지원해야 합니다.

이러한 질문에 예라고 답하는 경우 문제가 발생할 수 있습니다.

민감한 코드의 예

public User getUser(Connection con, String user) throws SQLException {
    Statement stmt1 = null;
    Statement stmt2 = null;
    PreparedStatement pstmt;
    try {
      stmt1 = con.createStatement();
      ResultSet rs1 = stmt1.executeQuery("GETDATE()"); // 문제 없음; 하드코딩된 쿼리

      stmt2 = con.createStatement();
      ResultSet rs2 = stmt2.executeQuery("select FNAME, LNAME, SSN " +
      "from USERS where UNAME=" + user);  // 민감한 코드

      pstmt = con.prepareStatement("select FNAME, LNAME, SSN " +
      "from USERS where UNAME=" + user);  // 민감한 코드
      ResultSet rs3 = pstmt.executeQuery();

    //...
}

public User getUserHibernate(org.hibernate.Session session, String data) {
    org.hibernate.Query query = session.createQuery(
    "FROM students where fname = " + data);  // Sensitive
    // ...
}

규칙을 준수한 해결책

public User getUser(Connection con, String user) throws SQLException {

  Statement stmt1 = null;
  PreparedStatement pstmt = null;
  String query = "select FNAME, LNAME, SSN " +
                 "from USERS where UNAME=?"
  try {
    stmt1 = con.createStatement();
    ResultSet rs1 = stmt1.executeQuery("GETDATE()");

    pstmt = con.prepareStatement(query);
    pstmt.setString(1, user);  // 좋은 코드; PreparedStatements 은 인풋을 escape 처리합니다.
    ResultSet rs2 = pstmt.executeQuery();

    //...
  }
}

public User getUserHibernate(org.hibernate.Session session, String data) {

  org.hibernate.Query query =  session.createQuery("FROM students where fname = ?");
  query = query.setParameter(0,data);  // Good; Parameter binding escapes all input

  org.hibernate.Query query2 =  session.createQuery("FROM students where fname = " + data); // 민감한 코드
  // ...