Mercurial > cpython-lru_cache-weakref

diff Python/getargs.c @ 12652:866365b44f23 legacy-trunk

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Trent Mick <trentm@activestate.com>: Limit the 'b' formatter of PyArg_ParseTuple to valid values of an unsigned char, i.e. [0,UCHAR_MAX]. It is expected that this is the common usage of 'b'. An OverflowError is raised if the parsed value is outside this range.
author Fred Drake <fdrake@acm.org>
date Tue, 09 May 2000 21:50:00 +0000
parents 5648df74395f
children 68e57c5d1bec
line wrap: on
line diff
--- a/Python/getargs.c	Tue May 09 19:58:19 2000 +0000
+++ b/Python/getargs.c	Tue May 09 21:50:00 2000 +0000
@@ -465,28 +465,28 @@
 	
 	switch (c) {
 	
-	case 'b': /* byte -- very short int */
+	case 'b': /* unsigned byte -- very short int */
 		{
 			char *p = va_arg(*p_va, char *);
 			long ival = PyInt_AsLong(arg);
 			if (ival == -1 && PyErr_Occurred())
 				return "integer<b>";
-			else if (ival < CHAR_MIN) {
+			else if (ival < 0) {
 				PyErr_SetString(PyExc_OverflowError,
-					"byte integer is less than minimum");
+					"unsigned byte integer is less than minimum");
 				return "integer<b>";
 			}
-			else if (ival > CHAR_MAX && ival >= 256) {
+			else if (ival > UCHAR_MAX) {
 				PyErr_SetString(PyExc_OverflowError,
-				    "byte integer is greater than maximum");
+				    "unsigned byte integer is greater than maximum");
 				return "integer<b>";
 			}
 			else
-				*p = (char) ival;
+				*p = (unsigned char) ival;
 			break;
 		}
 	
-	case 'h': /* short int */
+	case 'h': /* signed short int */
 		{
 			short *p = va_arg(*p_va, short *);
 			long ival = PyInt_AsLong(arg);
@@ -494,12 +494,12 @@
 				return "integer<h>";
 			else if (ival < SHRT_MIN) {
 				PyErr_SetString(PyExc_OverflowError,
-					"short integer is less than minimum");
+					"signed short integer is less than minimum");
 				return "integer<h>";
 			}
 			else if (ival > SHRT_MAX) {
 				PyErr_SetString(PyExc_OverflowError,
-				  "short integer is greater than maximum");
+				  "signed short integer is greater than maximum");
 				return "integer<h>";
 			}
 			else
@@ -507,7 +507,7 @@
 			break;
 		}
 	
-	case 'i': /* int */
+	case 'i': /* signed int */
 		{
 			int *p = va_arg(*p_va, int *);
 			long ival = PyInt_AsLong(arg);
@@ -515,12 +515,12 @@
 				return "integer<i>";
 			else if (ival < INT_MIN) {
 				PyErr_SetString(PyExc_OverflowError,
-					"integer is less than minimum");
+					"signed integer is less than minimum");
 				return "integer<i>";
 			}
 			else if (ival > INT_MAX) {
 				PyErr_SetString(PyExc_OverflowError,
-				  "integer is greater than maximum");
+				  "signed integer is greater than maximum");
 				return "integer<i>";
 			}
 			else