HexBrain

Ian Dunn · Report #145086

Stored XSS in SupportFlow Ticket Subject

Validstatus
$50bounty
5upvotes
28 Jun 2016disclosed

How it was found

SupportFlow contains an XSS vulnerability in how it handles ticket subjects in the admin-side ticket form, because the subject is not escaped before being used in the value attribute of the subject input element.

This first requires wptexturize to be disabled (not that uncommon):

add_filter( 'run_wptexturize', '__return_false' );

Then, create a ticket with a subject like this:

"><script>alert('hi');</script>

The next time someone views that ticket (any other user) in the admin area at SupportFlow -> All Tickets -> [Ticket], XSS is triggered.

This is due to this line not using esc_attr() on the title value (core does not do this automatically):

https://github.com/SupportFlow/supportflow/blob/71a6053848c523f7b50b61a1f3770013badc76c0/classes/class-supportflow-admin.php#L905

I've attached a screenshot demonstrating the XSS alert - please let me know if you have any questions or if I can clarify anything.

This report was disclosed publicly on HackerOne and belongs to its author and the program. See the original