CVE-2026-11294
Overview
Files Changed
components/password_manager/core/browser/sharing/password_receiver_service_impl.cccomponents/password_manager/core/browser/sharing/password_receiver_service_impl_unittest.cc
Patch
From 6524df89a164effa713d11878a576eaded9daf65 Mon Sep 17 00:00:00 2001 From: Mohamed Amir Yosef <[email protected]> Date: Tue, 14 Apr 2026 11:37:17 -0700 Subject: [PATCH] [Passwords] Prevent auto-signin for shared passwords This change ensures that passwords received through the sharing feature do not trigger automatic sign-in without user consent. This is achieved by setting skip_zero_click to true when a shared password is processed. Fixed: 502403953 Change-Id: I6ad4122c2fec098b7044e3dc9ef6b99ffa07b518 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7761765 Commit-Queue: Mohamed Amir Yosef <[email protected]> Reviewed-by: Vasilii Sukhanov <[email protected]> Cr-Commit-Position: refs/heads/main@{#1614623} --- diff --git a/components/password_manager/core/browser/sharing/password_receiver_service_impl.cc b/components/password_manager/core/browser/sharing/password_receiver_service_impl.cc index f8d3dc7..55877f9d 100644 --- a/components/password_manager/core/browser/sharing/password_receiver_service_impl.cc +++ b/components/password_manager/core/browser/sharing/password_receiver_service_impl.cc @@ -140,6 +140,7 @@ form.icon_url = GURL(password_group_element_data.avatar_url()); form.date_created = base::Time::Now(); form.type = PasswordForm::Type::kReceivedViaSharing; + form.skip_zero_click = true; // Invitation metadata. const sync_pb::UserDisplayInfo& sender_info = diff --git a/components/password_manager/core/browser/sharing/password_receiver_service_impl_unittest.cc b/components/password_manager/core/browser/sharing/password_receiver_service_impl_unittest.cc index 2393955..a42e39dc 100644 --- a/components/password_manager/core/browser/sharing/password_receiver_service_impl_unittest.cc +++ b/components/password_manager/core/browser/sharing/password_receiver_service_impl_unittest.cc @@ -245,6 +245,7 @@ Field(&PasswordForm::sender_name, kSenderName), Field(&PasswordForm::sender_profile_image_url, GURL(kSenderProfileImagerUrl)), + Field(&PasswordForm::skip_zero_click, true), Field(&PasswordForm::sharing_notification_displayed, false)))); EXPECT_TRUE(
Regression Test / PoC
diff --git a/components/password_manager/core/browser/sharing/password_receiver_service_impl_unittest.cc b/components/password_manager/core/browser/sharing/password_receiver_service_impl_unittest.cc
index 2393955..a42e39dc 100644
--- a/components/password_manager/core/browser/sharing/password_receiver_service_impl_unittest.cc
+++ b/components/password_manager/core/browser/sharing/password_receiver_service_impl_unittest.cc
@@ -245,6 +245,7 @@
Field(&PasswordForm::sender_name, kSenderName),
Field(&PasswordForm::sender_profile_image_url,
GURL(kSenderProfileImagerUrl)),
+ Field(&PasswordForm::skip_zero_click, true),
Field(&PasswordForm::sharing_notification_displayed, false))));
EXPECT_TRUE(
Original Bug Report
Potential CM API Auto-Signin Bypass via Shared Passwords leads to Login CSRF
Project Fortify, an experimental security project, has identified the following potential security issue. If you’re a feature owner CC-ed on this bug, please do your best to review these reports without the Chrome Security team.
Overview: A potential logic flaw in the password manager allows credentials received via Google Family sharing to be used for Credential Management API (CM API) zero-click auto-signin. The handler for incoming shared credentials fails to set the skip_zero_click flag, causing the auto-signin flow to bypass the mandatory SharedPasswordsNotificationBubble security warning. This could enable an attacker in the same Family group to silently log a victim into an attacker-controlled account (Login CSRF).
Affected files:
components/password_manager/core/browser/sharing/password_receiver_service_impl.cccomponents/password_manager/core/browser/password_form.hcomponents/password_manager/core/browser/credential_manager_pending_request_task.ccchrome/browser/ui/passwords/manage_passwords_ui_controller.ccchrome/browser/password_manager/chrome_password_manager_client.cc
Estimated timestamp from git blame: 2023-11-29
Summary
There is a potential logic flaw in the handling of shared credentials that allows the Credential Management API (CM API) to perform automatic sign-ins (zero-click) using passwords received via Google Family sharing without appropriately notifying the user. This bypasses the SharedPasswordsNotificationBubble, which is the primary security mitigation designed to inform users when a shared credential is used. This can lead to a login-CSRF vulnerability where an attacker (a family member) can silently sign a victim into an attacker-controlled account on a target site.
Root Cause Analysis
The issue originates in components/password_manager/core/browser/sharing/password_receiver_service_impl.cc within the IncomingSharingInvitationToPasswordForms function. When an incoming sharing invitation is parsed into a PasswordForm object, the code accurately sets form.type = PasswordForm::Type::kReceivedViaSharing; but fails to explicitly assign a value to form.skip_zero_click.
Because skip_zero_click is defined with a default value of false in components/password_manager/core/browser/password_form.h, credentials received via sharing are persisted in the password store with skip_zero_click = false.
Potential Attacker Steps & Execution Flow
Note: These are theoretically derived steps based on code analysis; our tooling agent does not execute live code.
- Setup: An attacker and victim are in the same Google Family group. The victim has Chrome Sync enabled and “Auto Sign-in” active (default). The victim has no existing credentials for
https://target.com. - Sharing: The attacker creates an account on
https://target.com(which uses the CM API for auto-signin) and shares the credential with the victim via Chrome. - Sync & Storage: The victim’s Chrome instance processes the incoming invitation via
IncomingSharingInvitationToPasswordForms. The credential is saved to the SQLite database withskip_zero_click = false. - Navigation: The victim navigates to
https://target.com/. - CM API Invocation: The site calls
navigator.credentials.get({password:true}). - Validation Bypass: In
CredentialManagerPendingRequestTask::ProcessForms,IsFormValidForAutoSignInonly checks for exact origin matches, completely ignoringform->type. Because the victim has exactly one credential for the site, auto-signin is permitted. - Zero-Click Execution: The code evaluates
if (can_use_autosignin && !results[0]->skip_zero_click). Because the credential was saved with the defaultfalsevalue, this check passes, and the CM API auto-signin path proceeds. - Notification UI Bypass: The code calls
NotifyUserAutoSignin, triggeringManagePasswordsUIController::OnAutoSignin. Unlike the standard autofill path (OnPasswordAutofilled),OnAutoSigninunconditionally transitions the UI toAUTO_SIGNIN_STATE(displaying a brief toast) and does not check ifform->type == kReceivedViaSharing. - Impact: The mandatory
SharedPasswordsNotificationBubble(which identifies the sender) is never displayed. The victim is silently logged into the attacker’s account.
Suggested Fix
- In
components/password_manager/core/browser/sharing/password_receiver_service_impl.cc, withinIncomingSharingInvitationToPasswordForms, explicitly disable zero-click auto-signin for shared credentials:
form.skip_zero_click = true;
- (Optional but recommended for defense-in-depth) In
CredentialManagerPendingRequestTask::ProcessFormsorIsFormValidForAutoSignIn, explicitly exclude credentials wheretype == PasswordForm::Type::kReceivedViaSharingfrom being eligible for auto-signin.
Evaluated with Chrome root at commit: 096fc8fdbfacf2546485756d03f160a3d04fcc9b
Results so far have been promising, but there can be wrong deductions. If this proves to be a false positive, please close as WAI; data from false positives will be used to improve accuracy over time. And please feel free to reach out to me directly if you have concerns or feedback on the project.