Clean up recent Coverity complaints.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 23 Jan 2022 17:51:38 +0000 (12:51 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 23 Jan 2022 17:51:38 +0000 (12:51 -0500)
commit353708e1fb2d8b1a34f6da2c25d4bb6633cf4493
treef94ce7c1ca2162319b6a541c4cbafeafddf3b06d
parentdc43fc9b3aa3e0fa9c84faddad6d301813580f88
Clean up recent Coverity complaints.

Commit 5c649fe15 introduced a memory leak into pg_basebackup's
parse_compress_options.  (I simplified nearby code while at it.)

Commit 9a974cbcb introduced a memory leak into pg_dump's
binary_upgrade_set_pg_class_oids.

Coverity also complained about a call of SnapBuildProcessChange that
ignored the result, unlike every other call of that function.  This
is evidently intentional, so add a (void) cast to indicate that.
(It's also old, dating to b89e15105; I suppose the reason it showed
up now is 7a5f6b474's recent rearrangement of nearby code.)
src/backend/replication/logical/decode.c
src/bin/pg_basebackup/pg_basebackup.c
src/bin/pg_dump/pg_dump.c