Error out on send failure in walsender loop.

hlinnaka · hlinnaka · commit 38587d7d286f · 2014-03-04T15:43:27.000+02:00
I changed the loop in 9.3 to use "goto send_failure" instead of "break" on
errors, but I missed this one case. It was a relatively harmless bug: if
the flush fails once it will most likely fail again as soon as we try to
flush the output again. But it's a bug nevertheless.

Report and fix by Andres Freund.
diff --git a/src/backend/replication/walsender.c b/src/backend/replication/walsender.c
@@ -1100,7 +1100,7 @@ WalSndLoop(void)
 					ping_sent = true;
 					/* Try to flush pending output to the client */
 					if (pq_flush_if_writable() != 0)
-						break;
+						goto send_failure;
 				}
 			}
 

Original file line number	Diff line number	Diff line change
`@@ -1100,7 +1100,7 @@ WalSndLoop(void)`
`1100`	`1100`	`ping_sent = true;`
`1101`	`1101`	`/* Try to flush pending output to the client */`
`1102`	`1102`	`if (pq_flush_if_writable() != 0)`
`1103`		`- break;`
	`1103`	`+ goto send_failure;`
`1104`	`1104`	`}`
`1105`	`1105`	`}`
`1106`	`1106`