NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit9bacdf9

committed

Don't leak malloc'd error string in libpqrcv_check_conninfo().

We leaked the error report from PQconninfoParse, when there wasone. It seems unlikely that real usage patterns would repeatthe failure often enough to create serious bloat, but let'sback-patch anyway to keep the code similar in all branches.Found via valgrind testing.Back-patch to v10 where this code was added.Discussion:https://postgr.es/m/3816764.1616104288@sss.pgh.pa.us

1 parent377b7a8 commit9bacdf9Copy full SHA for 9bacdf9

File tree

1 file changed

-1

lines changed

src/backend/replication/libpqwalreceiver
- libpqwalreceiver.c

1 file changed

-1

lines changed

`‎src/backend/replication/libpqwalreceiver/libpqwalreceiver.c`

Lines changed: 7 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -246,9 +246,15 @@ libpqrcv_check_conninfo(const char *conninfo)`
`246`	`246`
`247`	`247`	`opts=PQconninfoParse(conninfo,&err);`
`248`	`248`	`if (opts==NULL)`
	`249`	`+{`
	`250`	`+/* The error string is malloc'd, so we must free it explicitly */`
	`251`	`+char*errcopy=err ?pstrdup(err) :"out of memory";`
	`252`	`+`
	`253`	`+PQfreemem(err);`
`249`	`254`	`ereport(ERROR,`
`250`	`255`	`(errcode(ERRCODE_SYNTAX_ERROR),`
`251`		`-errmsg("invalid connection string syntax: %s",err)));`
	`256`	`+errmsg("invalid connection string syntax: %s",errcopy)));`
	`257`	`+}`
`252`	`258`
`253`	`259`	`PQconninfoFree(opts);`
`254`	`260`	`}`

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!

Commit9bacdf9

File tree

1 file changed

1 file changed

`‎src/backend/replication/libpqwalreceiver/libpqwalreceiver.c`

0 commit comments