asdlei00 / crashrpt

Automatically exported from code.google.com/p/crashrpt
0 stars 0 forks source link

Application is restarted twice after crash #115

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
***NOTE*** Please do create a NEW issue per each detected bug! Do not list
all detected problems in single issue record!

What steps will reproduce the problem?
1. Use the following flags :
      info.dwFlags |= CR_INST_ALL_EXCEPTION_HANDLERS;
      info.dwFlags |= CR_INST_HTTP_BINARY_ENCODING; 
      info.dwFlags |= CR_INST_APP_RESTART; 
      info.dwFlags |= CR_INST_SEND_QUEUED_REPORTS;
2. Let the application crash
3. The application is restarted twice

What is the expected output? What do you see instead?
Only restart once

What version of the product are you using? On what operating system?
1.2.10

Please provide any additional information below.

g_ErrorReportSender.DoWork(RESTART_APP);  is called twice, once in Finalize()
and once in CErrorReportDlg::OnSend()

It seems also to be called in CErrorReportDlg::OnPopupCloseTheProgram(), but I 
haved have debugged this case yet.

Original issue reported on code.google.com by weavelink on 3 Nov 2011 at 9:39

GoogleCodeExporter commented 9 years ago
The best solution is to only include RESTART_APP in the Finalize() call, not in 
the others. This way, RESTART_APP seems to be always called.

Original comment by weavelink on 3 Nov 2011 at 10:18

GoogleCodeExporter commented 9 years ago
I've reproduced the problem, thanks.

Original comment by zexspect...@gmail.com on 6 Nov 2011 at 4:34

GoogleCodeExporter commented 9 years ago
Fixed in rev. 1336 of trunk

Original comment by zexspect...@gmail.com on 25 Aug 2012 at 5:48