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
0004-memcpy-null.patch
Description: Binary data