> > On Fri, Dec 21, 2001 at 11:43:21AM +0800, Christopher Kings-Lynne
authorBruce Momjian <bruce@momjian.us>
Thu, 3 Jan 2002 07:21:48 +0000 (07:21 +0000)
committerBruce Momjian <bruce@momjian.us>
Thu, 3 Jan 2002 07:21:48 +0000 (07:21 +0000)
commita5e8242afed70860b02f31e7588e7a192546fb2d
tree5022317ac4c0051407eabc34dbb96f1e7dbbc6cf
parent7514fcd4c06ed3859e6a899e3e092c80642f89dd
> > On Fri, Dec 21, 2001 at 11:43:21AM +0800, Christopher Kings-Lynne
wrote:
> > > Just testing pgcrypto on freebsd/alpha.  I get some warnings:
> > They should be harmless, although I should fix them.
>
> The actual code is:
>
>     if ((dlen & 15) || (((unsigned) res) & 3))
>         return -1;

> Hard to imagine how (uint *) & 3 makes any sense, unless res isn't
> always a (uint8 *).  Is that true?

At some point it was casted to (uint32*) so I wanted to be sure its ok.
ATM its pointless.  Please apply the following patch.

--
marko
contrib/pgcrypto/internal.c