New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 321370 link

Starred by 5 users

Issue metadata

Status: Fixed
Not on Chrome anymore
Closed: Feb 2014
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug

Sign in to add a comment

Chrome: Crash Report - Magic Signature: syncer::SyncMergeResult::model_type

Project Member Reported by, Nov 19 2013

Issue description

Product: Chrome
Stack Signature: syncer::SyncMergeResult::model_type()-168BDD2
New Signature Label: syncer::SyncMergeResult::model_type()
New Signature Hash: b8a04e5e_7b7c5b06_5d7e203c_8104ec47_9ed913b5

Report link: http://go/crash-old/reportdetail?reportid=5a09e5c90fa57809

Meta information:
Product Name: Chrome
Product Version: 32.0.1700.19
Report ID: 5a09e5c90fa57809
Report Time: 2013/11/19 21:11:46, Tue
Uptime: 0 sec
Cumulative Uptime: 0 sec
OS Name: Windows NT

OS Version: 6.1.7601 Service Pack 1
CPU Architecture: x86
CPU Info: GenuineIntel family 6 model 37 stepping 2


0x6541d03e	 [chrome.dll]	 -]	syncer::SyncMergeResult::model_type()
0x65990caf	 [chrome.dll]	 -]	ProfileOAuth2TokenService::AccountInfo::SetLastAuthError(GoogleServiceAuthError const &)
0x65990d58	 [chrome.dll]	 -]	ProfileOAuth2TokenService::UpdateAuthError(std::basic_string<char,std::char_traits<char>,std::allocator<char> > const &,GoogleServiceAuthError const &)
0x65990dd7	 [chrome.dll]	 -]	ProfileOAuth2TokenService::Observe(int,content::NotificationSource const &,content::NotificationDetails const &)
0x652dc5b6	 [chrome.dll]	 -]	content::NotificationServiceImpl::Notify(int,content::NotificationSource const &,content::NotificationDetails const &)
0x658da8ca	 [chrome.dll]	 -]	TokenService::EraseTokensFromDB()
0x658b1ee8	 [chrome.dll]	 -]	SigninManager::SignOut()
0x655f8a66	 [chrome.dll]	 -]	PrefChangeRegistrar::InvokeUnnamedCallback(base::Callback<void (void)> const &,std::basic_string<char,std::char_traits<char>,std::allocator<char> > const &)
0x655f8a00	 [chrome.dll]	 - bind_internal.h:1224]	base::internal::Invoker<1,base::internal::BindState<base::internal::RunnableAdapter<void (*)(base::Callback<void (void)> const &,std::basic_string<char,std::char_traits<char>,std::allocator<char> > const &)>,void (base::Callback<void (void)> const &,std::basic_string<char,std::char_traits<char>,std::allocator<char> > const &),void (base::Callback<void (void)>)>,void (base::Callback<void (void)> const &,std::basic_string<char,std::char_traits<char>,std::allocator<char> > const &)>::Run(base::internal::BindStateBase *,std::basic_string<char,std::char_traits<char>,std::allocator<char> > const &)
0x6567b773	 [chrome.dll]	 - bind_internal.h:1169]	base::internal::Invoker<1,base::internal::BindState<base::Callback<base::PlatformFileError (void *)>,void (void *),void (void *)>,void (void *)>::Run(base::internal::BindStateBase *)
0x6531323f	 [chrome.dll]	 -]	subtle::PrefMemberBase::Internal::UpdateValue(base::Value *,bool,bool,base::Callback<void (void)> const &)
0x65312f01	 [chrome.dll]	 -]	subtle::PrefMemberBase::UpdateValueFromPref(base::Callback<void (void)> const &)
0x6589efb6	 [chrome.dll]	 -]	subtle::PrefMemberBase::OnPreferenceChanged(PrefService *,std::basic_string<char,std::char_traits<char>,std::allocator<char> > const &)
0x65306367	 [chrome.dll]	 -]	PrefNotifierImpl::FireObservers(std::basic_string<char,std::char_traits<char>,std::allocator<char> > const &)
0x65306265	 [chrome.dll]	 -]	PrefValueStore::NotifyPrefChanged(char const *,PrefValueStore::PrefStoreType)
0x6530620c	 [chrome.dll]	 -]	PrefValueStore::PrefStoreKeeper::OnPrefValueChanged(std::basic_string<char,std::char_traits<char>,std::allocator<char> > const &)
0x653fe522	 [chrome.dll]	 -]	SupervisedUserPrefStore::OnNewSettingsAvailable(base::DictionaryValue const *)
0x653cd660	 [chrome.dll]	 - bind_internal.h:909]	base::internal::InvokeHelper<1,void,base::internal::RunnableAdapter<void ( prerender::PrerenderLocalPredictor::*)(content::SessionStorageNamespace::MergeResult)>,void (base::WeakPtr<prerender::PrerenderLocalPredictor> const &,content::SessionStorageNamespace::MergeResult const &)>::MakeItSo(base::internal::RunnableAdapter<void ( prerender::PrerenderLocalPredictor::*)(content::SessionStorageNamespace::MergeResult)>,base::WeakPtr<prerender::PrerenderLocalPredictor> const &,content::SessionStorageNamespace::MergeResult const &)
0x653fe179	 [chrome.dll]	 - bind_internal.h:1224]	base::internal::Invoker<1,base::internal::BindState<base::internal::RunnableAdapter<void ( net::URLRequestFileJob::*)(int)>,void (net::URLRequestFileJob *,int),void (base::WeakPtr<net::URLRequestFileJob>)>,void (net::URLRequestFileJob *,int)>::Run(base::internal::BindStateBase *,int const &)
0x653fdcd9	 [chrome.dll]	 -]	ManagedUserSettingsService::InformSubscribers()
0x6546e922	 [chrome.dll]	 -]	ManagedUserService::Init()
0x65432bb3	 [chrome.dll]	 -]	ProfileManager::DoFinalInitForServices(Profile *,bool)
0x65432a0c	 [chrome.dll]	 -]	ProfileManager::DoFinalInit(Profile *,bool)
0x654326bf	 [chrome.dll]	 -]	ProfileManager::AddProfile(Profile *)
0x653f3be9	 [chrome.dll]	 -]	ProfileManager::GetProfile(base::FilePath const &)
0x653f3919	 [chrome.dll]	 -]	`anonymous namespace'::CreateProfile(content::MainFunctionParams const &,base::FilePath const &,CommandLine const &)
0x653e8cf3	 [chrome.dll]	 -]	ChromeBrowserMainParts::PreMainMessageLoopRunImpl()
0x653e85e5	 [chrome.dll]	 -]	ChromeBrowserMainParts::PreMainMessageLoopRun()
0x653e857b	 [chrome.dll]	 -]	content::BrowserMainLoop::PreMainMessageLoopRun()
0x652ccfae	 [chrome.dll]	 -]	content::StartupTaskRunner::RunAllTasksNow()
0x652ccdba	 [chrome.dll]	 -]	content::BrowserMainLoop::CreateStartupTasks()
0x652be131	 [chrome.dll]	 -]	content::BrowserMainRunnerImpl::Initialize(content::MainFunctionParams const &)
0x652bdb98	 [chrome.dll]	 -]	content::BrowserMain(content::MainFunctionParams const &)
0x652bdad0	 [chrome.dll]	 -]	content::RunNamedProcessTypeMain(std::basic_string<char,std::char_traits<char>,std::allocator<char> > const &,content::MainFunctionParams const &,content::ContentMainDelegate *)
0x652bda57	 [chrome.dll]	 -]	content::ContentMainRunnerImpl::Run()
0x652aabe1	 [chrome.dll]	 -]	content::ContentMain(HINSTANCE__ *,sandbox::SandboxInterfaceInfo *,content::ContentMainDelegate *)
0x652aa626	 [chrome.dll]	 -]	ChromeMain
0x0131877e	 [chrome.exe]	 -]	MainDllLoader::Launch(HINSTANCE__ *,sandbox::SandboxInterfaceInfo *)
0x0131895c	 [chrome.exe]	 -]	`anonymous namespace'::RunChrome(HINSTANCE__ *)
0x013189d9	 [chrome.exe]	 -]	wWinMain
0x01338c4a	 [chrome.exe]	 - crt0.c:275]	__tmainCRTStartup
0x765a3369	 [kernel32.dll]	 + 0x00013369]	BaseThreadInitThunk
0x778b9f71	 [ntdll.dll]	 + 0x00039f71]	__RtlUserThreadStart
0x778b9f44	 [ntdll.dll]	 + 0x00039f44]	

