search

jbruinaud / WebGoatNet

WebGoat .Net for demos
0 stars 0 forks source link

CX Reflected_XSS_All_Clients @ WebGoatCoins/MainPage.aspx.cs [master] #223

Open jbruinaud opened 4 years ago

jbruinaud commented 4 years ago

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

Method Page_Load at line 22 of WebGoatCoins\MainPage.aspx.cs gets user input for the row 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 22 of WebGoatCoins\MainPage.aspx.cs. This may enable a Cross-Site-Scripting attack.

Severity: High

CWE:79

Checkmarx

Lines: 32 42

Code (Line #32):

                Image1.ImageUrl = "images/logos/" + row["logoFileName"];

Code (Line #42):

                    cell2.Text = row[col].ToString();