"Qingqing Zhou" <zhouqq(at)cs(dot)toronto(dot)edu> writes:
> In general, code snippet like this:
> if (hash_search(..., HASH_ENTER, ...) == NULL)
> action_except_elog__ERROR__;
> are considered unsafe if: (1) the allocation method of the target hash table
> could elog(ERROR) themselves and (2) the reaction to the failure of
> hash_search() is not elog(ERROR).
I've made some changes to hopefully prevent this type of thinko again.
Thanks for spotting it.
regards, tom lane