ProcessIncomingNotify *must* reset notifyInterruptOccurred when called.

tglsfdc · tglsfdc · commit a959c67cb73c · 2010-09-23T17:16:51.000-04:00
This was broken in 9.0 by careless addition of an early-exit path.
Bug report and diagnosis by Jeff Davis.
diff --git a/src/backend/commands/async.c b/src/backend/commands/async.c
@@ -2090,7 +2090,10 @@ ProcessIncomingNotify(void)
 {
 	bool		catchup_enabled;
 
-	/* Do nothing if we aren't actively listening */
+	/* We *must* reset the flag */
+	notifyInterruptOccurred = 0;
+
+	/* Do nothing else if we aren't actively listening */
 	if (listenChannels == NIL)
 		return;
 
@@ -2102,8 +2105,6 @@ ProcessIncomingNotify(void)
 
 	set_ps_display("notify interrupt", false);
 
-	notifyInterruptOccurred = 0;
-
 	/*
 	 * We must run asyncQueueReadAllNotifications inside a transaction, else
 	 * bad things happen if it gets an error.
