Opened 12 years ago
Last modified 8 years ago
#9434 assigned bug
[Tracker] wrong column display + crash in BBitmap::Lock()
Reported by: | ttcoder | Owned by: | nobody |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Applications/Tracker | Version: | R1/alpha4.1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | All |
Description
A tunetracker customer made a detailed report to us; seems using folders that were originally created/manipulated in Zeta sometimes makes Haiku's Tracker display the columns as 'blank' and even crash if re-arranging them.
Attachments (5)
Change History (14)
by , 12 years ago
Attachment: | tracker_crash added |
---|
comment:1 by , 12 years ago
Do you happen to be able to reproduce it? If so, can you zip up such a folder, and attach it to this ticket?
by , 12 years ago
Attachment: | tracker-crash_screenshotB.png added |
---|
the trigger event: adding a new column
by , 12 years ago
Attachment: | tracker-crash_screenshotC.png added |
---|
immediate result: columns become messed up, "infinite" in width
by , 12 years ago
Attachment: | tracker-crash_screenshotE.png added |
---|
a third symptom that occurs sometimes: column cells display "- -" when empty (instead of the normal "-")
comment:2 by , 12 years ago
@axel that's my thought as well: this should be reproducible if getting the "trk_/*" attributes; I'll ask him to empty the folder of its files, and zip it up (to preserve the layout attributes) and attach it here. Hang on.
by , 12 years ago
Attachment: | Gladys Kinight And The Pips.zip added |
---|
Triggers the "column width" problem with the right FileTypes config
comment:3 by , 12 years ago
Seems #9444 which I just filed, might be the root cause of it all.
- I unzipped the above in a "factory settings" haiku install, which had just Artist/Title/Album ..etc attributes configured, and could not see problems when adding any of these.
- I then tried in my dev. partition, which is setup with out "tunetracker:volume_boost", "tunetracker:off_ramp" ..etc attributes, and there I was able to see the "wide column" problem indeed, when adding VolumeBoost (didn't see a crash, but didn't dig for it very deep either).
So the economy -of-force approach here is probably to tackle #9444 first, and maybe it will fix the problem and then we can close this ticket as duplicate of that ticket :-)
comment:4 by , 12 years ago
That would be the lazy way, as Tracker should not crash on reading invalid settings. One should fix this bug first, and the go for the root cause instead :-)
comment:5 by , 12 years ago
This would probably be a quick fix once pinpointed, so I won't be a PITA arguing with you about how you should spend or not your coding time ;-) A couple thoughts on the "pinpointing" however..
- didn't reproduce the crash itself in my quick test in #9444, so either I didn't try hard enough, or it's not enough to have the column-width set to 4000 pixels and one needs something else, maybe the type_code set to #?*! or maybe something else
- even if not primarily aimed at closing this ticket, #9444 could however inspire a "test case", pinpoint which field Tracker is "crashy" on.. I see that siarzhuk has assigned himself to that other ticket.
comment:6 by , 12 years ago
For the record,
The fix in hrev45275 has this key part:
+ fStatus(B_NO_INIT), + fType('CSTR'), + fViewable(true), + fEditable(false), + fExtra(false), + fWidth(0), + fAlignment(0)
So anyone looking for a vulnerability / a way to crash Tracker might want to create an intentionally broken version of setmime that changes the above to fWidth(4000.0), fType(0xdeadbeef)
..etc and maybe that will do the trick, opening a folder / the resulting column will crash...
comment:7 by , 10 years ago
Should this be closed since it's fixed? Or left open because the bug in Tracker still exists, only buried?
comment:8 by , 10 years ago
No opinion either way from me, we're moving away from Tracker and to ArmyKnife instead for our editing needs, since we've added complex attributes (BMessage-in-BMessage ..etc) that won't fit in Tracker columns any more. I see the logic in axel's wish to leave this open though.
comment:9 by , 8 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
backtrace of segv crash