Rating 0 (worst) for backtrace on which the parser fails.

1 files changed, 10 insertions, 4 deletions

diff --git a/lib/Plugins/CCpp.cpp b/lib/Plugins/CCpp.cpp
index b028c416..4006918c 100644
--- a/lib/Plugins/CCpp.cpp
+++ b/lib/Plugins/CCpp.cpp
@@ -669,16 +669,22 @@ void CAnalyzerCCpp::CreateReport(const char *pDebugDumpDir, int force)
     {
         VERB1 log(_("Backtrace parsing failed for %s"), pDebugDumpDir);
         VERB1 log("%d:%d: %s", location.line, location.column, location.message);
-        /* If the parser failed compute the UUID from the executable
-           and package only.  This is not supposed to happen often.
-           Do not store the rating, as we do not know how good the
-           backtrace is. */
+        /* If the parser failed, compute the UUID from the executable
+         * and package only.  This is not supposed to happen often.
+         */
         struct strbuf *emptybt = strbuf_new();
         strbuf_prepend_str(emptybt, executable.c_str());
         strbuf_prepend_str(emptybt, package.c_str());
         string hash_str = create_hash(emptybt->buf);
         dd.SaveText(FILENAME_GLOBAL_UUID, hash_str.c_str());
 
+        /* Other parts of ABRT assume that if no rating is available,
+         * it is ok to allow reporting of the bug. To be sure no bad
+         * backtrace is reported, rate the backtrace with the lowest
+         * rating.
+         */
+        dd.SaveText(FILENAME_RATING, "0");
+
         strbuf_free(emptybt);
         dd.Close();
         return;