sane/ledm.c: debug header is missing, so /usr/lib/sane/libsane-hpaio.so.1.0.0 U DBG x86_64: NEW bad_elf_symbols detected: hplip-sane-3.10.9-alt1.x86_64.rpm /usr/lib64/sane/libsane-hpaio.so.1.0.0 is referencing DBG as external symbol

Bug #676660 reported by Igor Vlasenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
HPLIP
Fix Released
Undecided
Unassigned

Bug Description

scan/sane/ledm.c does not include "sanei_debug.h"

as a result, compiled /usr/lib/sane/libsane-hpaio.so.1.0.0 references DBG
as an external symbol
 i586: NEW bad_elf_symbols detected:
hplip-sane-3.10.9-alt1.i586.rpm /usr/lib/sane/libsane-hpaio.so.1.0.0 U DBG
 x86_64: NEW bad_elf_symbols detected:
hplip-sane-3.10.9-alt1.x86_64.rpm /usr/lib64/sane/libsane-hpaio.so.1.0.0 U DBG

It might crash the app if ledm protocol is used.
Proposed patch:

diff --git a/hplip/scan/sane/ledm.c b/hplip/scan/sane/ledm.c
index 4c053dd..fcea4fe 100644
--- a/hplip/scan/sane/ledm.c
+++ b/hplip/scan/sane/ledm.c
@@ -42,6 +42,7 @@
 # include "sane.h"
 # include "saneopts.h"
 # include "io.h"
+# include "sanei_debug.h"

 static struct ledm_session *session = NULL;

Revision history for this message
Igor Vlasenko (viy-altlinux) wrote :
Revision history for this message
Igor Vlasenko (viy-altlinux) wrote :

sorry, wrong patch added.

added the correct patch, with

+#define DEBUG_DECLARE_ONLY
+#include "sanei_debug.h"

Revision history for this message
Naga Samrat Chowdary, Narla (samrat-hplip) wrote :

Thanks for provide the patch. similar patch has been applied to tip.

Thanks,
Naga Samrat Chowdary, Narla

Changed in hplip:
status: New → Confirmed
status: Confirmed → Fix Committed
Changed in hplip:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.