pluck-cms / pluck

Central repo for pluck cms

Home Page:http://www.pluck-cms.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[vulnerability] xss or other vulns

LLfam opened this issue · comments

commented

Hi! I think regexp in function latinOnlyInput is incorrect. So I find a xss in /data/inc/images.php line42, and I think this problem may lead to other vulnerabilities.
In regexp, brackets have special meanings. [blank]-[underline] means blank to underline in ascii.

Screenshot 2019-07-17 at 19 15 35

I run It trough the validator and you are correct. there is one character missing in the regexp. it should be: [^a-zA-Z0-9.\ \-_]+ (Notice the \ before the -)

Screenshot 2019-07-17 at 19 17 40

I drafted a new dev release. can you test this please? if correct I can push to production.

https://github.com/pluck-cms/pluck/releases/tag/4.7.9-dev4
pluck-4.7.9-dev4.tar.gz

commented

Be careful in coding xD. If find other vulns, I will tell u immediately.

Thank you. I pushed it to production. all admins will get a notification to update when they logon.