Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code Explorer crashes Excel while examining a property of a class. #2638

Closed
PeterMTaylor opened this issue Feb 5, 2017 · 4 comments
Closed
Labels
bug Identifies work items for known bugs critical Marks a bug as a must-fix, showstopper issue feature-code-explorer
Milestone

Comments

@PeterMTaylor
Copy link

Using Production RD 2.0.12 this time, not my development build fork in the background.
After removing some old classes of old code which works well then began focusing on one of the property when (all of a sudden!!) the entire Excel.exe thread crashed but VS2015 was able to capture the debug details as follows.

Here is the dialog box came up which I captured firstly.
capture rd prod 2 0 11 failed inspecting class

Here is TRACE in my log before I started - lucky!
RubberduckLog - RD 2.0.12 crash.txt

Now to set the scene what VS2015 is telling me where this problem is...
capture rd prod 2 0 12 vs code
Had a chuckle knowing we are trying to uncover why it sometimes fails but for VS2015 to tell me is saying something...
capture rd prod 2 0 12 vs todolist
The call stack as follows
capture rd prod 2 0 12 vs callstack
The disassembly.
capture rd prod 2 0 12 vs disassembly

@MDoerner and @comintern is this the QMN issue?

@PeterMTaylor
Copy link
Author

@comintern Here is the excel file remembering you wanted a copy I used with the source code described in the logs. Have fun hacking it!.

emptycdataset.zip

@MDoerner
Copy link
Contributor

MDoerner commented Feb 5, 2017

This seems to be an issue with the concurrent reparses triggered by removing multiple components.
More specifically, the resolver does not handle cancellation properly.

@Vogel612
Copy link
Member

Did this show up again since #2639 was merged?

@Vogel612 Vogel612 added bug Identifies work items for known bugs critical Marks a bug as a must-fix, showstopper issue feature-code-explorer labels Oct 13, 2017
@Vogel612 Vogel612 added this to the 2.1.x Cycle milestone Nov 3, 2017
@Vogel612 Vogel612 added this to TODO in Release 2.3.0 Apr 9, 2018
@bclothier
Copy link
Contributor

Using the same file that was provided above, and deleting few random classes, I no longer can reproduce the crash when trying to view the same property mentioned. Unless someone can provide show otherwise, this will be closed.

Semi-automatic bug tracker automation moved this from ToDo to Done Apr 22, 2018
Release 2.3.0 automation moved this from TODO to Done Apr 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Identifies work items for known bugs critical Marks a bug as a must-fix, showstopper issue feature-code-explorer
Projects
Development

No branches or pull requests

4 participants