postgres.git / summary / log / commit / refs

commit    096f1ccd5290286b135822bb282fa884454d4b69
Author:   Robert Haas <rhaas@postgresql.org>
Date:     Tue Jun 13 16:36:45 2017 +0000
First released in: 10.0

    Always initialize PartitionBoundInfoData's null_index.
    
    This doesn't actually matter at present, because the current code
    never consults null_index for range partitions.  However, leaving
    it uninitialized is still a bad idea, so let's not do that.
    
    Amul Sul, reviewed by Ashutosh Bapat
    
    Discussion: http://postgr.es/m/CAAJ_b94AkEzcx+12ySCnbMDX7=UdF4BjnoBGfMQbB0RNSTo3Ng@mail.gmail.com


 src/backend/catalog/partition.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/backend/catalog/partition.c b/src/backend/catalog/partition.c
index 5c5a9e11ab1..a7c9b9a46c1 100644
--- a/src/backend/catalog/partition.c
+++ b/src/backend/catalog/partition.c
@@ -454,6 +454,7 @@ RelationBuildPartitionDesc(Relation rel)
 			palloc0(sizeof(PartitionBoundInfoData));
 		boundinfo->strategy = key->strategy;
 		boundinfo->ndatums = ndatums;
+		boundinfo->null_index = -1;
 		boundinfo->datums = (Datum **) palloc0(ndatums * sizeof(Datum *));
 
 		/* Initialize mapping array with invalid values */
@@ -503,8 +504,6 @@ RelationBuildPartitionDesc(Relation rel)
 							mapping[null_index] = next_index++;
 						boundinfo->null_index = mapping[null_index];
 					}
-					else
-						boundinfo->null_index = -1;
 
 					/* All partition must now have a valid mapping */
 					Assert(next_index == nparts);


[parent: b6263cd851ef]