pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Nathan Bossart <nathandbossart@gmail.com>
To: osumi.takamichi@fujitsu.com <osumi.takamichi@fujitsu.com>
Cc: Amit Kapila <amit.kapila16@gmail.com>
Cc: Masahiko Sawada <sawada.mshk@gmail.com>
Cc: Peter Smith <smithpb2250@gmail.com>
Cc: tanghy.fnst@fujitsu.com <tanghy.fnst@fujitsu.com>
Cc: wangw.fnst@fujitsu.com <wangw.fnst@fujitsu.com>
Cc: vignesh C <vignesh21@gmail.com>
Cc: Greg Nancarrow <gregn4422@gmail.com>
Cc: Mark Dilger <mark.dilger@enterprisedb.com>
Cc: Smith, Peter <peters@fast.au.fujitsu.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Subject: Re: Optionally automatically disable logical replication subscriptions on error
Date: Mon, 14 Mar 2022 16:04:24 -0700
Message-ID: <20220314230424.GA1085716@nathanxps13> (raw)
In-Reply-To: <TYCPR01MB83733807BF9349D8D0BD6DB4ED0F9@TYCPR01MB8373.jpnprd01.prod.outlook.com>
References: <CAA4eK1+6Gk9yRgEo8UbkJHut3jZLgdX30iULACMjeOgJefn9HA@mail.gmail.com>
	<TYCPR01MB8373B74627C6BAF2F146D779ED099@TYCPR01MB8373.jpnprd01.prod.outlook.com>
	<CAA4eK1+6_1fdo=EhLee_2OtX_bjj6chyeWtZexcKfSwAK3tuxQ@mail.gmail.com>
	<TYCPR01MB83731B6EBA7D7ACF3C70B0F9ED0A9@TYCPR01MB8373.jpnprd01.prod.outlook.com>
	<CAD21AoDgzFygdYJ9c5eSdYHP1YDkzkiEhzQvW5mrNCiPxkxKAw@mail.gmail.com>
	<CAA4eK1KfdZ4kY5J7-KM9OoWHdUW2MuTdUUgBV+S53S3h7CREkw@mail.gmail.com>
	<TYCPR01MB8373D74FC61748A668072D77ED0A9@TYCPR01MB8373.jpnprd01.prod.outlook.com>
	<CAA4eK1Kes9TsMpGL6m+AJNHYCGRvx6piYQt5v6TEbH_t9jh8nA@mail.gmail.com>
	<CAA4eK1JK2cW0gsFn==hgBR7K52Jkmb8R+cHiN+D8TbWjCSvdyw@mail.gmail.com>
	<TYCPR01MB83733807BF9349D8D0BD6DB4ED0F9@TYCPR01MB8373.jpnprd01.prod.outlook.com>

My compiler is worried that syncslotname may be used uninitialized in
start_table_sync().  The attached patch seems to silence this warning.

-- 
Nathan Bossart
Amazon Web Services: https://aws.amazon.com

Attachments:

v1-0001-silence-compiler-warning.patch (text/x-diff, 818B) download From 05e4e03af5afa1658ede8d78b31c1c999b5c7deb Mon Sep 17 00:00:00 2001 From: Nathan Bossart <nathandbossart@gmail.com> Date: Mon, 14 Mar 2022 16:00:18 -0700 Subject: [PATCH v1 1/1] silence compiler warning --- src/backend/replication/logical/worker.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/backend/replication/logical/worker.c b/src/backend/replication/logical/worker.c index a1fe81b34f..03e069c7cd 100644 --- a/src/backend/replication/logical/worker.c +++ b/src/backend/replication/logical/worker.c @@ -3387,7 +3387,7 @@ TwoPhaseTransactionGid(Oid subid, TransactionId xid, char *gid, int szgid) static void start_table_sync(XLogRecPtr *origin_startpos, char **myslotname) { - char *syncslotname; + char *syncslotname = NULL; Assert(am_tablesync_worker()); -- 2.25.1

view thread (133+ messages)  latest in thread

Message-ID: <20220314230424.GA1085716@nathanxps13>
Permalink:  ../20220314230424.GA1085716@nathanxps13/
Also on:    postgresql.org/message-id/20220314230424.GA1085716@nathanxps13

 ·  · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: nathandbossart@gmail.com, osumi.takamichi@fujitsu.com, amit.kapila16@gmail.com, sawada.mshk@gmail.com, smithpb2250@gmail.com, tanghy.fnst@fujitsu.com, wangw.fnst@fujitsu.com, vignesh21@gmail.com, gregn4422@gmail.com, mark.dilger@enterprisedb.com, peters@fast.au.fujitsu.com, pgsql-hackers@lists.postgresql.org
  Subject: Re: Optionally automatically disable logical replication subscriptions on error
  In-Reply-To: <20220314230424.GA1085716@nathanxps13>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox