Browse Source

fixes #1075 - XSS in "Active Devices" Panel of Admin Dashboard

Luke 9 years ago
parent
commit
9c3119808b

+ 12 - 1
MediaBrowser.Server.Implementations/HttpServer/Security/AuthorizationContext.cs

@@ -175,11 +175,22 @@ namespace MediaBrowser.Server.Implementations.HttpServer.Security
 
 
                 if (param.Length == 2)
                 if (param.Length == 2)
                 {
                 {
-                    result.Add(param[0], param[1].Trim(new[] { '"' }));
+					var value = NormalizeValue (param[1].Trim(new[] { '"' }));
+                    result.Add(param[0], value);
                 }
                 }
             }
             }
 
 
             return result;
             return result;
         }
         }
+
+		private string NormalizeValue(string value)
+		{
+			if (string.IsNullOrWhiteSpace (value)) 
+			{
+				return value;
+			}
+
+			return System.Net.WebUtility.HtmlEncode(value);
+		}
     }
     }
 }
 }