diff options
| author | Nathan Bossart <nathan@postgresql.org> | 2025-07-24 10:13:45 -0500 |
|---|---|---|
| committer | Nathan Bossart <nathan@postgresql.org> | 2025-07-24 10:13:45 -0500 |
| commit | 15d33eb1924c1093102b8ce142ede4cb3912e85e (patch) | |
| tree | 41da66b92de95c1bdc203fd1621865d85ce8aadd /src | |
| parent | e1c3654839e464957675344a1e949489d98b103b (diff) | |
| download | postgresql-15d33eb1924c1093102b8ce142ede4cb3912e85e.tar.gz postgresql-15d33eb1924c1093102b8ce142ede4cb3912e85e.zip | |
Fix return value of visibilitymap_get_status().
This function is declared as returning a uint8, but it returns a
bool in one code path. To fix, return (uint8) 0 instead of false
there. This should behave exactly the same as before, but it might
prevent future compiler complaints.
Oversight in commit a892234f83.
Author: Julien Rouhaud <rjuju123@gmail.com>
Discussion: https://postgr.es/m/aIHluT2isN58jqHV%40jrouhaud
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/access/heap/visibilitymap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/access/heap/visibilitymap.c b/src/backend/access/heap/visibilitymap.c index 745a04ef26e..8f918e00af7 100644 --- a/src/backend/access/heap/visibilitymap.c +++ b/src/backend/access/heap/visibilitymap.c @@ -364,7 +364,7 @@ visibilitymap_get_status(Relation rel, BlockNumber heapBlk, Buffer *vmbuf) { *vmbuf = vm_readbuf(rel, mapBlock, false); if (!BufferIsValid(*vmbuf)) - return false; + return (uint8) 0; } map = PageGetContents(BufferGetPage(*vmbuf)); |