Commitf6a6d20

committed

Check for NULL result from strdup

Per Coverity Scan

1 parent0483608 commitf6a6d20Copy full SHA for f6a6d20

File tree

+11

-1

lines changed

+11

-1

lines changed

Lines changed: 11 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1130,7 +1130,17 @@ initialize_SSL(PGconn *conn)`
`1130`	`1130`	`{`
`1131`	`1131`	`/* Colon, but not in second character, treat as engine:key */`
`1132`	`1132`	`char*engine_str=strdup(conn->sslkey);`
`1133`		`-char*engine_colon=strchr(engine_str,':');`
	`1133`	`+char*engine_colon;`
	`1134`	`+`
	`1135`	`+if (engine_str==NULL)`
	`1136`	`+{`
	`1137`	`+printfPQExpBuffer(&conn->errorMessage,`
	`1138`	`+libpq_gettext("out of memory\n"));`
	`1139`	`+return-1;`
	`1140`	`+}`
	`1141`	`+`
	`1142`	`+/* cannot return NULL because we already checked before strdup */`
	`1143`	`+engine_colon=strchr(engine_str,':');`
`1134`	`1144`
`1135`	`1145`	`engine_colon='\0';/ engine_str now has engine name */`
`1136`	`1146`	`engine_colon++;/* engine_colon now has key name */`

Comments

(0)