diff options
| author | Tomas Vondra | 2018-10-12 12:32:54 +0000 |
|---|---|---|
| committer | Tomas Vondra | 2018-10-12 12:35:34 +0000 |
| commit | 1cc1df104888bd4fd2c9c86aa6846543c78609be (patch) | |
| tree | dbe06f2daabdfa661b418d8e91d81dc88a7c6b6f | |
| parent | cf2518f48d2d1daf0807a774177b21f9549ec8a7 (diff) | |
The check is supposed to ensure NULL/empty nodename gets hashed to 0,
but (nodename == '\0') is comparing the pointer itself, not the first
character. So dereference that correctly.
| -rw-r--r-- | src/gtm/recovery/register_common.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/gtm/recovery/register_common.c b/src/gtm/recovery/register_common.c index ff0f960e33..e0afe1425f 100644 --- a/src/gtm/recovery/register_common.c +++ b/src/gtm/recovery/register_common.c @@ -174,7 +174,7 @@ pgxcnode_find_info(GTM_PGXCNodeType type, char *node_name) /* * Get the Hash Key depending on the node name - * We do not except to have hundreds of nodes yet, + * We do not expect to have hundreds of nodes yet, * This function could be replaced by a better one * such as a double hash function indexed on type and Node Name */ @@ -186,7 +186,7 @@ pgxcnode_gethash(char *nodename) int value; uint32 hash = 0; - if (nodename == NULL || nodename == '\0') + if (nodename == NULL || *nodename == '\0') { return 0; } |