[prev in list] [next in list] [prev in thread] [next in thread] 

List:       opensuse-commit
Subject:    commit llvm8 for openSUSE:Factory
From:       root <root () suse ! de>
Date:       2020-09-30 17:54:54
Message-ID: 20200930175454.099B29B883 () Hilbert ! suse ! de
[Download RAW message or body]

Hello community,

here is the log from the commit of package llvm8 for openSUSE:Factory checked in at \
2020-09-30 19:54:40 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 Comparing /work/SRC/openSUSE:Factory/llvm8 (Old)
 and      /work/SRC/openSUSE:Factory/.llvm8.new.4249 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "llvm8"

Wed Sep 30 19:54:40 2020 rev:14 rq:838613 version:8.0.1

Changes:
--------
--- /work/SRC/openSUSE:Factory/llvm8/llvm8.changes	2020-07-14 08:00:32.170031811 \
                +0200
+++ /work/SRC/openSUSE:Factory/.llvm8.new.4249/llvm8.changes	2020-09-30 \
19:54:50.888809209 +0200 @@ -1,0 +2,6 @@
+Mon Sep 28 21:10:54 UTC 2020 - Tom de Vries <tdevries@suse.com>
+
+- Fix dsymutil crash on ELF file. (bsc#1176964)
+  * llvm-fix-a-copy-and-paste-error-that-would-cause-a-crash.patch
+
+-------------------------------------------------------------------

New:
----
  llvm-fix-a-copy-and-paste-error-that-would-cause-a-crash.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ llvm8.spec ++++++
--- /var/tmp/diff_new_pack.0aN0UQ/_old	2020-09-30 19:54:53.800811813 +0200
+++ /var/tmp/diff_new_pack.0aN0UQ/_new	2020-09-30 19:54:53.804811816 +0200
@@ -106,6 +106,8 @@
 Patch29:        compiler-rt-sanitizer-ipc-perm.patch
 # PATCH-FIX-UPSTREAM fix-ppcle64-build.patch -- Fix ppcle64 build with newer gcc5
 Patch30:        fix-ppcle64-build.patch
+# PATCH-FIX-UPSTREAM llvm-fix-a-copy-and-paste-error-that-would-cause-a-crash.patch \
-- Fix dsymutil crash on ELF file. +Patch31:        \
llvm-fix-a-copy-and-paste-error-that-would-cause-a-crash.patch  BuildRequires:  \
binutils-devel >= 2.21.90  %if %{with gold}
 BuildRequires:  binutils-gold
@@ -503,6 +505,7 @@
 %patch24 -p1
 %patch27 -p2
 %patch28 -p2
+%patch31 -p2
 
 pushd cfe-%{_relver}.src
 %patch1 -p1

++++++ llvm-fix-a-copy-and-paste-error-that-would-cause-a-crash.patch ++++++
Fix a copy and paste error that would cause a crash.

Summary: The wrong variable was being checked for an error, which mean a llvm::Error \
went unchecked and crashes dsymutil. Discovered this when trying to feed an ELF file \
to "dsymutil --update" and running into the crash.

Reviewers: aprantl, JDevlieghere

Subscribers: llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D75777

---
 llvm/tools/dsymutil/MachODebugMapParser.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/llvm/tools/dsymutil/MachODebugMapParser.cpp \
b/llvm/tools/dsymutil/MachODebugMapParser.cpp index 83703fb13f0a..a9d6c876860c 100644
--- a/llvm/tools/dsymutil/MachODebugMapParser.cpp
+++ b/llvm/tools/dsymutil/MachODebugMapParser.cpp
@@ -383,7 +383,7 @@ ErrorOr<std::vector<std::unique_ptr<DebugMap>>> \
MachODebugMapParser::parse() {  
   auto Objects = ObjectEntry->getObjectsAs<MachOObjectFile>();
   if (!Objects) {
-    return errorToErrorCode(ObjectEntry.takeError());
+    return errorToErrorCode(Objects.takeError());
   }
 
   std::vector<std::unique_ptr<DebugMap>> Results;


[prev in list] [next in list] [prev in thread] [next in thread]