I am pretty knowledgable in C and dynamic linking. In theory, I would also be very happy to contribute to maintaining this package. However, time is an ongoing issue and I would need to learn to make packages for Debian.
I noticed how old was this patch. It doesn't feel very healthy to keep some non-bug-fixing paches for so long in debian. (But I saw on your blog you already have had this kind of discussion.) And the upstream project seems dead. The last commits dates back June or July 2002 and neither modules have much history. It looks like this project was written once and fogotten. Have you tried contacting the upstream author? It seems like the best course of action would be to adopte the upstream project, don't you think? This lack of upstream activity also means that the recent break was introduced by the last NMU by Andrej Shadura. I CC him too. Likely the patch 14-allocate-mixerdevice-only-once.patch. In fact, reading the code, I think the lines where the crash happen are just not needed anymore. Best regards, Celelibi Le Tue, Jan 19, 2021 at 09:35:03AM +0100, Joachim Breitner a écrit : > Control: tag -1 + help > > thanks for the report! This patch is 13 years old, oh wey. I might need > help from someone who nows C, X11 and dynamic linking better than I do. > > Also happy to pass on maintainership of this package to someone else, I > am not using it myself anymore. > > Cheers, > Joachim > > Am Dienstag, den 19.01.2021, 06:40 +0100 schrieb Celelibi: > > Package: osdsh > > Version: 0.7.0-10.4 > > Severity: grave > > Justification: renders package unusable > > > > Dear Maintainer, > > > > Nowadays, osdsh crashes without a message. > > The cause I identified is that it looks for a symbol "mixerdevice" in > > the shared library libosdshmixer.so which doesn't exist. > > Since the return of dlsym isn't checked, it dereferences a NULL pointer. > > > > In the function load_plugin in src/osdsh/controlsh.c: > > mod_mixerdev = dlsym(module, "mixerdevice"); > > *mod_mixerdev = mixerdevice; > > > > These lines seems to have been added by the debian patch > > 06-fix-m-option.patch. > > > > I guess this patch could be reworked to work with the new version. > > > > Best regards, > > Celelibi > > > > -- System Information: > > Debian Release: bullseye/sid > > APT prefers testing > > APT policy: (990, 'testing'), (500, 'testing-debug'), (500, 'unstable') > > Architecture: amd64 (x86_64) > > > > Kernel: Linux 5.10.0-1-amd64 (SMP w/2 CPU threads) > > Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8), LANGUAGE > > not set > > Shell: /bin/sh linked to /bin/dash > > Init: systemd (via /run/systemd/system) > > > > Versions of packages osdsh depends on: > > ii libc6 2.31-9 > > ii libxosd2 2.2.14-2.1+b1 > > ii tk 8.6.10 > > > > osdsh recommends no packages. > > > > osdsh suggests no packages. > > > > -- no debconf information > > > -- > Joachim “nomeata” Breitner • nome...@debian.org • https://j.oach.im/ >