zea@, can you triage this ASAP?

Labels: Cr-Services-SupervisedUser
Looks related to managed users and sign in.

I'll add the Cr-Services-SupervisedUser label to get more eyes on the problem.

Comment 2 by, Nov 20 2013

Labels: -Cr-Services-Sync Cr-Services-SignIn
Bernhard, is it correct to be attempting to sign the user out during startup like this?

+Signin folks, who might have more input about the right way to do this, or know what's going on. It appears the token service may not be fully initialized yet?

Comment 3 by, Nov 21 2013

Hmm... This is pre- r234029, in which ProfileOAuth2TokenService was refactored (in particular, it doesn't observe NOTIFICATION_TOKENS_CLEARED anymore, so UpdateAuthError() shouldn't be called).

In general, I think it's okay to make sure that the user is signed out (after all, signing in is not allowed for supervised users), but it should be a no-op, as the supervised user can never be signed in in the first place.

Also, I thought the TokenService is already initialized at this time?

Comment 4 by, Jan 14 2014

About half a dozen people looked into this, and our conclusion is that the Preference files of affected users must be getting corrupted by an external source, so that they have both non-empty managed-user IDs and non-empty fields indicating that they were signed in at some point in the past.

Bernhard is preparing a patch to recover more gracefully from such corrupted profiles, since that corruption itself isn't harmful.
Project Member

Comment 5 by, Jan 16 2014

r245242 | | 2014-01-16T18:00:09.393369Z

Changed paths:

Gracefully handle being signed out of a supervised profile.

This is not something that would happen under normal circumstances, but it may happen if the preferences become corrupted or are modified.

BUG= 321370 

Review URL:

Comment 6 by, Jan 28 2014

Labels: ReleaseBlock-Stable Merge-Requested
Requesting merge back to 1700.
Labels: TE-Verified-M34 TE-Verified-34.0.1810.0
This particular crash(syncer::SyncMergeResult::model_type) is not seen on Windows - chrome version 34.0.1810.0.

Comment 8 by, Feb 5 2014

there will be no more 32s, please make sure to request merge to M33 instead

Comment 9 by, Feb 6 2014

Labels: -M-32 M-33
Labels: -Merge-Requested Merge-Approved
Labels: -Merge-Approved
Status: Fixed
I don't think there's anything to merge here for M33+, as this particular crash only happens on M32 due to later refactorings.
Project Member

Comment 12 by, Feb 10 2014

Labels: Merge-TBD
Is there a merge required here?
Labels: -ReleaseBlock-Stable
Hi, bugdroid! *waves*

Comment 14 by Deleted ...@, Feb 3 2015

Haha he said waves...XXD

Comment 15 by, Oct 21 2015

Labels: -Merge-TBD

Sign in to add a comment