From e39bc695d683dcd6ae376d38e9acb91ae37642de Mon Sep 17 00:00:00 2001 From: Peter Eisentraut Date: Fri, 8 Aug 2003 13:16:20 +0000 Subject: Be a little bit more careful about using sqlstate, in case libpq returned NULL for it. --- src/interfaces/ecpg/ecpglib/error.c | 25 ++++++++++++++++++------- 1 file changed, 18 insertions(+), 7 deletions(-) (limited to 'src/interfaces/ecpg/ecpglib/error.c') diff --git a/src/interfaces/ecpg/ecpglib/error.c b/src/interfaces/ecpg/ecpglib/error.c index 33bac6e892..82f2ccca36 100644 --- a/src/interfaces/ecpg/ecpglib/error.c +++ b/src/interfaces/ecpg/ecpglib/error.c @@ -1,4 +1,4 @@ -/* $Header: /cvsroot/pgsql/src/interfaces/ecpg/ecpglib/error.c,v 1.6 2003/08/04 00:43:32 momjian Exp $ */ +/* $Header: /cvsroot/pgsql/src/interfaces/ecpg/ecpglib/error.c,v 1.7 2003/08/08 13:16:20 petere Exp $ */ #define POSTGRES_ECPG_INTERNAL #include "postgres_fe.h" @@ -159,18 +159,29 @@ void ECPGraise_backend(int line, PGresult *result, PGconn *conn, int compat) { struct sqlca_t *sqlca = ECPGget_sqlca(); + char *sqlstate; + char *message; + + if (result) + { + sqlstate = PQresultErrorField(result, 'C'); + if (sqlstate == NULL) + sqlstate = ECPG_SQLSTATE_ECPG_INTERNAL_ERROR; + message = PQresultErrorField(result, 'M'); + } + else + { + sqlstate = ECPG_SQLSTATE_ECPG_INTERNAL_ERROR; + message = PQerrorMessage(conn); + } /* copy error message */ snprintf(sqlca->sqlerrm.sqlerrmc, sizeof(sqlca->sqlerrm.sqlerrmc), - "'%s' in line %d.", - result ? PQresultErrorField(result, 'M') : PQerrorMessage(conn), - line); + "'%s' in line %d.", message, line); sqlca->sqlerrm.sqlerrml = strlen(sqlca->sqlerrm.sqlerrmc); /* copy SQLSTATE */ - strncpy(sqlca->sqlstate, - result ? PQresultErrorField(result, 'C') : ECPG_SQLSTATE_ECPG_INTERNAL_ERROR, - sizeof(sqlca->sqlstate)); + strncpy(sqlca->sqlstate, sqlstate, sizeof(sqlca->sqlstate)); /* assign SQLCODE for backward compatibility */ if (strncmp(sqlca->sqlstate, "23505", sizeof(sqlca->sqlstate)) == 0) -- cgit v1.2.1