Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

win: Crash in autofill::ShowSaveBubble #3796

Closed
mattwildgoose opened this issue Oct 2, 2024 · 9 comments
Closed

win: Crash in autofill::ShowSaveBubble #3796

mattwildgoose opened this issue Oct 2, 2024 · 9 comments
Labels
alloy-style Related to Chrome runtime + Alloy style bug Bug report needs user feedback Additional feedback required windows Windows platform

Comments

@mattwildgoose
Copy link

mattwildgoose commented Oct 2, 2024

Describe the bug
When running a .NET 4.8 WinForms application using the CefSharp ChromiumWebBrowser control and navigating to a html page containing inputs for Street, Town and Postcode, typing any value in all these fields then submitting the form, the hosting application crashes with a fault in libcef.dll

To Reproduce
I have created a sample repo at https://github.com/mattwildgoose/CefSharpIssue. This contains both a sample WinForms app and a web app with a simple html file used to reproduce the issue.

Steps to reproduce the behavior:

  1. Create a simple WinForms app with a form containing a CefSharp browser control
  2. Ensure you set the CefSettings.Locale value to "en-GB" (I used current culture but that's the value that's being set)
  3. Navigate to a page containing inputs with Street, Town and Postcode in their names and a submit button
  4. Type a dot into each field and click the submit button
  5. The browser's hosting application crashes

Expected behavior
The form should submit successfully and browser remain functional

Versions (please complete the following information):

  • OS: Windows 11
  • CEF Version: Issue started in CefSharp 126.2.180 but also tested with 128.4.90 and issue persists

Additional context
Does the problem reproduce with the cefclient or cefsimple sample application at the same version?
I tried to reproduce with the cefclient trying each of the following command lines but could not reproduce with any
.\cefclient.exe --multi-threaded-message-loop --no-sandbox --use-alloy-style
.\cefclient.exe --multi-threaded-message-loop --no-sandbox --use-alloy-style --lang=en-GB
.\cefclient.exe --multi-threaded-message-loop --no-sandbox --use-alloy-style --enable-chrome-runtime --lang=en-GB

With the cefclient after submitting the form it opens the dialog asking if you want to save the address against your profile, this isn't available when using CefSharp to my knowledge.

Does the problem reproduce with Google Chrome at the same version?
The problem does not reproduce in Google Chrome which opens the same dialog as cefclient asking about saving the address

Add any other context about the problem here.
The problem appears to have come in with the switch from Alloy to Chrome Bootstrap and reverting to Alloy resolves the problem. Also if you don't set the CefSettings.Locale and leave as default (en-US) then you don't get the issue either.

On the CefSharp issue related to the change to the Chrome Bootstrap it requests to log related issues here, hence why I'm raising here rather than in the CefSharp repo.

The exception you get is
Unhandled exception at 0x00007FFE050B86D4 (libcef.dll) in Browser.exe.43284.dmp: 0xC0000005: Access violation reading location 0x00000000000001F8.

Stack Trace:
StackTrace.txt

@mattwildgoose mattwildgoose added the bug Bug report label Oct 2, 2024
@mattwildgoose mattwildgoose reopened this Oct 3, 2024
@mattwildgoose mattwildgoose changed the title Winforms: Submitting a form with newly entered address details causes the hosting app to crash CefSharp Winforms: Submitting a form with newly entered address details causes the hosting app to crash Oct 3, 2024
@magreenblatt magreenblatt changed the title CefSharp Winforms: Submitting a form with newly entered address details causes the hosting app to crash win: Crash in autofill::ShowSaveBubble Oct 3, 2024
@magreenblatt
Copy link
Collaborator

magreenblatt commented Oct 3, 2024

From the call stack (and code here) it looks like another instance of #3763 (comment) (the part about FindBrowserWithTab).

@magreenblatt
Copy link
Collaborator

You can disable autofill as described at https://www.magpcss.org/ceforum/viewtopic.php?f=6&t=19898&start=10#p55813

@magreenblatt magreenblatt added windows Windows platform alloy-style Related to Chrome runtime + Alloy style labels Oct 3, 2024
@magreenblatt
Copy link
Collaborator

@mattwildgoose Can you provide the HTML (or URL) for reproducing these steps?

Navigate to a page containing inputs with Street, Town and Postcode in their names and a submit button
Type a dot into each field and click the submit button

@magreenblatt magreenblatt added the needs user feedback Additional feedback required label Oct 3, 2024
@magreenblatt
Copy link
Collaborator

Can you provide the HTML (or URL) for reproducing these steps?

Thanks, found it at https://github.com/mattwildgoose/CefSharpIssue/blob/main/WebApp/wwwroot/Profile.html

@magreenblatt magreenblatt removed the needs user feedback Additional feedback required label Oct 3, 2024
@magreenblatt
Copy link
Collaborator

I'm also unable to reproduce this crash in cefclient (M130). The autofill bubble works as expected with --use-alloy-style [--use-native].

@magreenblatt
Copy link
Collaborator

Looks like the autocomplete code was substantially rewritten in https://issuetracker.google.com/issues/40281981 (~M129 timeframe), and this crashing code path is no longer triggered.

@magreenblatt
Copy link
Collaborator

@mattwildgoose Can you test if this issue is resolved for you in M129? Thanks.

@magreenblatt magreenblatt added the needs user feedback Additional feedback required label Oct 4, 2024
@magreenblatt
Copy link
Collaborator

Closing this issue as likely fixed in M130 (and maybe M129).

@zelpnir
Copy link

zelpnir commented Nov 7, 2024

I had a similar problem using M129:
On my page was a Field called "IBAN" and if you enter a valid IBAN number the whole application would crash.
We assume that the problem is that the chromium autofill for payments was triggered.

Starting up with the --disable-features flag did not work.

My workaround was to disable the settings in an CustomLifespanHandler.

    internal class CustomLIfeSpanHandler : CefSharp.Handler.LifeSpanHandler, ILifeSpanHandler
    {
        protected override void OnAfterCreated(IWebBrowser chromiumWebBrowser, IBrowser browser)
        {
            var context = Cef.GetGlobalRequestContext();
            string errorString = null;
            context.SetPreference("autofill.save_data", false, out errorString);
            context.SetPreference("autofill.credit_card_enabled", false, out errorString);
            context.SetPreference("autofill.profile_enabled", false, out errorString);
            context.SetPreference("autofill.save_data", false, out errorString);
            context.SetPreference("payments.can_make_payment_enabled", false, out errorString);
            context.SetPreference("search.suggest_enabled", false, out errorString);
            context.SetPreference("url_keyed_anonymized_data_collection.enabled", false, out errorString);
        }
    }

Maybe this helps someone.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alloy-style Related to Chrome runtime + Alloy style bug Bug report needs user feedback Additional feedback required windows Windows platform
Projects
None yet
Development

No branches or pull requests

3 participants