From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Sat, 30 Dec 2000 19:17:47 +0000 (+0000)
Subject: Be more careful about the difference between signed and unsigned ints.
X-Git-Tag: REL7_1_BETA2~56
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=4723b2b99bc2d4b52a7c8a047485707aeb643895;p=postgresql

Be more careful about the difference between signed and unsigned ints.
Bug is revealed by OID regress test on 64-bit platforms.
---

diff --git a/src/backend/port/snprintf.c b/src/backend/port/snprintf.c
index dc85c3f681..6cf45ca992 100644
--- a/src/backend/port/snprintf.c
+++ b/src/backend/port/snprintf.c
@@ -74,7 +74,7 @@ typedef unsigned long ulong_long;
  * causing nast effects.
  **************************************************************/
 
-/*static char _id[] = "$Id: snprintf.c,v 1.28 2000/12/03 20:45:34 tgl Exp $";*/
+/*static char _id[] = "$Id: snprintf.c,v 1.29 2000/12/30 19:17:47 tgl Exp $";*/
 static char *end;
 static int	SnprfOverflow;
 
@@ -192,12 +192,12 @@ dopr(char *buffer, const char *format, va_list args)
 						if (longflag)
 						{
 							if (longlongflag)
-								value = va_arg(args, long_long);
+								value = va_arg(args, ulong_long);
 							else
-								value = va_arg(args, long);
+								value = va_arg(args, unsigned long);
 						}
 						else
-							value = va_arg(args, int);
+							value = va_arg(args, unsigned int);
 						fmtnum(value, 10, 0, ljust, len, zpad);
 						break;
 					case 'o':
@@ -206,12 +206,12 @@ dopr(char *buffer, const char *format, va_list args)
 						if (longflag)
 						{
 							if (longlongflag)
-								value = va_arg(args, long_long);
+								value = va_arg(args, ulong_long);
 							else
-								value = va_arg(args, long);
+								value = va_arg(args, unsigned long);
 						}
 						else
-							value = va_arg(args, int);
+							value = va_arg(args, unsigned int);
 						fmtnum(value, 8, 0, ljust, len, zpad);
 						break;
 					case 'd':
@@ -225,31 +225,30 @@ dopr(char *buffer, const char *format, va_list args)
 						}
 						else
 							value = va_arg(args, int);
-
 						fmtnum(value, 10, 1, ljust, len, zpad);
 						break;
 					case 'x':
 						if (longflag)
 						{
 							if (longlongflag)
-								value = va_arg(args, long_long);
+								value = va_arg(args, ulong_long);
 							else
-								value = va_arg(args, long);
+								value = va_arg(args, unsigned long);
 						}
 						else
-							value = va_arg(args, int);
+							value = va_arg(args, unsigned int);
 						fmtnum(value, 16, 0, ljust, len, zpad);
 						break;
 					case 'X':
 						if (longflag)
 						{
 							if (longlongflag)
-								value = va_arg(args, long_long);
+								value = va_arg(args, ulong_long);
 							else
-								value = va_arg(args, long);
+								value = va_arg(args, unsigned long);
 						}
 						else
-							value = va_arg(args, int);
+							value = va_arg(args, unsigned int);
 						fmtnum(value, -16, 0, ljust, len, zpad);
 						break;
 					case 's':