Better test the content of the SQLSTATE string in ecpglib than the pointer.

Michael Meskes · Michael Meskes · commit 6e2cf34ec12f · 2010-03-08T12:15:24.000Z
diff --git a/src/interfaces/ecpg/ecpglib/error.c b/src/interfaces/ecpg/ecpglib/error.c
@@ -1,4 +1,4 @@
-/* $PostgreSQL: pgsql/src/interfaces/ecpg/ecpglib/error.c,v 1.24 2010/03/05 13:57:08 meskes Exp $ */
+/* $PostgreSQL: pgsql/src/interfaces/ecpg/ecpglib/error.c,v 1.25 2010/03/08 12:15:24 meskes Exp $ */
 
 #define POSTGRES_ECPG_INTERNAL
 #include "postgres_fe.h"
@@ -306,7 +306,7 @@ ecpg_raise_backend(int line, PGresult *result, PGconn *conn, int compat)
 		message = PQerrorMessage(conn);
 	}
 
-	if (sqlstate == ECPG_SQLSTATE_ECPG_INTERNAL_ERROR)
+	if (strcmp(sqlstate, ECPG_SQLSTATE_ECPG_INTERNAL_ERROR) == 0)
 	{
 		/* we might get here if the connection breaks down, so let's
 		 * check for this instead of giving just the generic internal error */

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-/* $PostgreSQL: pgsql/src/interfaces/ecpg/ecpglib/error.c,v 1.24 2010/03/05 13:57:08 meskes Exp $ */`
	`1`	`+/* $PostgreSQL: pgsql/src/interfaces/ecpg/ecpglib/error.c,v 1.25 2010/03/08 12:15:24 meskes Exp $ */`
`2`	`2`
`3`	`3`	`#define POSTGRES_ECPG_INTERNAL`
`4`	`4`	`#include "postgres_fe.h"`
`@@ -306,7 +306,7 @@ ecpg_raise_backend(int line, PGresult result, PGconn conn, int compat)`
`306`	`306`	`message = PQerrorMessage(conn);`
`307`	`307`	`}`
`308`	`308`
`309`		`- if (sqlstate == ECPG_SQLSTATE_ECPG_INTERNAL_ERROR)`
	`309`	`+ if (strcmp(sqlstate, ECPG_SQLSTATE_ECPG_INTERNAL_ERROR) == 0)`
`310`	`310`	`{`
`311`	`311`	`/* we might get here if the connection breaks down, so let's`
`312`	`312`	`* check for this instead of giving just the generic internal error */`