postgres.git / summary / log / commit / refs

commit    308813846e737f7d92ffb4e033d65e3df6cbddba
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Sun Sep 22 18:21:07 2019 +0000
First released in: 13.0, 12.0

    Fix typo in tts_virtual_copyslot.
    
    The code used the destination slot's natts where it intended to
    use the source slot's natts.  Adding an Assert shows that there
    is no case in "make check-world" where these counts are different,
    so maybe this is a harmless bug, but it's still a bug.
    
    Takayuki Tsunakawa
    
    Discussion: https://postgr.es/m/0A3221C70F24FB45833433255569204D1FD34C0E@G01JPEXMBYT05


 src/backend/executor/execTuples.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/backend/executor/execTuples.c b/src/backend/executor/execTuples.c
index a5cb7bba0f8..8c550f160f6 100644
--- a/src/backend/executor/execTuples.c
+++ b/src/backend/executor/execTuples.c
@@ -243,7 +243,7 @@ tts_virtual_materialize(TupleTableSlot *slot)
 static void
 tts_virtual_copyslot(TupleTableSlot *dstslot, TupleTableSlot *srcslot)
 {
-	TupleDesc	srcdesc = dstslot->tts_tupleDescriptor;
+	TupleDesc	srcdesc = srcslot->tts_tupleDescriptor;
 
 	Assert(srcdesc->natts <= dstslot->tts_tupleDescriptor->natts);
 


[parent: ee98adede535]