Fehlermeldung, falls Plugin nicht geladen werden kann, ist nichtssagend z.B. "\240ŇUUU"

MarkusE · 12. Dezember 2020

Hi,

In vdr 2.4.5., plugin.c, Zeile 234, ist error noch "/usr/lib/vdr/plugins/libvdr-wirbelscan.so.2.4.5: undefined symbol: VDRPluginCreator".

Dann wird in Zeile 234:

Code

destroy = (destroy_t *)dlsym(handle, "VDRPluginDestroyer");

mit einem ungültigen handle gerufen.

In Zeile 284

Code

esyslog("ERROR: %s", error);

Steht in error dann "\240ŇUUU"

~ Markus

kls · 12. Dezember 2020

Kannst du bitte mal diesen Patch probieren?

wirbel · 13. Dezember 2020

Vielleicht anstelle

Code

if (!error) {

Code

if (create) {

MarkusE · 13. Dezember 2020

Hi Klaus,

Ich habe den Patch getestet, ich bekomme jetzt den korrekten Fehler:

Code

Dec 13 10:51:56 vdr1 vdr: [30655] loading plugin: /usr/lib/vdr/plugins/libvdr-softhddevice.so.2.4.5
Dec 13 10:51:56 vdr1 vdr: [30655] loading plugin: /usr/lib/vdr/plugins/libvdr-wirbelscan.so.2.4.5
Dec 13 10:51:56 vdr1 vdr: [30655] ERROR: /usr/lib/vdr/plugins/libvdr-wirbelscan.so.2.4.5: undefined symbol: VDRPluginCreator
Dec 13 10:51:56 vdr1 vdr[30655]: vdr: /usr/lib/vdr/plugins/libvdr-wirbelscan.so.2.4.5: undefined symbol: VDRPluginCreator
Dec 13 10:51:56 vdr1 vdr: [30655] deleting plugin: softhddevice
Dec 13 10:51:56 vdr1 vdr: [30655] exiting, exit code 2

~Markus

kls · 13. Dezember 2020

Zitat von wirbel

Vielleicht anstelle...

Ich glaube, dlerror() ist besser:

man dlsym:

Since the value of the symbol could actually be NULL (so that a NULL return from dlsym() need not indicate an error), the correct
way to test for an error is to call dlerror() to clear any old error conditions, then call dlsym(), and then call dlerror() again, saving
its return value into a variable, and check whether this saved value is not NULL.

Aber ich werde zusätzlich noch 'create' abfragen, zur Sicherheit.

wirbel · 13. Dezember 2020

Das symbol VDRPluginCreator() macht nur als Funktions Pointer Sinn, auch wenn der Rückgabewert NULL für dlsym() im Allgemeinen ein gültiges Symbol ist., wenn das in der lib so definiert wurde. Was VDR braucht, ist ein Funktionspointer ungleich NULL.

kls · 13. Dezember 2020

Code

if (!error && create) {

Damit sollte es wasserdicht sein.

Fehlermeldung, falls Plugin nicht geladen werden kann, ist nichtssagend z.B. "\240ŇUUU"

Jetzt mitmachen!

Teilen