From b2c59af84de8d35c1eee38878053206a62756968 Mon Sep 17 00:00:00 2001
From: mj-xmr <mjxmr@protonmail.com>
Date: Tue, 2 Mar 2021 09:36:04 +0100
Subject: [PATCH] EasyLogging++: new anti-UB test and propagating exception

---
 external/easylogging++/easylogging++.h | 1 +
 tests/unit_tests/logging.cpp           | 7 +++++++
 2 files changed, 8 insertions(+)

diff --git a/external/easylogging++/easylogging++.h b/external/easylogging++/easylogging++.h
index c4a88339f..b983a796c 100644
--- a/external/easylogging++/easylogging++.h
+++ b/external/easylogging++/easylogging++.h
@@ -2026,6 +2026,7 @@ class TypedConfigurations : public base::threading::ThreadSafe {
         ELPP_INTERNAL_ERROR("Unable to get configuration [" << confName << "] for level ["
                             << LevelHelper::convertToString(level) << "]"
                             << std::endl << "Please ensure you have properly configured logger.", false);
+        throw; // The exception has to be rethrown, to abort a branch leading to UB.
       }
     }
     return it->second;
diff --git a/tests/unit_tests/logging.cpp b/tests/unit_tests/logging.cpp
index f11b17412..b3ffb9aa6 100644
--- a/tests/unit_tests/logging.cpp
+++ b/tests/unit_tests/logging.cpp
@@ -208,3 +208,10 @@ TEST(logging, operator_equals_segfault)
     el::Logger log2("id2", nullptr);
     log2 = log1;
 }
+
+TEST(logging, empty_configurations_throws)
+{
+    el::Logger log1("id1", nullptr);
+    const el::Configurations cfg;
+    EXPECT_ANY_THROW(log1.configure(cfg));
+}