File: predictor-denial-of-service.patch

package info (click to toggle)
chromium 141.0.7390.65-1
  • links: PTS, VCS
  • area: main
  • in suites: forky, sid
  • size: 6,246,236 kB
  • sloc: cpp: 35,264,825; ansic: 7,169,920; javascript: 4,250,185; python: 1,460,623; asm: 950,788; xml: 751,717; pascal: 187,972; sh: 89,459; perl: 88,691; objc: 79,953; sql: 53,924; cs: 44,622; fortran: 24,137; makefile: 22,313; tcl: 15,277; php: 14,018; yacc: 8,995; ruby: 7,553; awk: 3,720; lisp: 3,096; lex: 1,330; ada: 727; jsp: 228; sed: 36
file content (34 lines) | stat: -rw-r--r-- 1,264 bytes parent folder | download | duplicates (5) 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
--- a/chrome/browser/predictors/loading_data_collector.h
+++ b/chrome/browser/predictors/loading_data_collector.h
@@ -22,6 +22,9 @@
 #include "url/gurl.h"
 #include "url/origin.h"
 
+// base::flat_set is O(n), limit to a reasonable history length
+#define MAX_PREDICTOR_SUBRESOURCE_URL_HISTORY 4096
+
 namespace content {
 class NavigationHandle;
 }  // namespace content
--- a/chrome/browser/predictors/loading_data_collector.cc
+++ b/chrome/browser/predictors/loading_data_collector.cc
@@ -125,8 +125,18 @@
           net::SimplifyUrlForRequest(resource_load_info.final_url);
       if (is_low_priority) {
         low_priority_subresource_urls.insert(final_url);
+        // Prevent history bloat on long running pathological Web sites
+	if (low_priority_subresource_urls.size() >
+              MAX_PREDICTOR_SUBRESOURCE_URL_HISTORY) {
+          low_priority_subresource_urls.erase(
+              low_priority_subresource_urls.begin());
+        }
       } else {
         subresource_urls.insert(final_url);
+        // Prevent history bloat on long running pathological Web sites
+	if (subresource_urls.size() > MAX_PREDICTOR_SUBRESOURCE_URL_HISTORY) {
+          subresource_urls.erase(subresource_urls.begin());
+        }
       }
       return;
   }
k