]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Handle empty-string edge cases correctly in strpos().
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 28 Oct 2019 16:21:13 +0000 (12:21 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 28 Oct 2019 16:21:13 +0000 (12:21 -0400)
commit43e43771bc4b03a9e33be7261cc84dc7f65ae9ef
tree0abbf3cca8c93d5aa7d45ac80424aedaccc5d96b
parentb1ad895e12f099f41c8dc1a16f1b3ba2a6e29e06
Handle empty-string edge cases correctly in strpos().

Commit 9556aa01c rearranged the innards of text_position() in a way
that would make it not work for empty search strings.  Which is fine,
because all callers of that code special-case an empty pattern in
some way.  However, the primary use-case (text_position itself) got
special-cased incorrectly: historically it's returned 1 not 0 for
an empty search string.  Restore the historical behavior.

Per complaint from Austin Drenski (via Shay Rojansky).
Back-patch to v12 where it got broken.

Discussion: https://postgr.es/m/CADT4RqAz7oN4vkPir86Kg1_mQBmBxCp-L_=9vRpgSNPJf0KRkw@mail.gmail.com
src/backend/utils/adt/varlena.c
src/test/regress/expected/strings.out
src/test/regress/sql/strings.sql