]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Ensure Soundex difference() function handles empty input sanely.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 16 May 2023 14:53:42 +0000 (10:53 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 16 May 2023 14:53:42 +0000 (10:53 -0400)
commiteaf99e4c4ae4084b8261e5e1f9c2c93004a98638
tree4432e741afe2b22cc98fa1b9ada1e4572b432173
parentf06156da18f67bc2c904c0a76b70dafcb14ca7c2
Ensure Soundex difference() function handles empty input sanely.

fuzzystrmatch's difference() function assumes that _soundex()
always initializes its output buffer fully.  This was not so for
the case of a string containing no alphabetic characters, resulting
in unstable output and Valgrind complaints.

Fix by using memset() to fill the whole buffer in the early-exit
case.  Also make some cosmetic improvements (I didn't care for the
random switches between "instr[0]" and "*instr" notation).

Report and diagnosis by Alexander Lakhin (bug #17935).
Back-patch to all supported branches.

Discussion: https://postgr.es/m/17935-b99316aa79c18513@postgresql.org
contrib/fuzzystrmatch/expected/fuzzystrmatch.out
contrib/fuzzystrmatch/fuzzystrmatch.c
contrib/fuzzystrmatch/sql/fuzzystrmatch.sql