Fix blatantly broken record_image_cmp() logic for pass-by-value fields.

Doesn't anybody here pay attention to compiler warnings?

diff --git a/src/backend/utils/adt/rowtypes.c b/src/backend/utils/adt/rowtypes.c
index ae007cfe319e2f823480ad7c632bc48e7d79ddd2..cb04a724d3ba7a3794c742b987b59906efde5dc4 100644 (file)
--- a/src/backend/utils/adt/rowtypes.c
+++ b/src/backend/utils/adt/rowtypes.c
@@ -1430,7 +1430,7 @@ record_image_cmp(FunctionCallInfo fcinfo)
         */
        if (!nulls1[i1] || !nulls2[i2])
        {
-           int         cmpresult;
+           int         cmpresult = 0;
 
            if (nulls1[i1])
            {