Closed Bug 1705731 Opened 3 years ago Closed 2 years ago

Crash in [@ nsFormFillController::HandleEvent]

Categories

(Toolkit :: Form Autofill, defect, P3)

Unspecified
Windows 7
defect

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: sefeng, Unassigned)

Details

(Keywords: crash)

Crash Data

Crash report: https://crash-stats.mozilla.org/report/index/d0437a9d-3144-4b99-921a-99e790210412

Reason: EXCEPTION_BREAKPOINT

Top 10 frames of crashing thread:

0 xul.dll nsFormFillController::HandleEvent toolkit/components/satchel/nsFormFillController.cpp:858
1 xul.dll trunc 
2 nss3.dll memmove 
3 nss3.dll _PR_NativeRunThread nsprpub/pr/src/threads/combined/pruthr.c:395
4 nss3.dll PR_MD_INIT_THREAD nsprpub/pr/src/md/windows/w95thred.c:110
5 ucrtbase.dll thread_start<unsigned int > 
6 kernel32.dll BaseThreadInitThunk 
7 ntdll.dll RtlUserThreadStart 
8 kernel32.dll BasepReportFault 
9 kernel32.dll BasepReportFault 

Not sure what is happening here, the call stack looks weird.

Component: DOM: Forms → Form Manager
Product: Core → Toolkit

I'm going to move this to the Form Autofill component where we are more actively triaging.
I don't know this history of this component, but at least at this point it seems a bit redundant.

Component: Form Manager → Form Autofill

This looks like a crash in the form autofill code on a new install, when the user focused an input field.
It needs some more investigation to know if this is a race condition we need to solve.

I'm seeing about 20 instances over the last 6 months, so I don't think this is very severe.

Severity: -- → S4
Priority: -- → P3

(In reply to Sam Foster [:sfoster] (he/him) from comment #1)

I'm going to move this to the Form Autofill component where we are more actively triaging.
I don't know this history of this component, but at least at this point it seems a bit redundant.

It's not redundant, it handles the toolkit/components/satchel code to provide the form fill controller and the form history results and storage. That code is a dependency for login, address and credit card autocomplete so IMO it should stay separate from pwmgr and autofill components.

Closing because no crashes reported for 12 weeks.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.