Opened 7 months ago

Closed 5 months ago

#15244 closed bug (fixed)

KDL when attempting to burn CD

Reported by: vidrep Owned by: nobody
Priority: normal Milestone: Unscheduled
Component: - General Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

hrev53365 x86_64

Attempting to burn a CD caused a KDL.

Attached syslog and photo.

Attachments (4)

syslog.txt (2.2 KB ) - added by vidrep 7 months ago.
IMG_0102.JPG (1.1 MB ) - added by vidrep 7 months ago.
cdrecord-17714-debug-02-09-2019-00-54-50.report (6.1 KB ) - added by vidrep 6 months ago.
cdrecord-17715-debug-02-09-2019-00-53-00.report (5.9 KB ) - added by vidrep 6 months ago.

Download all attachments as: .zip

Change History (12)

by vidrep, 7 months ago

Attachment: syslog.txt added

by vidrep, 7 months ago

Attachment: IMG_0102.JPG added

comment:1 by waddlesplash, 7 months ago

Resolution: invalid
Status: newclosed

It appears cdrecord is actually using BeOS areas internally for operation. They now need to have the B_CLONEABLE_AREA flag applied; please open a bug at HaikuPorts.

comment:2 by korli, 6 months ago

Resolution: invalid
Status: closedreopened

IMO binaries built for r1b1 shouldn't be affected by the B_CLONEABLE_AREA flag policy change of a public API. It should be possible to fix in Haiku (if r1b1 binary => use always B_CLONEABLE_AREA ).

comment:3 by waddlesplash, 6 months ago

Writing and then applying some sort of heuristic to detect that would be pretty tricky, I don't know what you could look at to determine that. But even if there was, is it worth the time and effort to fix one application? If there were a lot more that might make sense, but so far no others have been reported.

comment:4 by korli, 6 months ago

One can check image.api_version < B_HAIKU_VERSION_1_PRE_BETA_2.

comment:5 by waddlesplash, 6 months ago

That would require fetching the image of the return address before every create_area, which would complicate the code significantly; or in the case where we used __gABIVersion, it would pick the oldest version, which would always be a pre-beta2 image, making the clone_area changes useless.

I still maintain that if cdrecord is the only affected application, we should just add some patch to it (or even just wait for the beta2 branch cut) and move on.

comment:6 by vidrep, 6 months ago

I tried to get a little more info.When attempting to burn a cd, cdrecord crashes. After that, it is impossible to eject the disc unless cdrecord -eject is invoked from the CLI. I attached debugger to cdrecord in this case. Debug reports attached.

comment:8 by waddlesplash, 5 months ago

Resolution: fixed
Status: reopenedclosed

Change merged in hrev53495.

Note: See TracTickets for help on using tickets.