summaryrefslogtreecommitdiffstats
path: root/drkonqi
diff options
context:
space:
mode:
authorTimothy Pearson <kb9vqf@pearsoncomputing.net>2014-03-28 18:54:47 -0500
committerTimothy Pearson <kb9vqf@pearsoncomputing.net>2014-03-28 18:54:47 -0500
commit71be0ee5d441711ed7070bb69f4db18f775b1030 (patch)
treee036aef0ac313a1e974275ad3c81b4272719a3a4 /drkonqi
parent448a0d753fd4bfcf29235abe56afcfa74ff97ac2 (diff)
downloadtdebase-71be0ee5d441711ed7070bb69f4db18f775b1030.tar.gz
tdebase-71be0ee5d441711ed7070bb69f4db18f775b1030.zip
Extract and include any pertinent SCM metadata from the crashed application in its crash report
Diffstat (limited to 'drkonqi')
-rw-r--r--drkonqi/CMakeLists.txt1
-rw-r--r--drkonqi/backtrace.cpp82
-rw-r--r--drkonqi/debuggers/gdbrc3
-rw-r--r--drkonqi/krashconf.cpp1
-rw-r--r--drkonqi/krashconf.h2
5 files changed, 86 insertions, 3 deletions
diff --git a/drkonqi/CMakeLists.txt b/drkonqi/CMakeLists.txt
index 502ba5e9..47866c46 100644
--- a/drkonqi/CMakeLists.txt
+++ b/drkonqi/CMakeLists.txt
@@ -18,6 +18,7 @@ include_directories(
${CMAKE_BINARY_DIR}
${TDE_INCLUDE_DIR}
${TQT_INCLUDE_DIRS}
+ ${LIBR_INCLUDEDIR}
)
link_directories(
diff --git a/drkonqi/backtrace.cpp b/drkonqi/backtrace.cpp
index 61ae8130..95bc0e05 100644
--- a/drkonqi/backtrace.cpp
+++ b/drkonqi/backtrace.cpp
@@ -25,6 +25,8 @@
* THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*****************************************************************/
+#include "config.h"
+
#include <tqfile.h>
#include <tqregexp.h>
@@ -36,6 +38,16 @@
#include <tdetempfile.h>
#include <tdehardwaredevices.h>
+#ifdef HAVE_ELFICON
+ // Elven things
+ extern "C" {
+ #include <libr.h>
+ #include <libr-icons.h>
+ }
+ #include <tqfileinfo.h>
+ #include <tdeio/tdelficon.h>
+#endif // HAVE_ELFICON
+
#include "krashconf.h"
#include "backtrace.h"
#include "backtrace.moc"
@@ -199,8 +211,16 @@ bool BackTrace::usefulBacktrace()
TQString strBt = '\n' + m_strBt + '\n';
// how many " ?? " in the bt ?
int unknown = 0;
- if( !m_krashconf->invalidStackFrameRegExp().isEmpty())
- unknown = strBt.contains( TQRegExp( m_krashconf->invalidStackFrameRegExp()));
+ if( !m_krashconf->invalidStackFrameRegExp().isEmpty()) {
+ int isfPos = strBt.find( TQRegExp( m_krashconf->invalidStackFrameRegExp()));
+ int islPos = strBt.find( m_krashconf->infoSharedLibraryHeader());
+ if ((isfPos >=0) && (isfPos < islPos)) {
+ unknown = true;
+ }
+ else {
+ unknown = false;
+ }
+ }
// how many stack frames in the bt ?
int frames = 0;
if( !m_krashconf->frameRegExp().isEmpty())
@@ -278,6 +298,64 @@ void BackTrace::processBacktrace()
}
}
+#ifdef HAVE_ELFICON
+ m_strBt.append("\n==== (tdemetainfo) application version information ====\n");
+
+ // Extract embedded SCM metadata from the crashed application
+ TQString crashedExec = TDEStandardDirs::findExe(m_krashconf->execName());
+ if (crashedExec.startsWith("/")) {
+ libr_file *handle = NULL;
+ libr_access_t access = LIBR_READ;
+
+ if((handle = libr_open(const_cast<char*>(crashedExec.ascii()), access)) == NULL) {
+ kdWarning() << "failed to open file" << crashedExec << endl;
+ }
+ else {
+ TQString scmModule = elf_get_resource(handle, ".metadata_scmmodule");
+ TQString scmRevision = elf_get_resource(handle, ".metadata_scmrevision");
+ if (scmRevision != "") {
+ m_strBt.append(TQString("%1:\t%2:%3\n").arg(TQFileInfo(crashedExec).fileName()).arg(scmModule).arg(scmRevision));
+ }
+ }
+
+ libr_close(handle);
+ }
+
+ m_strBt.append("\n==== (tdemetainfo) library version information ====\n");
+
+ // Extract embedded SCM metadata from shared libraries
+ int islPos = m_strBt.find( m_krashconf->infoSharedLibraryHeader());
+ TQString infoSharedLibraryText = m_strBt.mid(islPos);
+ TQTextStream infoSharedLibraryTextStream(&infoSharedLibraryText, IO_ReadOnly);
+ infoSharedLibraryTextStream.readLine(); // Skip info header x1
+ infoSharedLibraryTextStream.readLine(); // Skip info header x2
+ TQString infoSharedLibraryLine = infoSharedLibraryTextStream.readLine();
+ while (infoSharedLibraryLine != TQString::null) {
+ TQStringList libraryInfoList = TQStringList::split(" ", infoSharedLibraryLine, false);
+ if (libraryInfoList.count() > 0) {
+ TQString libraryName = libraryInfoList[libraryInfoList.count()-1];
+ if (libraryName.startsWith("/")) {
+ libr_file *handle = NULL;
+ libr_access_t access = LIBR_READ;
+
+ if((handle = libr_open(const_cast<char*>(libraryName.ascii()), access)) == NULL) {
+ kdWarning() << "failed to open file" << libraryName << endl;
+ }
+ else {
+ TQString scmModule = elf_get_resource(handle, ".metadata_scmmodule");
+ TQString scmRevision = elf_get_resource(handle, ".metadata_scmrevision");
+ if (scmRevision != "") {
+ m_strBt.append(TQString("%1:\t%2:%3\n").arg(TQFileInfo(libraryName).fileName()).arg(scmModule).arg(scmRevision));
+ }
+ }
+
+ libr_close(handle);
+ }
+ }
+ infoSharedLibraryLine = infoSharedLibraryTextStream.readLine();
+ }
+#endif // HAVE_ELFICON
+
// Append potentially important hardware information
m_strBt.append("\n==== (tdehwlib) hardware information ====\n");
TDEHardwareDevices *hwdevices = TDEGlobal::hardwareDevices();
diff --git a/drkonqi/debuggers/gdbrc b/drkonqi/debuggers/gdbrc
index 1aa1f8c3..28e804ed 100644
--- a/drkonqi/debuggers/gdbrc
+++ b/drkonqi/debuggers/gdbrc
@@ -158,7 +158,7 @@ Comment[zu]=Umcoshi wamaphutha osekelwe umbhalo osuka kwi-GNU kwikhonsoli
Exec=konsole -e gdb -nw %execname %pid
ExecBatch=gdb -nw -n -batch -x %tempfile %execname %pid
TryExec=gdb
-BacktraceCommand=echo \\n\necho ==== (gdb) bt ====\\n\nbt\necho \\n\\n\necho ==== (gdb) bt full ====\\n\nbt full\necho\\n\\n\necho ==== (gdb) info thread ====\\n\ninfo thread\necho\\n\\n\necho ==== (gdb) thread apply all bt ====\\n\nthread apply all bt
+BacktraceCommand=echo \\n\necho ==== (gdb) bt ====\\n\nbt\necho \\n\\n\necho ==== (gdb) bt full ====\\n\nbt full\necho\\n\\n\necho ==== (gdb) info thread ====\\n\ninfo thread\necho\\n\\n\necho ==== (gdb) thread apply all bt ====\\n\nthread apply all bt\necho\\n\\n\necho ==== (gdb) info sharedlibrary ====\\n\ninfo sharedlibrary
RemoveFromBacktraceRegExp=\(no debugging symbols found\)\.\.\.\\n?
InvalidStackFrameRegExp=\\n#[0-9]+\\s+0x[0-9A-Fa-f]+ \w* \?\?
FrameRegExp=\\n#[0-9]+\\s+0x[0-9A-Fa-f]+
@@ -166,3 +166,4 @@ NeededInValidBacktraceRegExp=\\n#5
TDECrashRegExp=\\n#0[ ]*0x[0123456789abcdefABCDEF]+.*<signal handler called>[ ]*\\n
TDECrashRegExpSingle=\\n0x[0123456789abcdefABCDEF][^\\n]*\\n
ThreadRegExp=\\nThread [0123456789]
+InfoSharedLibraryHeader= ==== (gdb) info sharedlibrary ==== \ No newline at end of file
diff --git a/drkonqi/krashconf.cpp b/drkonqi/krashconf.cpp
index 84bbfee8..12c8064e 100644
--- a/drkonqi/krashconf.cpp
+++ b/drkonqi/krashconf.cpp
@@ -113,6 +113,7 @@ void KrashConfig :: readConfig()
m_kcrashRegExp = debuggers.readEntry("TDECrashRegExp");
m_kcrashRegExpSingle = debuggers.readEntry("TDECrashRegExpSingle");
m_threadRegExp = debuggers.readEntry("ThreadRegExp");
+ m_infoSharedLibraryHeader = debuggers.readEntry("InfoSharedLibraryHeader");
TDEConfig preset(TQString::fromLatin1("presets/%1rc").arg(configname),
true, false, "appdata");
diff --git a/drkonqi/krashconf.h b/drkonqi/krashconf.h
index 074cfce6..d3d27e16 100644
--- a/drkonqi/krashconf.h
+++ b/drkonqi/krashconf.h
@@ -68,6 +68,7 @@ public:
TQString kcrashRegExp() const { return m_kcrashRegExp; }
TQString kcrashRegExpSingle() const { return m_kcrashRegExpSingle; }
TQString threadRegExp() const { return m_threadRegExp; }
+ TQString infoSharedLibraryHeader() const { return m_infoSharedLibraryHeader; }
bool showBacktrace() const { return m_showbacktrace; };
bool showDebugger() const { return m_showdebugger && !m_debugger.isNull(); };
bool showBugReport() const { return m_showbugreport; };
@@ -112,6 +113,7 @@ private:
TQString m_kcrashRegExp;
TQString m_kcrashRegExpSingle;
TQString m_threadRegExp;
+ TQString m_infoSharedLibraryHeader;
};
#endif