Silence compiler warnings on clang 21

Clang 21 shows some new compiler warnings, for example:

warning: variable 'dstsize' is uninitialized when passed as a const pointer argument here [-Wuninitialized-const-pointer]

The fix is to initialize the variables when they are defined.  This is
similar to, for example, the existing situation in gistKeyIsEQ().

Discussion: https://www.postgresql.org/message-id/flat/6604ad6e-5934-43ac-8590-15113d6ae4b1%40eisentraut.org

diff --git a/src/backend/access/common/toast_internals.c b/src/backend/access/common/toast_internals.c
index 90d0654e6298bb40910ccd2b5e59ba217d709a91..52972d894e293bb104aedd1e576b7546a81ce1bb 100644 (file)
--- a/src/backend/access/common/toast_internals.c
+++ b/src/backend/access/common/toast_internals.c
@@ -135,7 +135,7 @@ toast_save_datum(Relation rel, Datum value,
                char            data[TOAST_MAX_CHUNK_SIZE + VARHDRSZ];
                /* ensure union is aligned well enough: */
                int32           align_it;
-       }                       chunk_data;
+       }                       chunk_data = {0};       /* silence compiler warning */
        int32           chunk_size;
        int32           chunk_seq = 0;
        char       *data_p;

diff --git a/src/backend/access/gist/gistutil.c b/src/backend/access/gist/gistutil.c
index 78e98d68b15184258684e5a6f92209ef8242deb2..7844c6074fefa5d0f663c3a64c15eed39040684f 100644 (file)
--- a/src/backend/access/gist/gistutil.c
+++ b/src/backend/access/gist/gistutil.c
@@ -156,7 +156,7 @@ gistMakeUnionItVec(GISTSTATE *giststate, IndexTuple *itvec, int len,
 {
        int                     i;
        GistEntryVector *evec;
-       int                     attrsize;
+       int                     attrsize = 0;   /* silence compiler warning */
 
        evec = (GistEntryVector *) palloc((len + 2) * sizeof(GISTENTRY) + GEVHDRSZ);
 
@@ -241,7 +241,7 @@ gistMakeUnionKey(GISTSTATE *giststate, int attno,
                char            padding[2 * sizeof(GISTENTRY) + GEVHDRSZ];
        }                       storage;
        GistEntryVector *evec = &storage.gev;
-       int                     dstsize;
+       int                     dstsize = 0;    /* silence compiler warning */
 
        evec->n = 2;