From 45cc0e9b27c12e0e7cfad23e9e3105159ed569b7 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Fri, 28 Apr 2000 00:12:44 +0000 Subject: [PATCH] plpgsql RAISE statement was careless about the possibility of a NULL field value being displayed; produced coredump instead of the expected display. --- src/pl/plpgsql/src/pl_exec.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/pl/plpgsql/src/pl_exec.c b/src/pl/plpgsql/src/pl_exec.c index d9acac4947..a2565a4363 100644 --- a/src/pl/plpgsql/src/pl_exec.c +++ b/src/pl/plpgsql/src/pl_exec.c @@ -3,7 +3,7 @@ * procedural language * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/pl/plpgsql/src/pl_exec.c,v 1.20 2000/04/12 17:17:19 momjian Exp $ + * $Header: /cvsroot/pgsql/src/pl/plpgsql/src/pl_exec.c,v 1.21 2000/04/28 00:12:44 tgl Exp $ * * This software is copyrighted by Jan Wieck - Hamburg. * @@ -1587,6 +1587,8 @@ exec_stmt_raise(PLpgSQL_execstate * estate, PLpgSQL_stmt_raise * stmt) if (fno == SPI_ERROR_NOATTRIBUTE) elog(ERROR, "record %s has no field %s", rec->refname, recfield->fieldname); extval = SPI_getvalue(rec->tup, rec->tupdesc, fno); + if (extval == NULL) + extval = ""; } plpgsql_dstring_append(&ds, extval); break; -- GitLab