diff --git a/server/src/webview2/skin_css_policy.cpp b/server/src/webview2/skin_css_policy.cpp index 4dd1db34..9daa5d31 100644 --- a/server/src/webview2/skin_css_policy.cpp +++ b/server/src/webview2/skin_css_policy.cpp @@ -214,9 +214,8 @@ bool IsRemoteImport(const std::wstring &rule) if (cursor >= rule.size() || rule[cursor] != L'(') return true; cursor = SkipCssTrivia(rule, cursor + 1); - const wchar_t quote = cursor < rule.size() && (rule[cursor] == L'\'' || rule[cursor] == L'"') - ? rule[cursor++] - : 0; + const wchar_t quote = + cursor < rule.size() && (rule[cursor] == L'\'' || rule[cursor] == L'"') ? rule[cursor++] : 0; const size_t start = cursor; while (cursor < rule.size()) { diff --git a/server/tests/src/test_skin_css_policy.cpp b/server/tests/src/test_skin_css_policy.cpp index 31da6319..f93a3c13 100644 --- a/server/tests/src/test_skin_css_policy.cpp +++ b/server/tests/src/test_skin_css_policy.cpp @@ -87,10 +87,8 @@ TEST_CASE(skin_css_import_stripping_only_matches_a_real_at_rule) std::wstring(L"@charset \"utf-8\";.b{}")); REQUIRE_EQ(StripRemoteImports(L"/* skin comment */ @import \"https://attacker.example/x\";.b{}"), std::wstring(L"/* skin comment */ .b{}")); - REQUIRE_EQ(StripRemoteImports(L"@import \"https:\\/\\/attacker.example/x\";.b{}"), - std::wstring(L".b{}")); - REQUIRE_EQ(StripRemoteImports(L"@import \"https\\3a \\2f\\2fattacker.example/x\";.b{}"), - std::wstring(L".b{}")); + REQUIRE_EQ(StripRemoteImports(L"@import \"https:\\/\\/attacker.example/x\";.b{}"), std::wstring(L".b{}")); + REQUIRE_EQ(StripRemoteImports(L"@import \"https\\3a \\2f\\2fattacker.example/x\";.b{}"), std::wstring(L".b{}")); } TEST_CASE(skin_css_local_imports_and_plain_stylesheets_are_untouched)