KirillOsenkov / MSBuildStructuredLog

A logger for MSBuild that records a structured representation of executed targets, tasks, property and item values.
MIT License
1.47k stars 198 forks source link

Crash when loading large binlog #831

Closed mthalman closed 3 weeks ago

mthalman commented 3 weeks ago

This large binlog (~ 62 MB) will cause the viewer to crash when opening: binlog link.

This is a regression that was introduced in version 2.2.342. It loads correctly in version 2.2.336. It still fails to load with the latest version (2.2.379) as well.

KirillOsenkov commented 3 weeks ago

Thanks for reporting!

The build had some circular project references. I made it so it reports them instead of stack overflowing:

image

KirillOsenkov commented 3 weeks ago

What's weird that neither WinDbg nor VS would show the call stack of the stack overflow, and I don't know why. Your hint was helpful, so I diffed all the changes between 336 and 342 and spotted a recursive algorithm I added.

KirillOsenkov commented 3 weeks ago

Published