Closed Bug 690841 Opened 13 years ago Closed 5 years ago

Use CrashSubmit.pruneSavedDumps after plugin crashes to limit number of pending minidumps

Categories

(Toolkit :: Crash Reporting, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jimott, Unassigned)

References

Details

User Agent: Mozilla/5.0 (Windows NT 5.1; rv:7.0.1) Gecko/20100101 Firefox/7.0.1
Build ID: 20110928134238

Steps to reproduce:

My pending crash reports folder grows about 1 GB per month.  I average about 4 new entries per minute.  Sites like Amazon and CNN cause these errors.  Google doesn't seem to.  These accumulate even if I am away from the PC as long as the sites are open in tabs.


Actual results:

No observed effect on Firefox operation.

Report URL: https://crash-stats.mozilla.com/report/index/623c620d-0c1b-467d-94d5-612c02110930
From bp-623c620d-0c1b-467d-94d5-612c02110930
Process Type: plugin Version: Filename: npmozax.dll

Does the crashing stop if you update that plugin to the latest version?
or if that doesn't help - disable this plugin?  (don't forget to restart Firefox)
Problem solved!  Disabling npmozax.dll (Mozilla activex control and plugin support) fixed my problem. I also found I could force the errors to stop by disabling the plugin container in about:config.  Is there a newer version of npmozax? I couldn't find one.  My version is 1.0.0.4.  Thank you for the help.
I think it's a plugin to host ActiveX controls:
http://www.adamlock.com/mozilla/plugin.htm
It looks unmaintained to me.  I would recommend that you uninstall it.
Would be nice to have a limit for the disk space used for crash reports though.
Component: General → Plug-ins
OS: Windows XP → All
Product: Firefox → Core
QA Contact: general → plugins
Hardware: x86 → All
Version: 7 Branch → unspecified
I have uninstalled the plugin npmozax.dll and everything is working.  I agree that it would be a good idea to limit the pending crash reports folder size. I first discovered the problem in August using treesize.  At that time my pending file was over 3GB and contained almost 1 million files.  Thanks again for your help.
Thank you for reporting the problem.
I found bug 471054 which looks like it should have prevented this problem,
maybe it's not working in some cases?
Component: Plug-ins → Breakpad Integration
Product: Core → Toolkit
QA Contact: plugins → breakpad.integration
FYI the date that the plugin (npmozax.dll) was installed in my PC was 12/05/2005.  I have been upgrading my original Firefox since then.
Mats: that bug is very similar. The code there runs in the crash reporter client, but this is just a plugin crash, so we never get to run it. I think there are probably two issues here, so we should spin one off into its own bug:
1) We don't clean up the pending crash reports folder if the browser never crashes. If you only have plugin crashes, they can build up indefinitely.
2) If you have a plugin that crashes incessantly, like npmozax.dll in this case, we should disable it instead of allowing it to run and crash continuously.
Summary: Huge pending crash reports folder. → Pending crash reports from plugin crashes don't get cleaned up if the browser never crashes
This bug will be about applying the cleanup logic to plugin-specific crashes. Ted will file a separate one about the incessantly-crashing-plugin issue, since that will be easier to do after click-to-play plugin support lands.
Status: UNCONFIRMED → NEW
Ever confirmed: true
From my dupe:
(In reply to Ted Mielczarek [:ted.mielczarek] from comment #0)
> bug 814086 added a CrashSubmit.pruneSavedDumps API that matches what the
> crash reporter client does for browser crashes. We don't currently have this
> behavior for plugin crashes, which means that they can pile up on disk. We
> should use this API after a plugin crash to limit the number of pending
> crashes.
Summary: Pending crash reports from plugin crashes don't get cleaned up if the browser never crashes → Use CrashSubmit.pruneSavedDumps after plugin crashes to limit number of pending minidumps

Old minidumps are already being pruned if they're not submitted, marking this as fixed.

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