30

我正在使用休眠开发应用程序。当我尝试创建登录页面时，出现了 Sql Injection 的问题。我有以下代码：

@Component
@Transactional(propagation = Propagation.SUPPORTS)
public class LoginInfoDAOImpl implements LoginInfoDAO{

@Autowired
private SessionFactory sessionFactory;      
@Override
public LoginInfo getLoginInfo(String userName,String password){
    List<LoginInfo> loginList = sessionFactory.getCurrentSession().createQuery("from LoginInfo where userName='"+userName+"' and password='"+password+"'").list();
    if(loginList!=null )
        return loginList.get(0);
    else return null;   
          }
      }

在这种情况下我将如何防止 Sql Injection？loginInfo 表的创建表语法如下：

create table login_info
  (user_name varchar(16) not null primary key,
  pass_word varchar(16) not null);
4

5 回答 5

27 
Query q = sessionFactory.getCurrentSession().createQuery("from LoginInfo where userName = :name");
q.setParameter("name", userName);
List<LoginInfo> loginList = q.list();

您还有其他选择，请参阅 mkyong 的这篇好文章。

于 2012-12-31T13:48:36.170 回答
20

您需要使用命名参数来避免 sql 注入。此外（与 sql 注入无关，但通常与安全性有关）不返回第一个结果，但使用getSingleResult所以如果由于某种原因有多个结果，查询将失败并出现NonUniqueResultException并且登录不会成功

 Query query= sessionFactory.getCurrentSession().createQuery("from LoginInfo where userName=:userName  and password= :password");
 query.setParameter("username", userName);
 query.setParameter("password", password);
 LoginInfo loginList = (LoginInfo)query.getSingleResult();
于 2012-12-31T13:51:00.760 回答
10

什么是 SQL 注入？

当恶意攻击者可以操纵查询构建过程以便执行与应用程序开发人员最初预期不同的 SQL 语句时，就会发生 SQL 注入

如何防止 SQL 注入攻击

解决方案非常简单直接。您只需要确保始终使用绑定参数：

public PostComment getPostCommentByReview(String review) {
    return doInJPA(entityManager -> {
        return entityManager.createQuery("""
            select p
            from PostComment p
            where p.review = :review
            """, PostComment.class)
        .setParameter("review", review)
        .getSingleResult();
    });
}

现在，如果有人试图破解这个查询：

getPostCommentByReview("1 AND 1 >= ALL ( SELECT 1 FROM pg_locks, pg_sleep(10) )");

SQL注入攻击将被阻止：

Time:1, Query:["select postcommen0_.id as id1_1_, postcommen0_.post_id as post_id3_1_, postcommen0_.review as review2_1_ from post_comment postcommen0_ where postcommen0_.review=?"], Params:[(1 AND 1 >= ALL ( SELECT 1 FROM pg_locks, pg_sleep(10) ))]

JPQL 注射液

SQL 注入也可能在使用 JPQL 或 HQL 查询时发生，如以下示例所示：

public List<Post> getPostsByTitle(String title) {
    return doInJPA(entityManager -> {
        return entityManager.createQuery(
            "select p " +
            "from Post p " +
            "where" +
            "   p.title = '" + title + "'", Post.class)
        .getResultList();
    });
}

上面的 JPQL 查询没有使用绑定参数，因此很容易受到SQL 注入的攻击。

看看当我像这样执行这个 JPQL 查询时会发生什么：

List<Post> posts = getPostsByTitle(
    "High-Performance Java Persistence' and " +
    "FUNCTION('1 >= ALL ( SELECT 1 FROM pg_locks, pg_sleep(10) ) --',) is '"
);

Hibernate 执行以下 SQL 查询：

Time:10003, QuerySize:1, BatchSize:0, Query:["select p.id as id1_0_, p.title as title2_0_ from post p where p.title='High-Performance Java Persistence' and 1 >= ALL ( SELECT 1 FROM pg_locks, pg_sleep(10) ) --()=''"], Params:[()]

动态查询

您应该避免使用字符串连接来动态构建查询的查询：

String hql = " select e.id as id,function('getActiveUser') as name from " + domainClass.getName() + " e ";
Query query=session.createQuery(hql);
return query.list();

如果要使用动态查询，则需要使用 Criteria API：

Class<Post> entityClass = Post.class;
CriteriaBuilder cb = entityManager.getCriteriaBuilder();
CriteriaQuery<Tuple> query = cb.createTupleQuery();
Root<?> root = query.from(entityClass);
query.select(
    cb.tuple(
        root.get("id"),
        cb.function("now", Date.class)
    )
);


return entityManager.createQuery(query).getResultList();
于 2019-01-10T05:39:41.930 回答
2

我想在这里添加一个特殊的 SQL 注入，可以在搜索中使用 Like 查询。

假设我们有一个查询字符串，如下所示：

queryString = queryString + " and c.name like :name";

在设置名称参数时，大多数人通常会使用它。

query.setParameter("name", "%" + name + "%");

现在，如上所述，由于 TypedQuery 和 Hibernate 将默认处理它，因此无法注入像“ 1=1 ”这样的传统参数。

但是这里可能存在特殊的 SQL 注入，这是因为 LIKE 查询结构使用了下划线

下划线通配符用于精确匹配 MySQL 含义中的一个字符，例如 select * from users where user like 'abc_de'；这将产生以 abc 开头、以 de 结尾并且中间只有 1 个字符的用户的输出。

现在，如果在我们的场景中，如果我们设置

  • name="_" 生成名称至少为 1 个字母的客户
  • name="__" 生成名称至少为 2 个字母的客户
  • name="___"生成名称至少为 3 个字母的客户

等等。

理想的修复：

为了缓解这种情况，我们需要使用前缀转义所有下划线。

___ 将变为 \_\_\_ （相当于 3 个原始下划线）

同样，反之亦然的查询也将导致需要转义 % 的注入。

于 2019-03-14T18:02:07.543 回答
0

我们应该始终尝试使用存储过程来防止 SQLInjection。如果存储过程是不可能的；我们应该尝试使用Prepared Statements。

于 2012-12-31T13:50:27.760 回答