Hello community,

here is the log from the commit of package llvm10 for openSUSE:Factory checked 
in at 2020-10-03 18:53:13
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/llvm10 (Old)
 and      /work/SRC/openSUSE:Factory/.llvm10.new.4249 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "llvm10"

Sat Oct  3 18:53:13 2020 rev:6 rq:838295 version:10.0.1

Changes:
--------
--- /work/SRC/openSUSE:Factory/llvm10/llvm10.changes    2020-08-03 
14:12:18.084313494 +0200
+++ /work/SRC/openSUSE:Factory/.llvm10.new.4249/llvm10.changes  2020-10-03 
18:53:24.437337225 +0200
@@ -1,0 +2,6 @@
+Mon Sep 28 07:04:16 UTC 2020 - Tom de Vries <[email protected]>
+
+- 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:
------------------
++++++ llvm10.spec ++++++
--- /var/tmp/diff_new_pack.TJa0Yc/_old  2020-10-03 18:53:26.825340327 +0200
+++ /var/tmp/diff_new_pack.TJa0Yc/_new  2020-10-03 18:53:26.829340332 +0200
@@ -134,6 +134,8 @@
 Patch26:        lld-default-sha1.patch
 # PATCH-FIX-UPSTREAM fix-atomics-test.patch -- Fix Clang test for arches 
without native atomics.
 Patch27:        fix-atomics-test.patch
+# PATCH-FIX-UPSTREAM 
llvm-fix-a-copy-and-paste-error-that-would-cause-a-crash.patch -- Fix dsymutil 
crash on ELF file.
+Patch28:        llvm-fix-a-copy-and-paste-error-that-would-cause-a-crash.patch
 BuildRequires:  binutils-devel >= 2.21.90
 BuildRequires:  cmake
 BuildRequires:  fdupes
@@ -555,6 +557,7 @@
 %patch22 -p1
 %patch24 -p1
 %patch25 -p1
+%patch28 -p2
 
 pushd clang-%{_version}.src
 %patch2 -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;

Reply via email to