Postgres Pro
Downloads
Home   >   mailing lists

Possible NULL dereferencing null pointer (src/backend/executor/nodeIncrementalSort.c) - Mailing list pgsql-hackers

From Ranier Vilela
Subject Possible NULL dereferencing null pointer (src/backend/executor/nodeIncrementalSort.c)
Date
Msg-id CAEudQAp90GsR2CK9BJcimj9E9Jw9bgm=-Tv3Bea3Co+1XSBkPw@mail.gmail.com
Whole thread Raw
Responses Re: Possible NULL dereferencing null pointer (src/backend/executor/nodeIncrementalSort.c)  (Tomas Vondra <tomas.vondra@2ndquadrant.com>)
List pgsql-hackers
Tree view
I think that TupIsNull macro is no longer appropriate, to protect
ExecCopySlot.

See at tuptable.h:
#define TupIsNull(slot) \
((slot) == NULL || TTS_EMPTY(slot))

If var node->group_pivot is NULL, ExecCopySlot will
dereference a null pointer (first arg).

Maybe, this can be related to a bug reported in the btree deduplication.

regards,
Ranier Vilela
Attachment

pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Wrong example in the bloom documentation
Next
From: Magnus Hagander
Date:
Subject: Re: [PATCH] ecpg: fix progname memory leak