pgcrypto: Remove internal padding implementation
authorPeter Eisentraut <peter@eisentraut.org>
Tue, 22 Mar 2022 07:51:05 +0000 (08:51 +0100)
committerPeter Eisentraut <peter@eisentraut.org>
Tue, 22 Mar 2022 07:58:44 +0000 (08:58 +0100)
commitf5576a21b0778f275d7418f6f7a44d9400ee90aa
treecaa2d5c08c8d5f6cb8786119a6a86682078be076
parent9ca234bae79358a24de2a8dc1ec8024656ca66a4
pgcrypto: Remove internal padding implementation

Use the padding provided by OpenSSL instead of doing it ourselves.
The internal implementation was once applicable to the non-OpenSSL
code paths, but those have since been removed.  The padding algorithm
is still the same.

The OpenSSL padding implementation is stricter than the previous
internal one: Bad padding during decryption is now an error, and
encryption without padding now requires the input size to be a
multiple of the block size, otherwise it is also an error.
Previously, these cases silently proceeded, in spite of the
documentation saying otherwise.

Add some test cases about this, too.  (The test cases are in
rijndael.sql, but they apply to all encryption algorithms.)

Reviewed-by: Jacob Champion <pchampion@vmware.com>
Reviewed-by: Nathan Bossart <nathandbossart@gmail.com>
Discussion: https://www.postgresql.org/message-id/flat/ba94c26b-0c58-c97e-7a44-f44e08b4cca2%40enterprisedb.com
contrib/pgcrypto/expected/rijndael.out
contrib/pgcrypto/openssl.c
contrib/pgcrypto/pgp-cfb.c
contrib/pgcrypto/px.c
contrib/pgcrypto/px.h
contrib/pgcrypto/sql/rijndael.sql