Fix memory leak in libpq when using sslmode=verify-full

michaelpq · michaelpq · commit e5e295d5008b · 2020-04-22T07:28:04.000+09:00
Checking if Subject Alternative Names (SANs) from a certificate match with the hostname connected to leaked memory after each lookup done. This is broken since acd08d7 that added support for SANs in SSL certificates, so backpatch down to 9.5. Author: Roman Peshkurov Reviewed-by: Hamid Akhtar, Michael Paquier, David Steele Discussion: https://postgr.es/m/CALLDf-pZ-E3mjxd5=bnHsDu9zHEOnpgPgdnO84E2RuwMCjjyPw@mail.gmail.com Backpatch-through: 9.5
diff --git a/src/interfaces/libpq/fe-secure-openssl.c b/src/interfaces/libpq/fe-secure-openssl.c
@@ -619,7 +619,7 @@ verify_peer_name_matches_certificate(PGconn *conn)
 			if (found_match || got_error)
 				break;
 		}
-		sk_GENERAL_NAME_free(peer_san);
+		sk_GENERAL_NAME_pop_free(peer_san, GENERAL_NAME_free);
 	}
 
 	/*

Original file line number	Diff line number	Diff line change
`@@ -619,7 +619,7 @@ verify_peer_name_matches_certificate(PGconn *conn)`
`619`	`619`	`if (found_match \|\| got_error)`
`620`	`620`	`break;`
`621`	`621`	`}`
`622`		`- sk_GENERAL_NAME_free(peer_san);`
	`622`	`+ sk_GENERAL_NAME_pop_free(peer_san, GENERAL_NAME_free);`
`623`	`623`	`}`
`624`	`624`
`625`	`625`	`/*`