On Sat, Jan 8, 2022 at 11:32 PM Tom Lane <t...@sss.pgh.pa.us> wrote:

> Noah Misch <n...@leadboat.com> writes:
> > On further thought, I would write it this way:
>
> > -             else
> > +             else if (ivlen != 0)
> >                       memcpy(ivbuf, iv, ivlen);
>
> FWIW, I liked the "ivlen > 0" formulation better.  They should be
> equivalent, because ivlen is unsigned, but it just seems like "> 0"
> is more natural.
>
>                         regards, tom lane
>

Patch v4 is attached.

Cheers

Attachment: 0004-memcpy-null.patch
Description: Binary data

Reply via email to