Can you provide us the server crash id from chrome://crashes?

I have the dump files from the H:\UserData\Chrome\Crashpad\reports folder.
Can I send one of the files?

Thanks,
Peter

Thank you for providing more feedback. Adding the requester to the cc list.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

We just need the server crash id. Do you have that? Or do you have chrome crash reporting disabled?

The problem is at this point I can't really start chrome unless I rename
the userdata folder to start a fresh profile, but once it crashes I cant
reopen chrome, the service runs but chrome does not open and the only way
to fix it is to rename the userdata folder so it can recreate a fresh
copy.  Is there any file I can upload from the current userdata folder that
would have that information?

Thanks,
Peter

Thank you for providing more feedback. Adding the requester to the cc list.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

+Stability-Sheriff-Desktop; hopefully they can help

Relevant stack trace:
"""
Thread 0 ( * CRASHED * EXCEPTION_ACCESS_VIOLATION_EXEC @ 0x444948 )
0	 [	 +	 0x444948]
1	 [chrome.dll	 -	 C:\b\c\b\win_clang\src\chrome_elf\nt_registry\nt_registry.cc:999] nt::SetRegValueDWORD(void *,wchar_t const *,unsigned long)
2	 [chrome.dll	 -	 C:\b\c\b\win_clang\src\chrome\install_static\install_util.cc:491] install_static::SetCollectStatsInSample(bool)
"""

Code in question:
"""
  std::wstring registry_path = GetRegistryPath();

  HANDLE key_handle = INVALID_HANDLE_VALUE;
  if (!nt::CreateRegKey(nt::HKCU, registry_path.c_str(),
                        KEY_SET_VALUE | KEY_WOW64_32KEY, &key_handle)) {
    return false;
  }

  bool success = nt::SetRegValueDWORD(key_handle, kRegValueChromeStatsSample,
                                      in_sample ? 1 : 0);
"""

The code in question looks correct. Given that this is a segfault in SetRegValueDWORD, I'm going to guess that there's some type of DLL/registry corruption happening. 

Unfortunately, I don't think there's much we can do here. Closing this out.

+grt -- any other insights?

Deleted: symbolized_windows_crash.txt 36.3 KB

symbolized_windows_crash.txt 36.3 KB View Download

This reminds me of  issue 794695 , which was a crash in App-V's shims following what seemed to be a valid way to call NtQueryValueKey. We worked around the problem in r527222 by changing our calling pattern. I wonder if something similar is going on here -- that we're making a seemingly valid call that some 3rd party code is choking on.

One thing I notice is that SetCollectStatsInSample is called from code in chrome.dll (long after processes initialization). I imagine this problem would go away if we moved SetCollectStatsInSample from chrome/install_static into chrome/installer/util so that it could use base::RegKey.

Chris: any chance that someone on your team could make this change? I'm happy to review it.

We really appreciate you guys looking into this and do let me know if you need any more info from my end. Our company really wants to keep using the Chrome browser and not switch to other.

Thanks,
Peter

+pmonette to try the fix from Comment #10, if you have time?

(FWIW, looking on crash this stack trace is very rare, occurring only a couple dozen times almost exclusively from two clients, all in one geographic location, and all running over Citrix. This seems like a very very isolated problem.)

Not sure if it helps but we tried reinstalling chrome and also installing Chrome Browser for enterprise with same results.  Users profile is stored on a network drive and problem cannot always be recreated.

Thanks,
Peter

[stability sheriff] Removing from queue as 1. this is not impacting a large number of users and 2. there is a fix proposed to try to resolve it.