jflyfox / jfinal_cms

jfinal cms是一个java开发的功能强大的信息咨询网站,采用了简洁强大的JFinal作为web框架,模板引擎用的是beetl,数据库用mysql,前端bootstrap框架。支持oauth2认证、帐号注册、密码加密、评论及回复,消息提示,网站访问量统计,文章评论数和浏览量统计,回复管理,支持权限管理。后台模块包含:栏目管理,栏目公告,栏目滚动图片,文章管理,回复管理,意见反馈,我的相册,相册管理,图片管理,专辑管理、视频管理、缓存更新,友情链接,访问统计,联系人管理,模板管理,组织机构管理,用户管理,角色管理,菜单管理,数据字典管理。
http://mtg.jflyfox.com/
Apache License 2.0
627 stars 285 forks source link

SQL injection in user role listing #12

Closed ztz472947849 closed 5 years ago

ztz472947849 commented 5 years ago

admin login required.

RoleController.java

public void list() {
        SysRole model = getModelByAttr(SysRole.class);

        SQLUtils sql = new SQLUtils(" from sys_role t where 1=1 ");
        if (model.getAttrValues().length != 0) {
            sql.setAlias("t");
            // 查询条件
            sql.whereLike("name", model.getStr("name"));
        }

SQLUtils.java

    public void whereLike(String attrName, String value) {
        if (StrUtils.isNotEmpty(value)) {
            sqlBuffer.append(" AND " + getAttrName(attrName) + " LIKE '%").append(value).append("%'");
        }
    }

query here doesn't go through ORM nor data binding procedures. full error msg appears in response too.

most queries DO go through data binding procedures. image The whereLike implementation is vulnerable to SQL injection. image image

EDIT: add more detail

zcool321 commented 5 years ago

已处理,下版本更新