Fix calculation of how much shared memory is required to store a TOC.

Commit ac883ac453 refactored shm_toc_estimate() but changed its calculation
of shared memory size for TOC incorrectly. Previously this could cause too
large memory to be allocated.

Back- to v11 where the bug was introduced.

Author: Takayuki Tsunakawa
Discussion: https://postgr.es/m/TYAPR01MB2990BFB73170E2C4921E2C4DFEA80@TYAPR01MB2990.jpnprd01.prod.outlook.com

diff --git a/src/backend/storage/ipc/shm_toc.c b/src/backend/storage/ipc/shm_toc.c
index 83c7928d8821722ddb88251f314dc7ccb8d4c72d..863b98bf0545d13aa214ee34c0f17664e9e29991 100644 (file)
--- a/src/backend/storage/ipc/shm_toc.c
+++ b/src/backend/storage/ipc/shm_toc.c
@@ -265,8 +265,8 @@ shm_toc_estimate(shm_toc_estimator *e)
    Size        sz;
 
    sz = offsetof(shm_toc, toc_entry);
-   sz += add_size(sz, mul_size(e->number_of_keys, sizeof(shm_toc_entry)));
-   sz += add_size(sz, e->space_for_chunks);
+   sz = add_size(sz, mul_size(e->number_of_keys, sizeof(shm_toc_entry)));
+   sz = add_size(sz, e->space_for_chunks);
 
    return BUFFERALIGN(sz);
 }