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

unrestricted file upload vulnerability #59

Open
P0wfuu opened this issue Aug 10, 2021 · 1 comment
Open

unrestricted file upload vulnerability #59

P0wfuu opened this issue Aug 10, 2021 · 1 comment

Comments

@P0wfuu
Copy link

P0wfuu commented Aug 10, 2021

You previously fixed one vulnerability of the theme file upload .
image
image

But now there is a similar vulnerability in /app/admin/appmini.php,because you didn't use the above method.
image
image

http://localhost:3000/admin.php?s=appmini/index/nav_type/weixin/view_type/upload.html
image
image
Then my file will be uploaded to sourcecode/weixin/
image

And i can bypass the !IS_AJAX,even upload my files to any writable directory using ../.
admin.php?s=appmini/themeupload&ajax=ajax
image
image

@gongfuxiang
Copy link
Owner

您之前修复了主题文件上传的一个漏洞。
图像
图像

但是现在有一个类似的漏洞/app/admin/appmini.php,因为你没有使用上面的方法。
图像
图像

http://localhost:3000/admin.php?s=appmini/index/nav_type/weixin/view_type/upload.html
图像
图像
然后我的文件将被上传到 sourcecode/weixin/
图像

而且我可以绕过!IS_AJAX,甚至使用 将我的文件上传到任何可写目录../
admin.php?s=appmini/themeupload&ajax=ajax
图像
图像

非常感谢您的反馈、已确定问题并在 dev 分支进行了优化
9b74bee

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

No branches or pull requests

2 participants