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

[jbruinaud]

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

Method LoadComments at line 53 of WebGoatCoins\ProductDetails.aspx.cs gets user input for the prodRow 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 LoadComments at line 53 of WebGoatCoins\ProductDetails.aspx.cs. This may enable a Cross-Site-Scripting attack.

Severity: High

CWE:79

Checkmarx

Lines: 64 65 68 76 77 63

---

Code (Line #64):

                output += "<strong>" + prodRow["productName"].ToString() + "</strong><br/>";

---

Code (Line #65):

                output += "<hr/>" + prodRow["productDescription"].ToString() + "<br/>";

---

Code (Line #68):

                hiddenFieldProductID.Value = prodRow["productCode"].ToString();

---

Code (Line #76):

                    comments += "<strong>Email:</strong>" + commentRow["email"] + "<span style='font-size: x-small;color: #E47911;'> (Email Address Verified!) </span><br/>";

---

Code (Line #77):

                    comments += "<strong>Comment:</strong><br/>" + commentRow["comment"] + "<br/><hr/>";

---

Code (Line #63):

                output += "<img src='./images/products/" + prodRow["productImage"] + "'/><br/>";

---