From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Wed, 17 Feb 2010 01:48:58 +0000 (+0000)
Subject: Prevent #option dump from crashing on FORI statement with null step.  Reported by... 
X-Git-Tag: REL8_3_10~29
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=192cd7f390b5593cf597e328aed1dcb99776b38b;p=postgresql

Prevent #option dump from crashing on FORI statement with null step.  Reported by Pavel.
---

diff --git a/src/pl/plpgsql/src/pl_funcs.c b/src/pl/plpgsql/src/pl_funcs.c
index dd2841e10e..1f1f1560e4 100644
--- a/src/pl/plpgsql/src/pl_funcs.c
+++ b/src/pl/plpgsql/src/pl_funcs.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/pl/plpgsql/src/pl_funcs.c,v 1.67 2008/01/01 19:46:00 momjian Exp $
+ *	  $PostgreSQL: pgsql/src/pl/plpgsql/src/pl_funcs.c,v 1.67.2.1 2010/02/17 01:48:58 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -749,10 +749,13 @@ dump_fori(PLpgSQL_stmt_fori *stmt)
 	printf("    upper = ");
 	dump_expr(stmt->upper);
 	printf("\n");
-	dump_ind();
-	printf("    step = ");
-	dump_expr(stmt->step);
-	printf("\n");
+	if (stmt->step)
+	{
+		dump_ind();
+		printf("    step = ");
+		dump_expr(stmt->step);
+		printf("\n");
+	}
 	dump_indent -= 2;
 
 	dump_stmts(stmt->body);