diff options
author | Bruce Momjian <bruce@momjian.us> | 2014-04-17 12:37:53 -0400 |
---|---|---|
committer | Bruce Momjian <bruce@momjian.us> | 2014-04-17 12:37:53 -0400 |
commit | fc02b87e2876b2492a3d5eebd3b70be383b08f40 (patch) | |
tree | 19301491c363e8f87b15d53f1f742b5975161749 /contrib/pgcrypto/pgp-mpi.c | |
parent | 179c45ae2fb9519a343bc2e38ebe4609097d14af (diff) | |
download | postgresql-fc02b87e2876b2492a3d5eebd3b70be383b08f40.tar.gz postgresql-fc02b87e2876b2492a3d5eebd3b70be383b08f40.zip |
pgcrypto: fix memset() calls that might be optimized away
Specifically, on-stack memset() might be removed, so:
* Replace memset() with px_memset()
* Add px_memset to copy_crlf()
* Add px_memset to pgp-s2k.c
Patch by Marko Kreen
Report by PVS-Studio
Backpatch through 8.4.
Diffstat (limited to 'contrib/pgcrypto/pgp-mpi.c')
-rw-r--r-- | contrib/pgcrypto/pgp-mpi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/contrib/pgcrypto/pgp-mpi.c b/contrib/pgcrypto/pgp-mpi.c index c8765b6d141..0daca3cda48 100644 --- a/contrib/pgcrypto/pgp-mpi.c +++ b/contrib/pgcrypto/pgp-mpi.c @@ -72,7 +72,7 @@ pgp_mpi_free(PGP_MPI *mpi) { if (mpi == NULL) return 0; - memset(mpi, 0, sizeof(*mpi) + mpi->bytes); + px_memset(mpi, 0, sizeof(*mpi) + mpi->bytes); px_free(mpi); return 0; } |