Loading...
stdio/FreeBSD/vfscanf.c.patch Libc-583 Libc-391.4.2
--- Libc/Libc-583/stdio/FreeBSD/vfscanf.c.patch
+++ Libc/Libc-391.4.2/stdio/FreeBSD/vfscanf.c.patch
@@ -1,6 +1,6 @@
---- vfscanf.c.orig	2009-02-15 03:11:22.000000000 -0800
-+++ vfscanf.c	2009-02-16 00:22:06.000000000 -0800
-@@ -40,6 +40,8 @@ static char sccsid[] = "@(#)vfscanf.c	8.
+--- vfscanf.c.orig	2004-11-25 11:38:35.000000000 -0800
++++ vfscanf.c	2005-02-23 19:24:50.000000000 -0800
+@@ -40,6 +40,8 @@
  #include <sys/cdefs.h>
  __FBSDID("$FreeBSD: src/lib/libc/stdio/vfscanf.c,v 1.37 2004/05/02 10:55:05 das Exp $");
  
@@ -9,15 +9,7 @@
  #include "namespace.h"
  #include <ctype.h>
  #include <inttypes.h>
-@@ -50,6 +52,7 @@ __FBSDID("$FreeBSD: src/lib/libc/stdio/v
- #include <string.h>
- #include <wchar.h>
- #include <wctype.h>
-+#include <pthread.h>
- #include "un-namespace.h"
- 
- #include "collate.h"
-@@ -97,10 +100,21 @@ __FBSDID("$FreeBSD: src/lib/libc/stdio/v
+@@ -97,10 +99,21 @@
  #define	CT_INT		3	/* %[dioupxX] conversion */
  #define	CT_FLOAT	4	/* %[efgEFG] conversion */
  
@@ -41,17 +33,16 @@
  
  __weak_reference(__vfscanf, vfscanf);
  
-@@ -108,12 +122,24 @@ __weak_reference(__vfscanf, vfscanf);
+@@ -108,12 +121,24 @@
   * __vfscanf - MT-safe version
   */
  int
 -__vfscanf(FILE *fp, char const *fmt0, va_list ap)
 +__vfscanf(FILE * __restrict fp, char const * __restrict fmt0, va_list ap)
- {
- 	int ret;
- 
- 	FLOCKFILE(fp);
--	ret = __svfscanf(fp, fmt0, ap);
++{
++	int ret;
++
++	FLOCKFILE(fp);
 +	ret = __svfscanf_l(fp, __current_locale(), fmt0, ap);
 +	FUNLOCKFILE(fp);
 +	return (ret);
@@ -59,16 +50,17 @@
 +
 +int
 +vfscanf_l(FILE * __restrict fp, locale_t loc, char const * __restrict fmt0, va_list ap)
-+{
-+	int ret;
-+
+ {
+ 	int ret;
+ 
 +	NORMALIZE_LOCALE(loc);
-+	FLOCKFILE(fp);
+ 	FLOCKFILE(fp);
+-	ret = __svfscanf(fp, fmt0, ap);
 +	ret = __svfscanf_l(fp, loc, fmt0, ap);
  	FUNLOCKFILE(fp);
  	return (ret);
  }
-@@ -121,8 +147,8 @@ __vfscanf(FILE *fp, char const *fmt0, va
+@@ -121,8 +146,8 @@
  /*
   * __svfscanf - non-MT-safe version of __vfscanf
   */
@@ -79,7 +71,7 @@
  {
  	const u_char *fmt = (const u_char *)fmt0;
  	int c;			/* character from format, or conversion */
-@@ -132,7 +158,6 @@ __svfscanf(FILE *fp, const char *fmt0, v
+@@ -132,7 +157,6 @@
  	int flags;		/* flags as defined above */
  	char *p0;		/* saves original value of p when necessary */
  	int nassigned;		/* number of fields assigned */
@@ -87,7 +79,7 @@
  	int nread;		/* number of characters consumed from fp */
  	int base;		/* base argument to conversion function */
  	char ccltab[256];	/* character class table for %[...] */
-@@ -140,29 +165,37 @@ __svfscanf(FILE *fp, const char *fmt0, v
+@@ -140,24 +164,29 @@
  	wchar_t *wcp;		/* handy wide character pointer */
  	wchar_t *wcp0;		/* saves original value of wcp */
  	size_t nconv;		/* length of multibyte sequence converted */
@@ -120,35 +112,7 @@
  				nread++, fp->_r--, fp->_p++;
  			continue;
  		}
--		if (c != '%')
-+		if (c != '%') {
-+			if (fp->_r <= 0 && __srefill(fp))
-+				goto input_failure;
- 			goto literal;
-+		}
- 		width = 0;
- 		flags = 0;
- 		/*
-@@ -172,15 +205,35 @@ __svfscanf(FILE *fp, const char *fmt0, v
- again:		c = *fmt++;
- 		switch (c) {
- 		case '%':
-+			/* Consume leading white space */
-+			for(;;) {
-+				if (fp->_r <= 0 && __srefill(fp))
-+					goto input_failure;
-+				if (!isspace_l(*fp->_p, loc))
-+					break;
-+				nread++;
-+				fp->_r--;
-+				fp->_p++;
-+			}
- literal:
--			if (fp->_r <= 0 && __srefill(fp))
--				goto input_failure;
- 			if (*fp->_p != c)
- 				goto match_failure;
- 			fp->_r--, fp->_p++;
+@@ -181,6 +210,18 @@
  			nread++;
  			continue;
  
@@ -167,7 +131,7 @@
  		case '*':
  			flags |= SUPPRESS;
  			goto again;
-@@ -267,7 +320,7 @@ literal:
+@@ -267,7 +308,7 @@
  			break;
  
  		case '[':
@@ -176,7 +140,7 @@
  			flags |= NOSKIP;
  			c = CT_CCL;
  			break;
-@@ -288,7 +341,6 @@ literal:
+@@ -288,7 +329,6 @@
  			break;
  
  		case 'n':
@@ -184,7 +148,7 @@
  			if (flags & SUPPRESS)	/* ??? */
  				continue;
  			if (flags & SHORTSHORT)
-@@ -330,7 +382,7 @@ literal:
+@@ -330,7 +370,7 @@
  		 * that suppress this.
  		 */
  		if ((flags & NOSKIP) == 0) {
@@ -193,7 +157,7 @@
  				nread++;
  				if (--fp->_r > 0)
  					fp->_p++;
-@@ -360,7 +412,7 @@ literal:
+@@ -360,7 +400,7 @@
  					wcp = NULL;
  				n = 0;
  				while (width != 0) {
@@ -202,7 +166,7 @@
  						fp->_flags |= __SERR;
  						goto input_failure;
  					}
-@@ -368,7 +420,7 @@ literal:
+@@ -368,7 +408,7 @@
  					fp->_p++;
  					fp->_r--;
  					mbs = initial;
@@ -211,7 +175,7 @@
  					if (nconv == (size_t)-1) {
  						fp->_flags |= __SERR;
  						goto input_failure;
-@@ -421,7 +473,6 @@ literal:
+@@ -421,7 +461,6 @@
  				nread += r;
  				nassigned++;
  			}
@@ -219,7 +183,7 @@
  			break;
  
  		case CT_CCL:
-@@ -440,7 +491,7 @@ literal:
+@@ -440,7 +479,7 @@
  				n = 0;
  				nchars = 0;
  				while (width != 0) {
@@ -228,7 +192,7 @@
  						fp->_flags |= __SERR;
  						goto input_failure;
  					}
-@@ -448,7 +499,7 @@ literal:
+@@ -448,7 +487,7 @@
  					fp->_p++;
  					fp->_r--;
  					mbs = initial;
@@ -237,7 +201,7 @@
  					if (nconv == (size_t)-1) {
  						fp->_flags |= __SERR;
  						goto input_failure;
-@@ -456,8 +507,8 @@ literal:
+@@ -456,8 +495,8 @@
  					if (nconv == 0)
  						*wcp = L'\0';
  					if (nconv != (size_t)-2) {
@@ -248,7 +212,7 @@
  							while (n != 0) {
  								n--;
  								__ungetc(buf[n],
-@@ -525,7 +576,6 @@ literal:
+@@ -525,7 +564,6 @@
  				nassigned++;
  			}
  			nread += n;
@@ -256,18 +220,18 @@
  			break;
  
  		case CT_STRING:
-@@ -540,8 +590,8 @@ literal:
+@@ -540,8 +578,8 @@
  				else
  					wcp = &twc;
  				n = 0;
 -				while (!isspace(*fp->_p) && width != 0) {
 -					if (n == MB_CUR_MAX) {
-+				while (width != 0) {
++				while (!isspace_l(*fp->_p, loc) && width != 0) {
 +					if (n == mb_cur_max) {
  						fp->_flags |= __SERR;
  						goto input_failure;
  					}
-@@ -549,7 +599,7 @@ literal:
+@@ -549,7 +587,7 @@
  					fp->_p++;
  					fp->_r--;
  					mbs = initial;
@@ -276,7 +240,7 @@
  					if (nconv == (size_t)-1) {
  						fp->_flags |= __SERR;
  						goto input_failure;
-@@ -557,7 +607,7 @@ literal:
+@@ -557,7 +595,7 @@
  					if (nconv == 0)
  						*wcp = L'\0';
  					if (nconv != (size_t)-2) {
@@ -285,7 +249,7 @@
  							while (n != 0) {
  								n--;
  								__ungetc(buf[n],
-@@ -585,7 +635,7 @@ literal:
+@@ -585,7 +623,7 @@
  				}
  			} else if (flags & SUPPRESS) {
  				n = 0;
@@ -294,7 +258,7 @@
  					n++, fp->_r--, fp->_p++;
  					if (--width == 0)
  						break;
-@@ -595,7 +645,7 @@ literal:
+@@ -595,7 +633,7 @@
  				nread += n;
  			} else {
  				p0 = p = va_arg(ap, char *);
@@ -303,7 +267,7 @@
  					fp->_r--;
  					*p++ = *fp->_p++;
  					if (--width == 0)
-@@ -607,7 +657,6 @@ literal:
+@@ -607,7 +645,6 @@
  				nread += p - p0;
  				nassigned++;
  			}
@@ -311,7 +275,7 @@
  			continue;
  
  		case CT_INT:
-@@ -738,9 +787,9 @@ literal:
+@@ -738,9 +775,9 @@
  
  				*p = 0;
  				if ((flags & UNSIGNED) == 0)
@@ -323,7 +287,7 @@
  				if (flags & POINTER)
  					*va_arg(ap, void **) =
  							(void *)(uintptr_t)res;
-@@ -763,43 +812,48 @@ literal:
+@@ -763,43 +800,52 @@
  				nassigned++;
  			}
  			nread += p - buf;
@@ -338,8 +302,11 @@
 -			if (width == 0 || width > sizeof(buf) - 1)
 -				width = sizeof(buf) - 1;
 -			if ((width = parsefloat(fp, buf, buf + width)) == 0)
-+			if ((width = parsefloat(fp, &pbuf, width, loc)) == 0)
++			if ((width = parsefloat(fp, &pbuf, width, loc)) == 0) {
++				if (pbuf)
++					free(pbuf);
  				goto match_failure;
++			}
  			if ((flags & SUPPRESS) == 0) {
  				if (flags & LONGDBL) {
 -					long double res = strtold(buf, &p);
@@ -355,13 +322,13 @@
  					*va_arg(ap, float *) = res;
  				}
 -				if (__scanfdebug && p - buf != width)
--					abort();
 +				if (__scanfdebug && p - pbuf != width)
-+					LIBC_ABORT("p - pbuf %ld != width %ld", (long)(p - pbuf), width);
+ 					abort();
  				nassigned++;
  			}
  			nread += width;
 -			nconversions++;
++			free(pbuf);
  			break;
 +		}
  #endif /* !NO_FLOATING_POINT */
@@ -383,7 +350,7 @@
  /*
   * Fill in the given table from the scanset at the given format
   * (just after `[').  Return a pointer to the character past the
-@@ -807,9 +861,10 @@ match_failure:
+@@ -807,9 +853,10 @@
   * considered part of the scanset.
   */
  static const u_char *
@@ -395,7 +362,7 @@
  {
  	int c, n, v, i;
  
-@@ -845,6 +900,7 @@ doswitch:
+@@ -845,6 +892,7 @@
  			return (fmt - 1);
  
  		case '-':
@@ -403,7 +370,7 @@
  			/*
  			 * A scanset of the form
  			 *	[01+-]
-@@ -865,8 +921,8 @@ doswitch:
+@@ -865,8 +913,8 @@
  			 */
  			n = *fmt;
  			if (n == ']'
@@ -414,7 +381,7 @@
  			       )
  			   ) {
  				c = '-';
-@@ -874,14 +930,14 @@ doswitch:
+@@ -874,14 +922,14 @@
  			}
  			fmt++;
  			/* fill in the range */
@@ -432,7 +399,7 @@
  					   )
  						tab[i] = v;
  			}
-@@ -901,7 +957,7 @@ doswitch:
+@@ -901,7 +949,7 @@
  				return (fmt);
  #endif
  			break;
@@ -441,79 +408,28 @@
  		case ']':		/* end of scanset */
  			return (fmt);
  
-@@ -914,19 +970,75 @@ doswitch:
- }
+@@ -915,7 +963,7 @@
  
  #ifndef NO_FLOATING_POINT
-+/*
-+ * Maintain a per-thread parsefloat buffer, shared by __svfscanf_l and
-+ * __vfwscanf.
-+ */
-+#ifdef BUILDING_VARIANT
-+extern char *__parsefloat_buf(size_t s);
-+#else /* !BUILDING_VARIANT */
-+__private_extern__ char *
-+__parsefloat_buf(size_t s)
-+{
-+	char *b;
-+	static pthread_key_t    parsefloat_tsd_key = (pthread_key_t)-1;
-+	static pthread_mutex_t  parsefloat_tsd_lock = PTHREAD_MUTEX_INITIALIZER;
-+	static size_t bsiz = 0;
-+
-+	if (parsefloat_tsd_key == (pthread_key_t)-1) {
-+		pthread_mutex_lock(&parsefloat_tsd_lock);
-+		if (parsefloat_tsd_key == (pthread_key_t)-1) {
-+			parsefloat_tsd_key = __LIBC_PTHREAD_KEY_PARSEFLOAT;
-+			pthread_key_init_np(parsefloat_tsd_key, free);
-+		}
-+		pthread_mutex_unlock(&parsefloat_tsd_lock);
-+	}
-+	if ((b = (char *)pthread_getspecific(parsefloat_tsd_key)) == NULL) {
-+		bsiz = s > BUF ? s : BUF;
-+		b = (char *)malloc(bsiz);
-+		if (b == NULL) {
-+			bsiz = 0;
-+			return NULL;
-+		}
-+		pthread_setspecific(parsefloat_tsd_key, b);
-+		return b;
-+	}
-+	if (s > bsiz) {
-+		b = (char *)reallocf(b, s);
-+		pthread_setspecific(parsefloat_tsd_key, b);
-+		if (b == NULL) {
-+			bsiz = 0;
-+			return NULL;
-+		}
-+		bsiz = s;
-+	}
-+	return b;
-+}
-+#endif /* BUILDING_VARIANT */
-+
  static int
 -parsefloat(FILE *fp, char *buf, char *end)
 +parsefloat(FILE *fp, char **buf, size_t width, locale_t loc)
  {
  	char *commit, *p;
  	int infnanpos = 0;
- 	enum {
- 		S_START, S_GOTSIGN, S_INF, S_NAN, S_MAYBEHEX,
--		S_DIGITS, S_FRAC, S_EXP, S_EXPDIGITS
-+		S_DIGITS, S_FRAC, S_EXP, S_EXPDIGITS, S_DECIMAL_POINT
+@@ -924,9 +972,18 @@
+ 		S_DIGITS, S_FRAC, S_EXP, S_EXPDIGITS
  	} state = S_START;
  	unsigned char c;
 -	char decpt = *localeconv()->decimal_point;
-+	unsigned char *decpt = (unsigned char *)localeconv_l(loc)->decimal_point;
-+	char *decpt_start;
++	char decpt = *localeconv_l(loc)->decimal_point;
  	_Bool gotmantdig = 0, ishex = 0;
--
-+	char *b;
-+	char *e;
++	char *b, *e;
 +	size_t s;
-+
-+	s = (width == 0 ? BUF : (width + 1));
-+	if ((b = __parsefloat_buf(s)) == NULL) {
+ 
++	s = (width == 0 ? BUF : width + 1);
++	b = (char *)malloc(s);
++	if (b == NULL) {
 +		*buf = NULL;
 +		return 0;
 +	}
@@ -521,7 +437,7 @@
  	/*
  	 * We set commit = p whenever the string we have read so far
  	 * constitutes a valid representation of a floating point
-@@ -936,8 +1048,8 @@ parsefloat(FILE *fp, char *buf, char *en
+@@ -936,8 +993,8 @@
  	 * always necessary to read at least one character that doesn't
  	 * match; thus, we can't short-circuit "infinity" or "nan(...)".
  	 */
@@ -532,7 +448,7 @@
  		c = *fp->_p;
  reswitch:
  		switch (state) {
-@@ -997,7 +1109,7 @@ reswitch:
+@@ -997,7 +1054,7 @@
  				if (c == ')') {
  					commit = p;
  					infnanpos = -2;
@@ -541,7 +457,7 @@
  					goto parsedone;
  				break;
  			}
-@@ -1013,16 +1125,33 @@ reswitch:
+@@ -1013,7 +1070,7 @@
  				goto reswitch;
  			}
  		case S_DIGITS:
@@ -549,37 +465,8 @@
 +			if ((ishex && isxdigit_l(c, loc)) || isdigit_l(c, loc))
  				gotmantdig = 1;
  			else {
--				state = S_FRAC;
--				if (c != decpt)
--					goto reswitch;
-+				state = S_DECIMAL_POINT;
-+				decpt_start = p;
-+				goto reswitch;
- 			}
- 			if (gotmantdig)
- 				commit = p;
- 			break;
-+		case S_DECIMAL_POINT:
-+			if (*decpt == 0) {
-+				if (gotmantdig)
-+					commit = p - 1;
-+				state = S_FRAC;
-+				goto reswitch;
-+			}
-+			if (*decpt++ == c)
-+				break;
-+			/* not decimal point */
-+			state = S_FRAC;
-+			if (decpt_start == p)
-+				goto reswitch;
-+			while (decpt_start < --p)
-+				__ungetc(*(u_char *)p, fp);
-+			c = *(u_char *)p;
-+			goto reswitch;
- 		case S_FRAC:
- 			if (((c == 'E' || c == 'e') && !ishex) ||
- 			    ((c == 'P' || c == 'p') && ishex)) {
-@@ -1030,7 +1159,7 @@ reswitch:
+ 				state = S_FRAC;
+@@ -1030,7 +1087,7 @@
  					goto parsedone;
  				else
  					state = S_EXP;
@@ -588,7 +475,7 @@
  				commit = p;
  				gotmantdig = 1;
  			} else
-@@ -1043,13 +1172,26 @@ reswitch:
+@@ -1043,7 +1100,7 @@
  			else
  				goto reswitch;
  		case S_EXPDIGITS:
@@ -597,27 +484,25 @@
  				commit = p;
  			else
  				goto parsedone;
- 			break;
+@@ -1051,6 +1108,17 @@
  		default:
--			abort();
-+			LIBC_ABORT("unknown state %d", state);
-+		}
+ 			abort();
+ 		}
 +		if (p >= e) {
-+			ssize_t diff = (p - b);
-+			ssize_t com = (commit - b);
++			size_t diff = (p - b);
 +			s += BUF;
-+			b = __parsefloat_buf(s);
++			b = (char *)reallocf(b, s);
 +			if (b == NULL) {
 +				*buf = NULL;
 +				return 0;
 +			}
 +			e = b + (s - 1);
 +			p = b + diff;
-+			commit = b + com;
- 		}
++		}
  		*p++ = c;
  		if (--fp->_r > 0)
-@@ -1062,6 +1204,7 @@ parsedone:
+ 			fp->_p++;
+@@ -1062,6 +1130,7 @@
  	while (commit < --p)
  		__ungetc(*(u_char *)p, fp);
  	*++commit = '\0';