Bug 195248

Summary:	Completion of data initialisation for WorkMan
Product:	kdemultimedia	Reporter:	Markus Elfring <Markus.Elfring>
Component:	general	Assignee:	Multimedia Developers <kde-multimedia>
Status:	REPORTED ---
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
Latest Commit:		Version Fixed In:

Description Markus Elfring 2009-06-04 20:20:18 UTC

Version:            (using KDE 4.1.3)
OS:                Linux
Installed from:    SuSE RPMs

The global variables "blks_mutex" and "wakeup_audio" are not properly initialised.
http://websvn.kde.org/trunk/KDE/kdemultimedia/libkcompactdisc/wmlib/cdda.c?view=markup&revision=826360

The functions "pthread_mutex_init" and "pthread_cond_init" should be called for this purpose to complete the initialisation of data structures for the used mutual exclusion.

Comment 1 Andrew Crouthamel 2018-11-02 23:01:20 UTC

Dear Bug Submitter,

This bug has been stagnant for a long time. Could you help us out and re-test if the bug is valid in the latest version? I am setting the status to NEEDSINFO pending your response, please change the Status back to REPORTED when you respond.

Thank you for helping us make KDE software even better for everyone!

Comment 2 Markus Elfring 2018-11-03 10:08:38 UTC

(In reply to Andrew Crouthamel from comment #1)

Do you care for the safe initialisation of data structures by calls of the Pthread API for the used mutual exclusion in this software library?
https://github.com/KDE/libkcompactdisc/blob/60049bba0704c62aa4b22437a52d4aa3075f8aa3/src/wmlib/cdda.c#L52

Comment 3 Justin Zobel 2022-12-02 01:22:48 UTC

Thank you for reporting this issue in KDE software. As it has been a while since this issue was reported, can we please ask you to see if you can reproduce the issue with a recent software version?

If you can reproduce the issue, please change the status to "REPORTED" when replying. Thank you!

Comment 4 Markus Elfring 2022-12-02 17:26:58 UTC

(In reply to Justin Zobel from comment #3)
Did you notice that the mentioned implementation details were not reconsidered so far?
https://github.com/KDE/libkcompactdisc/blob/1771e180811d8a3f87f3e1a0003ab0360564c021/src/wmlib/cdda.c#L52