CX Reflected_XSS_All_Clients @ WebGoatCoins/Catalog.aspx.cs [master]

[jbruinaud]

Reflected_XSS_All_Clients issue exists @ WebGoatCoins/Catalog.aspx.cs in branch master

Method Page_Load at line 17 of WebGoatCoins\Catalog.aspx.cs gets user input for the catRow element. This element’s value then flows through the code without being properly sanitized or validated and is eventually displayed to the user in method Page_Load at line 17 of WebGoatCoins\Catalog.aspx.cs. This may enable a Cross-Site-Scripting attack.

Severity: High

CWE:79

Checkmarx

Lines: 23 27 28 29

---

Code (Line #23):

                lblOutput.Text += "<p/><h2 class='title-regular-2 clearfix'>Category: " + catRow["catName"].ToString() + "</h2><hr/><p/>\n";

---

Code (Line #27):

                    lblOutput.Text += "<img src='./images/products/" + prodRow[3] + "'/><br/>\n";

---

Code (Line #28):

                    lblOutput.Text += "" + prodRow[1] + "<br/>\n";

---

Code (Line #29):

                    lblOutput.Text += "<a href=\"ProductDetails.aspx?productNumber=" + prodRow[0].ToString() + "\"><br/>\n";

---