Guard against stopping when numberTuples=0 and counter wraps around.

diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c
index f184265c49185943be8e6ddd9d289dbccd382f9e..613a62c3c372a83421fe6c735689e87d2c9630b0 100644 (file)
--- a/src/backend/executor/execMain.c
+++ b/src/backend/executor/execMain.c
@@ -26,7 +26,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/executor/execMain.c,v 1.195 2002/12/18 00:14:47 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/executor/execMain.c,v 1.196 2003/01/08 23:32:29 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -166,7 +166,7 @@ ExecutorStart(QueryDesc *queryDesc)
  *             except to start up/shut down the destination.  Otherwise,
  *             we retrieve up to 'count' tuples in the specified direction.
  *
- *             Note: count = 0 is interpreted as no portal limit, e.g. run to
+ *             Note: count = 0 is interpreted as no portal limit, i.e., run to
  *             completion.
  *
  * ----------------------------------------------------------------
@@ -846,6 +846,7 @@ ExecEndPlan(PlanState *planstate, EState *estate)
  *
  *             processes the query plan to retrieve 'numberTuples' tuples in the
  *             direction specified.
+ *
  *             Retrieves all tuples if numberTuples is 0
  *
  *             result is either a slot containing the last tuple in the case
@@ -1091,10 +1092,10 @@ lnext:  ;
                /*
                 * check our tuple count.. if we've processed the proper number
                 * then quit, else loop again and process more tuples.  Zero
-                * number_tuples means no limit.
+                * numberTuples means no limit.
                 */
                current_tuple_count++;
-               if (numberTuples == current_tuple_count)
+               if (numberTuples && numberTuples == current_tuple_count)
                        break;
        }