Re: 4 pgcrypto regressions failures - 1 unsolved

Marko Kreen <marko@l-t.ee> writes:
> Result is - it's not so bad.  As I used rijndael.c to provide
> OpenSSL's own interface, I even got rid of all the ifdefs inside
> the code.

Looks good, but I'm still getting these compile warnings:

openssl.c: In function `ossl_des3_ecb_encrypt':
openssl.c:484: warning: passing arg 1 of `DES_ecb3_encrypt' from incompatible pointer type
openssl.c:484: warning: passing arg 2 of `DES_ecb3_encrypt' from incompatible pointer type
openssl.c: In function `ossl_des3_ecb_decrypt':
openssl.c:498: warning: passing arg 1 of `DES_ecb3_encrypt' from incompatible pointer type
openssl.c:498: warning: passing arg 2 of `DES_ecb3_encrypt' from incompatible pointer type

The following addition to the patch shuts up gcc with openssl 0.9.7a,
but I'm not sure if it will break anything with older openssl ---
comments?
        regards, tom lane


*** /home/postgres/pgsql/contrib/pgcrypto/openssl.c    Sun Jul 10 12:35:38 2005
--- new/openssl.c    Mon Jul 11 10:06:30 2005
***************
*** 446,452 ****     ossldata   *od = c->ptr;      for (i = 0; i < dlen / bs; i++)
!         DES_ecb3_encrypt(data + i * bs, res + i * bs,                          &od->u.des3.k1, &od->u.des3.k2,
&od->u.des3.k3,1);     return 0; }
 
--- 480,487 ----     ossldata   *od = c->ptr;      for (i = 0; i < dlen / bs; i++)
!         DES_ecb3_encrypt((const_DES_cblock *) (data + i * bs),
!                          (DES_cblock *) (res + i * bs),                          &od->u.des3.k1, &od->u.des3.k2,
&od->u.des3.k3,1);     return 0; }
 
***************
*** 460,466 ****     ossldata   *od = c->ptr;      for (i = 0; i < dlen / bs; i++)
!         DES_ecb3_encrypt(data + i * bs, res + i * bs,                          &od->u.des3.k1, &od->u.des3.k2,
&od->u.des3.k3,0);     return 0; }
 
--- 495,502 ----     ossldata   *od = c->ptr;      for (i = 0; i < dlen / bs; i++)
!         DES_ecb3_encrypt((const_DES_cblock *) (data + i * bs),
!                          (DES_cblock *) (res + i * bs),                          &od->u.des3.k1, &od->u.des3.k2,
&od->u.des3.k3,0);     return 0; }