Postgres Pro
Downloads
Home   >   mailing lists

Re: Thread-unsafe MD5 on big-endian systems with no OpenSSL - Mailing list pgsql-hackers

From Robert Haas
Subject Re: Thread-unsafe MD5 on big-endian systems with no OpenSSL
Date
Msg-id CA+TgmobCwNGU+CKUruJQ_F1+++c9f+P0bh2An2sUO-bKSEC1Sg@mail.gmail.com
Whole thread Raw
In response to Thread-unsafe MD5 on big-endian systems with no OpenSSL  (Heikki Linnakangas <hlinnaka@iki.fi>)
Responses Re: Thread-unsafe MD5 on big-endian systems with no OpenSSL
List pgsql-hackers
Tree view 
On Tue, Aug 6, 2024 at 8:23 AM Heikki Linnakangas <hlinnaka@iki.fi> wrote:
> While browsing through all our global variables for the multithreading
> effort, I noticed that our MD5 implementation in src/common/md5.c uses a
> static buffer on big-endian systems, which makes it not thread-safe.
> That's a bug because that function is also used in libpq.
>
> This was introduced in commit b67b57a966, which replaced the old MD5
> fallback implementation with the one from pgcrypto. The thread-safety
> didn't matter for pgcrypto, but for libpq it does.
>
> This only affects big-endian systems that are compiled without OpenSSL.

LGTM.

--
Robert Haas
EDB: http://www.enterprisedb.com

pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: Detailed release notes
Next
From: Tom Lane
Date:
Subject: Re: Fix comments in instr_time.h and remove an unneeded cast to int64