commit 97ddb8d846056fd38fd8d951c0c2fa164ff2b865
author Asmitha Karunanithi <asmitk01@in.ibm.com> Thu May 05 01:00:18 2022 -0500
committer Asmitha Karunanithi <asmitk01@in.ibm.com> Thu Oct 27 10:51:32 2022 +0000
tree 12ad28288c5fb057dc0044fdc78c87213484a8e2
parent 003b8b7958e4324d4eea61860fd1bc7392d063ee

Move dbus method call into try-catch block

This change is to avoid networkd core dumping in systems
where resolved service is not active. Moving the dbus
call into try block will throw the exception instead of
networkd core dump.

Signed-off-by: Asmitha Karunanithi <asmitk01@in.ibm.com>
Change-Id: I547fcb910628fa20312099caa97fa0f76b6d140f

src/ethernet_interface.cpp

diff --git a/src/ethernet_interface.cpp b/src/ethernet_interface.cpp
index 4c31383..ff5a21b 100644
--- a/src/ethernet_interface.cpp
+++ b/src/ethernet_interface.cpp
@@ -837,10 +837,10 @@
                                       PROPERTY_INTERFACE, METHOD_GET);
 
     method.append(RESOLVED_INTERFACE, "DNS");
-    auto reply = bus.call(method);
 
     try
     {
+        auto reply = bus.call(method);
         reply.read(name);
     }
     catch (const sdbusplus::exception_t& e)