Browse Source

修复两处存在SQL注入漏洞问题

pull/274/head^2
RuoYi 4 years ago
parent
commit
e1cab589f2
  1. 14
      ruoyi-framework/src/main/java/com/ruoyi/framework/aspectj/DataScopeAspect.java
  2. 2
      ruoyi-system/src/main/resources/mapper/system/SysDeptMapper.xml

14
ruoyi-framework/src/main/java/com/ruoyi/framework/aspectj/DataScopeAspect.java

@ -63,6 +63,7 @@ public class DataScopeAspect
@Before("dataScopePointCut()")
public void doBefore(JoinPoint point) throws Throwable
{
clearDataScope(point);
handleDataScope(point);
}
@ -163,4 +164,17 @@ public class DataScopeAspect
}
return null;
}
/**
* 拼接权限sql前先清空params.dataScope参数防止注入
*/
private void clearDataScope(final JoinPoint joinPoint)
{
Object params = joinPoint.getArgs()[0];
if (StringUtils.isNotNull(params) && params instanceof BaseEntity)
{
BaseEntity baseEntity = (BaseEntity) params;
baseEntity.getParams().put(DATA_SCOPE, "");
}
}
}

2
ruoyi-system/src/main/resources/mapper/system/SysDeptMapper.xml

@ -152,7 +152,7 @@ PUBLIC "-//mybatis.org//DTD Mapper 3.0//EN"
<if test="updateBy != null and updateBy != ''">update_by = #{updateBy},</if>
update_time = sysdate()
</set>
where dept_id in (${ancestors})
where find_in_set(#{deptId}, ancestors)
</update>
</mapper>
Loading…
Cancel
Save