database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-11-03 09:13:20 +03:00
Code Activity

Avoid null pointer dereference if error result lacks SQLSTATE.

Browse Source 
Although error results received from the backend should always have
a SQLSTATE field, ones generated by libpq won't, making this code
vulnerable to a crash after, say, untimely loss of connection.
Noted by Coverity.

Oversight in commit 403a3d91c.  Back-patch to 9.5, as that was.
This commit is contained in:
Tom Lane
2020-11-01 11:26:16 -05:00
parent c39f4e81dc
commit dd7ae03f8c
1 changed files with 8 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/bin/pg_dump/pg_backup_db.c
View File

@@ -575,13 +575,15 @@ IsLockTableGeneric(Archive *AHX)
break; break;
case PGRES_FATAL_ERROR: case PGRES_FATAL_ERROR:
sqlstate = PQresultErrorField(res, PG_DIAG_SQLSTATE); sqlstate = PQresultErrorField(res, PG_DIAG_SQLSTATE);
if (strcmp(sqlstate, ERRCODE_WRONG_OBJECT_TYPE) == 0) if (sqlstate &&
strcmp(sqlstate, ERRCODE_WRONG_OBJECT_TYPE) == 0)
{ {
retval = false; retval = false;
break; break;
} }
else if (strcmp(sqlstate, ERRCODE_LOCK_NOT_AVAILABLE) == 0 || else if (sqlstate &&
strcmp(sqlstate, ERRCODE_INSUFFICIENT_PRIVILEGE) == 0) (strcmp(sqlstate, ERRCODE_LOCK_NOT_AVAILABLE) == 0 ||
strcmp(sqlstate, ERRCODE_INSUFFICIENT_PRIVILEGE) == 0))
{ {
retval = true; retval = true;
break; break;