diff --git a/src/bin/gs_guc/pg_guc.cpp b/src/bin/gs_guc/pg_guc.cpp
index 0c272a3a6..8772ba04b 100644
--- a/src/bin/gs_guc/pg_guc.cpp
+++ b/src/bin/gs_guc/pg_guc.cpp
@@ -1172,8 +1172,8 @@ do_gucset(const char *action_type, const char *data_dir)
             }
         }
 
-        if (strncmp(config_param[i], "replconninfo", strlen("replconninfo")) == 0 && 
-            config_value[i] != NULL && (config_value[i] == "" || strncmp(config_value[i], "''", strlen("''")) == 0) && 
+        if (strncmp(config_param[i], "replconninfo", strlen("replconninfo")) == 0 &&
+            config_value[i] != NULL && (strlen(config_value[i]) == 0 || strncmp(config_value[i], "''", strlen("''")) == 0) &&
             IsLastNotNullReplconninfo(opt_lines, config_param[i])) {
             write_stderr("\nWARNING: This is the last valid replConnInfo, once set to null, "
                 "the host role will be changed to Normal if the local_role is primary now.\n");