New issue
Advanced search Search tips

Issue 719281 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug



Sign in to add a comment

AutocompleteSyncBridge::GetAutofillTable() can pass null to AutofillTable::FromWebDatabase() via AutocompleteSyncBridge::LoadMetadata()

Project Member Reported by tapted@chromium.org, May 8 2017

Issue description

Chrome Version       : @r468261 ~  60.0.3080.5
OS Version: OS X 10.12.4

Possible regression from r444840 ->  https://codereview.chromium.org/2620783002

I started getting a crash at startup. GetAutofillTable() is passing null to FromWebDatabase() and forming a null `this` pointer.

    frame #4: 0x000000012e0a72d8 libwebdata_common.dylib`WebDatabase::GetTable(this=0x0000000000000000, key=0x000000010b47c384) + 56 at web_database.cc:57
    frame #5: 0x00000001069d4275 libchrome_dll.dylib`autofill::AutofillTable::FromWebDatabase(db=0x0000000000000000) + 37 at autofill_table.cc:419
    frame #6: 0x000000010699c0d5 libchrome_dll.dylib`autofill::AutocompleteSyncBridge::GetAutofillTable(this=0x0000000100418f00) const + 37 at autocomplete_sync_bridge.cc:488
    frame #7: 0x000000010699b386 libchrome_dll.dylib`autofill::AutocompleteSyncBridge::LoadMetadata(this=0x0000000100418f00) + 70 at autocomplete_sync_bridge.cc:452


There are some warnings:

[94096:31491:0508/104404.238276:WARNING:web_database.cc(111)] Web database is too new.
[94096:31491:0508/104404.238426:ERROR:web_database_backend.cc(113)] Cannot initialize the web database: 2


Full stack attached.

Blowing away my Chromium profile will probably fix it, but there may be a latent bug here. (Or this could be annoying to encounter while doing bisects or other developer flows).

Maybe AutocompleteSyncBridge::LoadMetadata() needs to check GetAutofillTable() for returning null (and GetAutofillTable() needs to return null when the database didn't initialize). Or something.
 
fullstack.txt
16.3 KB View Download
Owner: pnoland@chromium.org
Labels: TE-NeedsTriageHelp
Status: Started (was: Unconfirmed)
Status: Fixed (was: Started)

Sign in to add a comment