NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitc6605c1

committed

Don't test already-referenced pointer for nullness

Commitb8ba734 added in PQgetResult a derefence to a pointerreturned by pqPrepareAsyncResult(), before some other code that wasalready testing that pointer for nullness. But since commit618c167 (in Postgres 15), pqPrepareAsyncResult() doesn't everreturn NULL (a statically-allocated result is returned if OOM). So inbranches 15 and up, we can remove the redundant pointer check with noharm done.However, in branch 14, pqPrepareAsyncResult() can indeed return NULL ifit runs out of memory. Fix things there by adding a null pointer checkbefore dereferencing the pointer. This should hint Coverity that thepreexisting check is not redundant but necessary.Backpatch to 14, likeb8ba734.Per Coverity.

1 parenta37bb7c commitc6605c1Copy full SHA for c6605c1

File tree

1 file changed

-1

lines changed

src/interfaces/libpq
- fe-exec.c

1 file changed

-1

lines changed

`‎src/interfaces/libpq/fe-exec.c`

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -842,6 +842,8 @@ pqSaveWriteError(PGconn *conn)`
`842`	`842`	`* using whatever is in conn->errorMessage. In any case, clear the async`
`843`	`843`	`* result storage, and update our notion of how much error text has been`
`844`	`844`	`* returned to the application.`
	`845`	`+ *`
	`846`	`+ * Note that in no case (not even OOM) do we return NULL.`
`845`	`847`	`*/`
`846`	`848`	`PGresult*`
`847`	`849`	`pqPrepareAsyncResult(PGconn*conn)`
`@@ -2138,7 +2140,7 @@ PQgetResult(PGconn *conn)`
`2138`	`2140`	`* (In other words: we don't return a NULL after a pipeline`
`2139`	`2141`	`* sync.)`
`2140`	`2142`	`*/`
`2141`		`-if (res&&res->resultStatus==PGRES_PIPELINE_SYNC)`
	`2143`	`+if (res->resultStatus==PGRES_PIPELINE_SYNC)`
`2142`	`2144`	`pqPipelineProcessQueue(conn);`
`2143`	`2145`	`}`
`2144`	`2146`	`else`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitc6605c1

File tree

1 file changed

1 file changed

`‎src/interfaces/libpq/fe-exec.c`

0 commit comments