Changeset 7329

Timestamp:

03/01/08 19:16:07 (5 years ago)

Author:

drry

Message:

lang/javascript/userscripts/googlereaderfullfeed.user.js:

• removeXSSRisk() を改良しました。
  
  • ほかのパターンが有り得るかもしれないのでまだ考える必要があります。
    
    • まるごと諦めて除去してしまう。
      
    • ホワイトリスト方式。
      

Files:

• lang/javascript/userscripts/googlereaderfullfeed.user.js (modified) (3 diffs)

lang/javascript/userscripts/googlereaderfullfeed.user.js

@@ -103 +103 @@
   var text = res.responseText;
 
-  text = removeXSSRisk(text);
+  text = removeXSSRisks(text);
   if (REMOVE_SCRIPT) text = text.replace(/<script[^>]*>[\S\s]*?<\/script\s*>/gi, "");
   if (REMOVE_H2TAG)  text = text.replace(/<h2[^>]*>[\S\s]*?<\/h2\s*>/gi, "");
@@ -446 +446 @@
 
 // == [Utility] =====================================================
-function removeXSSRisk(text) {
-  var reg = /always/ig;
-  //embedによるXSSの対策処理
-  text = text.replace(/(<\s*?embed[^<>]*?allowScriptAccess\s*?=\s*?["'])([^"']*?)(["'][^<>]*?>)/ig, function(match, l, val, r) {
-    if (reg.test(val)) {
-      return l + "never" + r;
-    } else {
-      return match;
-    }
-  });
-  //objectのparam要素によるXSSの対策処理
-  text = text.replace(/(<\s*?param[^<>]*?name\s*?=\s*?["']\s*?allowScriptAccess[^"']*?["'][^<>]*?value\s*?=\s*?["']\s*?)([^"']*?)(["'][^<>]*?\/>)/ig, function(match, l, val, r) {
-    if (reg.test(val)) {
-      return l + "never" + r;
-    } else {
-      return match;
-    }
-  });
-  //イベントハンドラによるXSSの対策処理
-  text = text.replace(/(<[^>]*?)(on(load|unload|click|dblclick|mousedown|mouseup|mouseover|mouseout|mousemove|keypress)\s*?=\s*?["'][^"']*?["'])/ig, function(match, l, handler, val) {
-    return l;
-  });
-  return text;
+function removeXSSRisks(text) {
+  // <embed allowScriptAccess="always"><object onload =alert(this) ><param value= 'always' name = " allowScriptAccess "/></object>
+  // <embed allowScriptAccess="never"><object ><param value= 'never' name = " allowScriptAccess "/></object>
+  return text
+  // embedによるXSSの対策処理
+    .replace(/(<embed[^>]+?allowScriptAccess\s*=\s*)(?:(["'])\s*always\s*\2|always)(?=[^>]*?>)/gi, "$1$2never$2")
+  // objectのparam要素によるXSSの対策処理
+    .replace(/(<param[^>]+?)(?:(name\s*=\s*(?:(["'])\s*allowScriptAccess\s*\3|allowScriptAccess\s)[^>]*?value\s*=\s*)(?:(["'])\s*always\s*\4|always)(?=[^>]*?\/?>)|(value\s*=\s*)(?:(["'])\s*always\s*\6|always)(?=[^>]*?name\s*=\s*(?:(["'])\s*allowScriptAccess\s*\7|allowScriptAccess)[^>]*?\/?>))/gi, "$1$2$5$4$6never$4$6")
+  // イベントハンドラによるXSSの対策処理
+    .replace(/(<[^>]+?)\son(?:(?:un)?load|(?:dbl)?click|mouse(?:down|up|over|move|out)|key(?:press|down|up)|focus|blur|submit|reset|select|change)\s*=\s*(?:(["'])(?:\\\2|[^"'])*?\2|[^"'][^\s>]*(?=[\s>]))(?=[^>]*?>)/ig, "$1");
 }
 
@@ -513 +500 @@
       text-align : center;
     }
-  ]]></>);
+  ]]></>.toString().replace(/^\s{4}/gm, ""));
   var opacity = 0.9;
   var flash = document.createElement('div');