mirror of
https://github.com/mm201/pkmn-classic-framework.git
synced 2026-10-01 13:46:56 -05:00
Fixed (rather hilarious) XSS injection possibility on trainer names.
This commit is contained in:
@@ -57,7 +57,7 @@ namespace PkmnFoundations.GTS
|
||||
protected String CreateTrainer4(object DataItem)
|
||||
{
|
||||
GtsRecord4 record = (GtsRecord4)DataItem;
|
||||
return record.TrainerName.Text;
|
||||
return Common.HtmlEncode(record.TrainerName.Text);
|
||||
}
|
||||
|
||||
protected String CreateOffer5(object DataItem)
|
||||
@@ -75,7 +75,7 @@ namespace PkmnFoundations.GTS
|
||||
protected String CreateTrainer5(object DataItem)
|
||||
{
|
||||
GtsRecord5 record = (GtsRecord5)DataItem;
|
||||
return record.TrainerName.Text;
|
||||
return Common.HtmlEncode(record.TrainerName.Text);
|
||||
}
|
||||
|
||||
|
||||
|
||||
@@ -72,6 +72,9 @@ namespace PkmnFoundations.GTS
|
||||
return result.ToArray();
|
||||
}
|
||||
|
||||
|
||||
public static string HtmlEncode(string s)
|
||||
{
|
||||
return HttpUtility.HtmlEncode(s);
|
||||
}
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user