Don't read fields of a misaligned ExpandedObjectHeader or AnyArrayType.

UBSan complains about this.  Instead, cast to a suitable type requiring
only 4-byte alignment.  DatumGetAnyArrayP() already assumes one can cast
between AnyArrayType and ArrayType, so this doesn't introduce a new
assumption.  Back-patch to 9.5, where AnyArrayType was introduced.

Reviewed by Tom Lane.

Discussion: https://postgr.es/m/20190629210334.GA1244217@rfd.leadboat.com

diff --git a/src/backend/utils/adt/arrayfuncs.c b/src/backend/utils/adt/arrayfuncs.c
index 3ae15562465ba9b7d7b23167b8d88eda952ff171..f9f621e7591cdb5587f835add787d7e6091290ea 100644 (file)
--- a/src/backend/utils/adt/arrayfuncs.c
+++ b/src/backend/utils/adt/arrayfuncs.c
@@ -4159,7 +4159,7 @@ array_contain_compare(AnyArrayType *array1, AnyArrayType *array2, Oid collation,
                nelems2 = array2->xpn.nelems;
        }
        else
-               deconstruct_array(&(array2->flt),
+               deconstruct_array((ArrayType *) array2,
                                                  element_type, typlen, typbyval, typalign,
                                                  &values2, &nulls2, &nelems2);
 

diff --git a/src/include/utils/array.h b/src/include/utils/array.h
index 7c3f94706a35a1238a64440c21bf46fe874b67f4..b441eb452b94a71c92e2f174a6e657a6b57e7e7b 100644 (file)
--- a/src/include/utils/array.h
+++ b/src/include/utils/array.h
@@ -157,7 +157,10 @@ typedef struct ExpandedArrayHeader
 
 /*
  * Functions that can handle either a "flat" varlena array or an expanded
- * array use this union to work with their input.
+ * array use this union to work with their input.  Don't refer to "flt";
+ * instead, cast to ArrayType.  This struct nominally requires 8-byte
+ * alignment on 64-bit, but it's often used for an ArrayType having 4-byte
+ * alignment.  UBSan complains about referencing "flt" in such cases.
  */
 typedef union AnyArrayType
 {
@@ -311,17 +314,21 @@ typedef struct ArrayIteratorData *ArrayIterator;
  * Macros for working with AnyArrayType inputs.  Beware multiple references!
  */
 #define AARR_NDIM(a) \
-       (VARATT_IS_EXPANDED_HEADER(a) ? (a)->xpn.ndims : ARR_NDIM(&(a)->flt))
+       (VARATT_IS_EXPANDED_HEADER(a) ? \
+        (a)->xpn.ndims : ARR_NDIM((ArrayType *) (a)))
 #define AARR_HASNULL(a) \
        (VARATT_IS_EXPANDED_HEADER(a) ? \
         ((a)->xpn.dvalues != NULL ? (a)->xpn.dnulls != NULL : ARR_HASNULL((a)->xpn.fvalue)) : \
-        ARR_HASNULL(&(a)->flt))
+        ARR_HASNULL((ArrayType *) (a)))
 #define AARR_ELEMTYPE(a) \
-       (VARATT_IS_EXPANDED_HEADER(a) ? (a)->xpn.element_type : ARR_ELEMTYPE(&(a)->flt))
+       (VARATT_IS_EXPANDED_HEADER(a) ? \
+        (a)->xpn.element_type : ARR_ELEMTYPE((ArrayType *) (a)))
 #define AARR_DIMS(a) \
-       (VARATT_IS_EXPANDED_HEADER(a) ? (a)->xpn.dims : ARR_DIMS(&(a)->flt))
+       (VARATT_IS_EXPANDED_HEADER(a) ? \
+        (a)->xpn.dims : ARR_DIMS((ArrayType *) (a)))
 #define AARR_LBOUND(a) \
-       (VARATT_IS_EXPANDED_HEADER(a) ? (a)->xpn.lbound : ARR_LBOUND(&(a)->flt))
+       (VARATT_IS_EXPANDED_HEADER(a) ? \
+        (a)->xpn.lbound : ARR_LBOUND((ArrayType *) (a)))
 
 
 /*

diff --git a/src/include/utils/arrayaccess.h b/src/include/utils/arrayaccess.h
index 0917805a029afe32dfebed513abbf0f3a8396665..47e74260d8caecacb5c37c68f56f7e1ed5a3149e 100644 (file)
--- a/src/include/utils/arrayaccess.h
+++ b/src/include/utils/arrayaccess.h
@@ -71,8 +71,8 @@ array_iter_setup(array_iter *it, AnyArrayType *a)
        {
                it->datumptr = NULL;
                it->isnullptr = NULL;
-               it->dataptr = ARR_DATA_PTR(&a->flt);
-               it->bitmapptr = ARR_NULLBITMAP(&a->flt);
+               it->dataptr = ARR_DATA_PTR((ArrayType *) a);
+               it->bitmapptr = ARR_NULLBITMAP((ArrayType *) a);
        }
        it->bitmask = 1;
 }

diff --git a/src/include/utils/expandeddatum.h b/src/include/utils/expandeddatum.h
index 12e3ac2be17a3dc3d83c6131cc6dea8cf74a7da0..8970fa6b7a425d5e53334654ae0bc26981aef462 100644 (file)
--- a/src/include/utils/expandeddatum.h
+++ b/src/include/utils/expandeddatum.h
@@ -126,7 +126,7 @@ struct ExpandedObjectHeader
  */
 #define EOH_HEADER_MAGIC (-1)
 #define VARATT_IS_EXPANDED_HEADER(PTR) \
-       (((ExpandedObjectHeader *) (PTR))->vl_len_ == EOH_HEADER_MAGIC)
+       (((varattrib_4b *) (PTR))->va_4byte.va_header == EOH_HEADER_MAGIC)
 
 /*
  * Generic support functions for expanded objects.