Quantcast
Channel: WordPress.org Forums » [WP-Members Membership Plugin] Support
Viewing all articles
Browse latest Browse all 112

XSS Issue with TOS field

$
0
0

Replies: 2

Using the TOS checkbox on a form creates an XSS issue. This is due to the fact that the TOS has an anchor that is wrapped around the label with an unsanitized URL (line 2190):

$tos_link_tag = "<a href=\"#\" onClick=\"window.open('" . $tos_link_url . "','tos');\">";

Also, creating an anchor on a label is highly inaccessible.

Recommend reviewing this functionality. Have had to update this to sanitize the url:

$tos_link_tag = "<a href=\"#\" onClick=\"window.open('" . sanitize_url($tos_link_url) . "','tos');\">";


Viewing all articles
Browse latest Browse all 112

Trending Articles