plpgsql: make WHEN OTHERS distinct from WHEN SQLSTATE '00000'. The catchall exception condition OTHERS was represented as sqlerrstate == 0, which was a poor choice because that comes out the same as SQLSTATE '00000'. While we don't issue that as an error code ourselves, there isn't anything particularly stopping users from doing so. Use -1 instead, which can't match any allowed SQLSTATE string.
While at it, invent a macro PLPGSQL_OTHERS to use instead of a hard-coded magic number. While this seems like a bug fix, I'm inclined not to back-patch. It seems barely possible that someone has written code like this and would be annoyed by changing the behavior in a minor release. Reported-by: David Fiedler <david.fido.fied...@gmail.com> Author: Tom Lane <t...@sss.pgh.pa.us> Discussion: https://postgr.es/m/CAHjN70-=H5EpTOuZVbC8mPvRS5EfZ4MY2=oudvdwoygvkhb...@mail.gmail.com Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/58fdca2204de5f683f025df37553e5e69cb6adb1 Modified Files -------------- src/pl/plpgsql/src/pl_comp.c | 6 +----- src/pl/plpgsql/src/pl_exec.c | 2 +- src/pl/plpgsql/src/plpgsql.h | 5 ++++- 3 files changed, 6 insertions(+), 7 deletions(-)
