bad hostnames on remote crashes

Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>

1 files changed, 11 insertions, 7 deletions

diff --git a/src/Daemon/MiddleWare.cpp b/src/Daemon/MiddleWare.cpp
index cfec3c47..57b29677 100644
--- a/src/Daemon/MiddleWare.cpp
+++ b/src/Daemon/MiddleWare.cpp
@@ -794,14 +794,17 @@ static mw_result_t SavePackageDescriptionToDebugDump(
 
     std::string description = GetDescription(packageName.c_str());
 
-    // HOST_NAME_MAX is defined in limits.h
     char host[HOST_NAME_MAX + 1];
-    int ret = gethostname(host, HOST_NAME_MAX);
-    host[HOST_NAME_MAX] = '\0';
-    if (ret < 0)
+    if (!remote)
     {
-        perror_msg("gethostname");
-        host[0] = '\0';
+        // HOST_NAME_MAX is defined in limits.h
+        int ret = gethostname(host, HOST_NAME_MAX);
+        host[HOST_NAME_MAX] = '\0';
+        if (ret < 0)
+        {
+            perror_msg("gethostname");
+            host[0] = '\0';
+        }
     }
 
     try
@@ -811,7 +814,8 @@ static mw_result_t SavePackageDescriptionToDebugDump(
         dd.SaveText(FILENAME_PACKAGE, package.c_str());
         dd.SaveText(FILENAME_DESCRIPTION, description.c_str());
         dd.SaveText(FILENAME_COMPONENT, component.c_str());
-        dd.SaveText(FILENAME_HOSTNAME, host);
+        if (!remote)
+            dd.SaveText(FILENAME_HOSTNAME, host);
     }
     catch (CABRTException& e)
     {