Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sql Injection Problem #686

Closed
Virusday opened this issue Jul 13, 2022 · 2 comments
Closed

Sql Injection Problem #686

Virusday opened this issue Jul 13, 2022 · 2 comments
Labels

Comments

@Virusday
Copy link

作者您好,我在pageHelper的其他方法上也找到了一些sql的问题,这些参数容易在外部进行传入。
PageHelper.startPage(1, 20).setCountColumn(keyword);
这个方法在您的测试样例中有写到。
setCountColumn方法keyword参数如果是外部传入,会导致sql注入的问题。
期待您的回复。

Hello, author. I also found some SQL problems in other methods of PageHelper. These parameters are easy to be passed in externally.
PageHelper. startPage(1, 20). setCountColumn(keyword);
This method is written in your test sample.
If the keyword parameter of setcountcolumn method is imported externally, it will cause SQL injection problems.
I'm looking forward to your reply.

@abel533
Copy link
Collaborator

abel533 commented Jul 13, 2022

这个字段是给后端使用的,常用的值就是 *, 0, 1

为了避免有人故意制造业务漏洞,后续会限制这里可用的值。

@abel533 abel533 added the bug label Jul 13, 2022
@Virusday
Copy link
Author

感谢您的回复。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants