Closed Bug 742735 Opened 8 years ago Closed 8 years ago

No Flash version detected for npswf32_11_2_202_228.dll


(Socorro :: General, task, critical)

Not set


(Not tracked)



(Reporter: kairo, Assigned: peterbe)


(Whiteboard: [qa-])


(1 file)

We are missing the Flash version for npswf32_11_2_202_228.dll in our detection, that should be fixed.

bp-e8cf70c4-bfd4-4735-a95e-14f272120405 for example shows that DLL, and in "Modules" I see it has a version, but we don't put a Flash version on the report, I guess we might not look for this file name. and others are not showing any Flash version because of that.

Because of this bug, all my Flash reports are also missing out on crashes from this new version.
Given how much of our analysis looks at Flash versions, this is critical for CrashKill.
Severity: normal → critical
I created a patch that I hope fixes this:

The only problem I see is that I'd prefer this to be in production last week :(
Lars asked for some module info for such versions, here they are, taken from a search for crashes containing "npswf32_11_" (as we don't resolve symbols for a couple of those yet anyhow). :)
Assignee: nobody → kairo
Peter, this change needs review and a test.  Please write the test and then land the changes ASAP.
Assignee: kairo → peterbe
I wrote the tests and corrected the code based on how I think/guessed it should be interpreted.
Per Laura, this is qa-
Whiteboard: [qa-]
Commits pushed to master at
Fixes bug 742735 - Detect new Windows Flash versions that have the version in their file name
Merge pull request #484 from KaiRo-at/master

Bug 742735 - detect new Windows Flash versions correctly
Closed: 8 years ago
Resolution: --- → FIXED
Commit pushed to stage at
Fixes bug 742735 - Detect new Windows Flash versions that have the version in their file name
Target Milestone: --- → 5
You need to log in before you can comment on or make changes to this bug.