diff --git a/patches/chromium/.patches b/patches/chromium/.patches index e8f586de2908d..19a4eab9eff1a 100644 --- a/patches/chromium/.patches +++ b/patches/chromium/.patches @@ -129,3 +129,4 @@ cherry-pick-919b1ffe1fe7.patch cherry-pick-f1dd785e021e.patch cherry-pick-b03797bdb1df.patch posix_replace_doubleforkandexec_with_forkandspawn.patch +cherry-pick-22c61cfae5d1.patch diff --git a/patches/chromium/cherry-pick-22c61cfae5d1.patch b/patches/chromium/cherry-pick-22c61cfae5d1.patch new file mode 100644 index 0000000000000..cc1d425a6ae03 --- /dev/null +++ b/patches/chromium/cherry-pick-22c61cfae5d1.patch @@ -0,0 +1,97 @@ +From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 +From: Austin Sullivan +Date: Thu, 12 May 2022 04:52:20 +0000 +Subject: FSA: Sanitize .url files + +Bug: 1307930 +Change-Id: I7ed3cca5942a5334ba761d269bdd8961fa9d13fe +Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3638698 +Reviewed-by: Marijn Kruisselbrink +Commit-Queue: Marijn Kruisselbrink +Auto-Submit: Austin Sullivan +Cr-Commit-Position: refs/heads/main@{#1002495} + +diff --git a/content/browser/file_system_access/file_system_chooser.cc b/content/browser/file_system_access/file_system_chooser.cc +index 86b9af148a86e64f9f4aedb6e39998bf83668745..f6018224c5f0aea3aba6735d49aaa25d077de600 100644 +--- a/content/browser/file_system_access/file_system_chooser.cc ++++ b/content/browser/file_system_access/file_system_chooser.cc +@@ -276,13 +276,15 @@ bool FileSystemChooser::IsShellIntegratedExtension( + base::FilePath::StringType extension_lower = + base::ToLowerASCII(GetLastExtension(extension)); + +- // .lnk and .scf files may be used to execute arbitrary code (see ++ // '.lnk' and '.scf' files may be used to execute arbitrary code (see + // https://nvd.nist.gov/vuln/detail/CVE-2010-2568 and +- // https://crbug.com/1227995, respectively). .local files are used by Windows +- // to determine which DLLs to load for an application. ++ // https://crbug.com/1227995, respectively). '.local' files are used by ++ // Windows to determine which DLLs to load for an application. '.url' files ++ // can be used to read arbirtary files (see https://crbug.com/1307930). + if ((extension_lower == FILE_PATH_LITERAL("lnk")) || + (extension_lower == FILE_PATH_LITERAL("local")) || +- (extension_lower == FILE_PATH_LITERAL("scf"))) { ++ (extension_lower == FILE_PATH_LITERAL("scf")) || ++ (extension_lower == FILE_PATH_LITERAL("url"))) { + return true; + } + +diff --git a/content/browser/file_system_access/file_system_chooser_browsertest.cc b/content/browser/file_system_access/file_system_chooser_browsertest.cc +index 9ea4db7807f6bbac799452fd138848b2a650d6fd..79dda31bd228e785d54e5486bb4417a75ee62b3a 100644 +--- a/content/browser/file_system_access/file_system_chooser_browsertest.cc ++++ b/content/browser/file_system_access/file_system_chooser_browsertest.cc +@@ -1556,13 +1556,21 @@ IN_PROC_BROWSER_TEST_F(FileSystemChooserBrowserTest, SuggestedName) { + name_infos.push_back({"not_matching.jpg", ListValueOf(".txt"), false, + "not_matching.jpg", false}); + +- // ".lnk", ".local", and ".scf" extensions should be sanitized. +- name_infos.push_back({"dangerous_extension.local", ListValueOf(".local"), +- true, "dangerous_extension.download", false}); ++ // ".lnk", ".local", ".scf", and ".url" extensions should be sanitized. + name_infos.push_back({"dangerous_extension.lnk", ListValueOf(".lnk"), true, + "dangerous_extension.download", false}); ++ name_infos.push_back({"dangerous_extension.lnk", ListValueOf(".LNK"), true, ++ "dangerous_extension.download", false}); ++ name_infos.push_back({"dangerous_extension.LNK", ListValueOf(".lnk"), true, ++ "dangerous_extension.download", false}); ++ name_infos.push_back({"dangerous_extension.LNK", ListValueOf(".LNK"), true, ++ "dangerous_extension.download", false}); ++ name_infos.push_back({"dangerous_extension.local", ListValueOf(".local"), ++ true, "dangerous_extension.download", false}); + name_infos.push_back({"dangerous_extension.scf", ListValueOf(".scf"), true, + "dangerous_extension.download", false}); ++ name_infos.push_back({"dangerous_extension.url", ListValueOf(".url"), true, ++ "dangerous_extension.download", false}); + // Compound extensions ending in a dangerous extension should be sanitized. + name_infos.push_back({"dangerous_extension.png.local", ListValueOf(".local"), + true, "dangerous_extension.png.download", false}); +@@ -1570,6 +1578,8 @@ IN_PROC_BROWSER_TEST_F(FileSystemChooserBrowserTest, SuggestedName) { + true, "dangerous_extension.png.download", false}); + name_infos.push_back({"dangerous_extension.png.scf", ListValueOf(".scf"), + true, "dangerous_extension.png.download", false}); ++ name_infos.push_back({"dangerous_extension.png.url", ListValueOf(".url"), ++ true, "dangerous_extension.png.download", false}); + // Compound extensions not ending in a dangerous extension should not be + // sanitized. + name_infos.push_back({"dangerous_extension.local.png", ListValueOf(".png"), +@@ -1578,6 +1588,8 @@ IN_PROC_BROWSER_TEST_F(FileSystemChooserBrowserTest, SuggestedName) { + true, "dangerous_extension.lnk.png", true}); + name_infos.push_back({"dangerous_extension.scf.png", ListValueOf(".png"), + true, "dangerous_extension.scf.png", true}); ++ name_infos.push_back({"dangerous_extension.url.png", ListValueOf(".png"), ++ true, "dangerous_extension.url.png", true}); + // Invalid characters should be sanitized. + name_infos.push_back({R"(inv*l:d\\ch%rבאמת!a({}), + std::vector( +- {"lnk", "foo.lnk", "foo.bar.local", "text", "local", "scf"}))); ++ {"lnk", "foo.lnk", "foo.bar.local", "text", "local", "scf", "url"}))); + SyncShowDialog(std::move(accepts), /*include_accepts_all=*/false); + + ASSERT_TRUE(dialog_params.file_types);