File: predictor-denial-of-service.patch

package info (click to toggle)
chromium 134.0.6998.117-1~deb12u1
  • links: PTS, VCS
  • area: main
  • in suites: bookworm-proposed-updates
  • size: 5,929,832 kB
  • sloc: cpp: 34,190,898; ansic: 7,041,458; javascript: 4,034,221; python: 1,459,275; asm: 846,276; xml: 726,586; pascal: 178,372; sh: 104,683; objc: 91,089; perl: 88,388; sql: 48,244; cs: 43,980; makefile: 28,768; fortran: 24,137; php: 20,829; ruby: 17,806; tcl: 10,142; yacc: 8,631; lisp: 3,097; lex: 1,327; ada: 727; awk: 329; jsp: 228; sed: 36
file content (34 lines) | stat: -rw-r--r-- 1,264 bytes parent folder | download | duplicates (3) 
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