From c859294ca2eabf65c680fe311596388b36a5fcb8 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Mon, 14 Aug 2006 12:39:55 +0000 Subject: [PATCH] Remove hash_destroy calls in hash_create's failure paths. As noted by a Coverity warning, these are risky since the hashtable isn't necessarily fully set up yet. They're unnecessary anyway: a deletable hashtable should be in a memory context that will be cleared following elog(ERROR). Per report from Martijn van Oosterhout. --- src/backend/utils/hash/dynahash.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/src/backend/utils/hash/dynahash.c b/src/backend/utils/hash/dynahash.c index 23200667af..b2974da999 100644 --- a/src/backend/utils/hash/dynahash.c +++ b/src/backend/utils/hash/dynahash.c @@ -26,7 +26,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/utils/hash/dynahash.c,v 1.70 2006/07/22 23:04:39 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/utils/hash/dynahash.c,v 1.71 2006/08/14 12:39:55 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -392,10 +392,7 @@ hash_create(const char *tabname, long nelem, HASHCTL *info, int flags) /* Build the hash directory structure */ if (!init_htab(hashp, nelem)) - { - hash_destroy(hashp); elog(ERROR, "failed to initialize hash table"); - } /* * For a shared hash table, preallocate the requested number of elements. @@ -409,12 +406,9 @@ hash_create(const char *tabname, long nelem, HASHCTL *info, int flags) nelem < hctl->nelem_alloc) { if (!element_alloc(hashp, (int) nelem)) - { - hash_destroy(hashp); ereport(ERROR, (errcode(ERRCODE_OUT_OF_MEMORY), errmsg("out of memory"))); - } } return hashp; -- 2.40.0