Re: Possible dereference null return (src/backend/replication/logical/reorderbuffer.c)

[Zhihong Yu]

Hi,
Patch v4 corrects a small typo:
+               (errmsg("BaseSnapshot cant't be setup at point %X/%X",

Cheers

On Sat, Feb 13, 2021 at 12:58 PM Ranier Vilela <ranier...@gmail.com> wrote:

> Em sáb., 13 de fev. de 2021 às 17:48, Zhihong Yu <z...@yugabyte.com>
> escreveu:
>
>> Hi,
>> +               (errmsg("BaseSnapshot cant't be setup at point %X/%X",
>> +                       (uint32) (lsn >> 32), (uint32) lsn),
>> +                errdetail("Top transaction is running.")));
>>
>> Did you mean this errdetail:
>>
>> Top transaction is not running.
>>
> Done.
>
> Thanks Zhihong.
> v3 based on your patch, attached.
>
> regards,
> Ranier Vilela
>

reorder-buffer-v4.patch
Description: Binary data