forprevious / zynamics

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

Bindiff standalone says "Couldn't calculate graph layout" #8

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Import existing, or create new diff (tested with win executable and ARM 
firmware) in Bindiff standalone workspace.
2. Open "Matched Functions" View.
3. Select "Show in Call Graph" from context menu for any line in table.

What is the expected output? What do you see instead?
Expected: A Call graph, generated without error message.
Instead: Error message "Couldn't calculate graph layout" two times, and a 
crippled graph.

What version of the product are you using? On what operating system?
Win 7 Prof. SP 1 x64
Java 7 update 5
Bindiff 4.0.2 (patched version with issue #6 fixed)

Original issue reported on code.google.com by L783x...@gmail.com on 22 Aug 2012 at 11:48

GoogleCodeExporter commented 9 years ago

Original comment by nilsheu...@google.com on 22 Aug 2012 at 12:11

GoogleCodeExporter commented 9 years ago

Original comment by nilsheu...@google.com on 22 Aug 2012 at 12:12

GoogleCodeExporter commented 9 years ago
Does this happen occasionally or is this reproducible?

Original comment by nilsheu...@google.com on 24 Aug 2012 at 11:41

GoogleCodeExporter commented 9 years ago
First a small window with progress bar opens, saying "Layouting graphs".
Short after, the first error window appears. This happens all the time.
After confirmation the graph view opens starting with a crippled display (all 
nodes in the left upper corner). From here sometimes the error appears a second 
time and graph stays this way, sometimes the graph unfolds in an animated 
manner.

Just tried something that might lead a way: With my ARM firmware diffs only one 
error appears, with a diff of a windows application 2 errors appear.

Original comment by L783x...@gmail.com on 24 Aug 2012 at 1:12

GoogleCodeExporter commented 9 years ago
Tested a bit more on Linux:
Java 6 with plain Bindiff 4.0.1 works
Java 7 with Bindiff 4.0.2 patch doesn't

Original comment by L783x...@gmail.com on 27 Aug 2012 at 6:36

GoogleCodeExporter commented 9 years ago
Thanks for testing. Today I finally found the time to debug this issue. Sorry 
for the delay. The bad news: It is a bug in our y files graph library. The good 
news: They have fixed it already. Will send you a new patch with the fixed 
library included in a few minutes.

Original comment by nilsheu...@google.com on 29 Aug 2012 at 1:40

GoogleCodeExporter commented 9 years ago
Fixed under Windows and Linux with Java 7

Original comment by L783x...@gmail.com on 29 Aug 2012 at 2:35

GoogleCodeExporter commented 9 years ago

Original comment by nilsheu...@google.com on 29 Aug 2012 at 2:41