Repository: incubator-hawq
Updated Branches:
  refs/heads/master 8928e88ea -> 4b3fdb262


HAWQ-1051. Failing in reverse DNS lookup causes resource manager core dump

Signed-off-by: jiny2 <y...@pivotal.io>


Project: http://git-wip-us.apache.org/repos/asf/incubator-hawq/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-hawq/commit/4b3fdb26
Tree: http://git-wip-us.apache.org/repos/asf/incubator-hawq/tree/4b3fdb26
Diff: http://git-wip-us.apache.org/repos/asf/incubator-hawq/diff/4b3fdb26

Branch: refs/heads/master
Commit: 4b3fdb262832f056cb39e725331938721adb39d7
Parents: 8928e88
Author: stanlyxiang <stanly.sxi...@gmail.com>
Authored: Wed Sep 21 23:57:27 2016 +0800
Committer: Ruilong Huo <r...@pivotal.io>
Committed: Sun Oct 9 11:35:45 2016 +0800

----------------------------------------------------------------------
 src/backend/resourcemanager/requesthandler.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-hawq/blob/4b3fdb26/src/backend/resourcemanager/requesthandler.c
----------------------------------------------------------------------
diff --git a/src/backend/resourcemanager/requesthandler.c 
b/src/backend/resourcemanager/requesthandler.c
index 9946322..976156b 100644
--- a/src/backend/resourcemanager/requesthandler.c
+++ b/src/backend/resourcemanager/requesthandler.c
@@ -627,7 +627,11 @@ bool handleRMSEGRequestIMAlive(void **arg)
        fts_client_ip_len = strlen(fts_client_ip);
        inet_aton(fts_client_ip, &fts_client_addr);
        fts_client_host = gethostbyaddr(&fts_client_addr, 4, AF_INET);
-       Assert(fts_client_host != NULL);
+       if (fts_client_host == NULL)
+       {
+               elog(WARNING, "Failed to reverse DNS lookup for ip %s.", 
fts_client_ip);
+               return true;
+       }
 
        /* Get the received machine id instance start address. */
        SegInfo fts_client_seginfo = 
(SegInfo)(SMBUFF_CONTENT(&(conntrack->MessageBuff)) +

Reply via email to