Opened 16 years ago
Closed 16 years ago
#4422 closed patch (fixed)
when linking errors occur while loading plugins, the errors are not displayed correctly
Reported by: | Owned by: | Anduin Withers | |
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | mythtv | Version: | unknown |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
dlerror() is called twice if loading a libmythplugin.so fails - the first call clears the error so the actual error displayed doesnt contain the reason.
Patch attached.
Attachments (4)
Change History (13)
comment:1 Changed 16 years ago by
comment:2 Changed 16 years ago by
Indeed - trac created the ticket before I attached the file - I thought it would only create the ticket once I had attached it...
comment:3 Changed 16 years ago by
Owner: | changed from Isaac Richards to Anduin Withers |
---|---|
Status: | new → accepted |
comment:4 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
comment:5 Changed 16 years ago by
Resolution: | fixed |
---|---|
Status: | closed → new |
methinks this broke the build (well it does on windows) here is the error message:
mythplugin.cpp: In member function `int MythPlugin::init(const char*)':
mythplugin.cpp:39: error: call of overloaded `QString(DWORD)' is ambiguous
C:/MSys/1.0/qt-3.3.x-p8/include/qstring.h:409: note: candidates are:
QString::QString(const std::string&) <near match>
C:/MSys/1.0/qt-3.3.x-p8/include/qstring.h:406: note:
QString::QString(const char*) <near match>
C:/MSys/1.0/qt-3.3.x-p8/include/qstring.h:403: note:
QString::QString(const QByteArray&)
C:/MSys/1.0/qt-3.3.x-p8/include/qstring.h:402: note:
QString::QString(const QString&) <near match>
C:/MSys/1.0/qt-3.3.x-p8/include/qstring.h:401: note:
QString::QString(QChar)
comment:6 Changed 16 years ago by
dlerr.win.patch tested with both SVN 15336 and 15340(HEAD):
./mpegts.o: In function `dlerror': c:/mythtv/mythtv/libs/libavformat/../libmyth/compat.h:205: undefined reference to `sprintf_is_forbidden_due_to_security_issues_use_snprintf' collect2: ld returned 1 exit status
comment:7 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Verified by Jay Modi. Applied in [15356]
Changed 16 years ago by
Attachment: | dlerr.win.3.patch added |
---|
New patch to fix multiple definition errors
comment:8 Changed 16 years ago by
Resolution: | fixed |
---|---|
Status: | closed → new |
Attached patch to fix a multiple definition error of dlerror
I think you forgot to attach the patch file...
-- Joe Ripley vitaminjoe@…