mirrors/postgres
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix dangling pointer problem in ReorderBufferSerializeChange.

Browse Source 
Commit 3fe3511d05127cc024b221040db2eeb352e7d716 introduced a new
case into this function, but neglected to ensure that the "ondisk"
pointer got updated after a possible reallocation as the code does
in other cases.

Stas Kelvich, per diagnosis by Konstantin Knizhnik.
This commit is contained in:
Robert Haas 2016-09-28 11:19:46 -04:00
parent babe05bc2b
commit 308985b0b4
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/backend/replication/logical/reorderbuffer.c
View File

@ -2250,6 +2250,9 @@ ReorderBufferSerializeChange(ReorderBuffer *rb, ReorderBufferTXN *txn,
data = ((char *) rb->outbuf) + sizeof(ReorderBufferDiskChange); data = ((char *) rb->outbuf) + sizeof(ReorderBufferDiskChange);
/* might have been reallocated above */
ondisk = (ReorderBufferDiskChange *) rb->outbuf;
/* write the prefix including the size */ /* write the prefix including the size */
memcpy(data, &prefix_size, sizeof(Size)); memcpy(data, &prefix_size, sizeof(Size));
data += sizeof(Size); data += sizeof(Size);