d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixed handling of input failure by the scanf family.

Browse Source 
- 0 was returned instead of EOF when an input failure occured while
  skipping white-space after 0 assignments.  This fixes PR2606.  The
  diagnosis in PR2606 is wrong.
- EOF was returned instead of 0 when an input failure occurred after
  zero assignments and nonzero suppressed assignments.
- EOF was spelled -1.

This should be in 2.2.
This commit is contained in:
Bruce Evans 1997-03-03 17:53:02 +00:00
parent 60e0f7e23d
commit e836e480dc
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=23352
1 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
lib/libc/stdio/vfscanf.c
View File

@ -39,7 +39,7 @@
static char sccsid[] = "@(#)vfscanf.c 8.1 (Berkeley) 6/4/93";
#endif
static const char rcsid[] =
"$Id$";
"$Id: vfscanf.c,v 1.7 1997/02/22 15:02:41 peter Exp $";
#endif /* LIBC_SCCS and not lint */
#include <stdio.h>
@ -115,6 +115,7 @@ __svfscanf(fp, fmt0, ap)
register int flags; /* flags as defined above */
register char *p0; /* saves original value of p when necessary */
int nassigned; /* number of fields assigned */
int nconversions; /* number of conversions */
int nread; /* number of characters consumed from fp */
int base; /* base argument to strtol/strtoul */
u_long (*ccfn)(); /* conversion function (strtol/strtoul) */
@ -126,6 +127,7 @@ __svfscanf(fp, fmt0, ap)
{ 10, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16 };
nassigned = 0;
nconversions = 0;
nread = 0;
base = 0; /* XXX just to keep gcc happy */
ccfn = NULL; /* XXX just to keep gcc happy */
@ -136,7 +138,7 @@ __svfscanf(fp, fmt0, ap)
if (isspace(c)) {
for (;;) {
if (fp->_r <= 0 && __srefill(fp))
return (nassigned);
goto input_failure;
if (!isspace(*fp->_p))
break;
nread++, fp->_r--, fp->_p++;
@ -261,6 +263,7 @@ again: c = *fmt++;
break;
case 'n':
nconversions++;
if (flags & SUPPRESS) /* ??? */
continue;
if (flags & SHORT)
@ -349,6 +352,7 @@ again: c = *fmt++;
nread += r;
nassigned++;
}
nconversions++;
break;
case CT_CCL:
@ -390,6 +394,7 @@ again: c = *fmt++;
nassigned++;
}
nread += n;
nconversions++;
break;
case CT_STRING:
@ -420,6 +425,7 @@ again: c = *fmt++;
nread += p - p0;
nassigned++;
}
nconversions++;
continue;
case CT_INT:
@ -556,6 +562,7 @@ again: c = *fmt++;
nassigned++;
}
nread += p - buf;
nconversions++;
break;
#ifdef FLOATING_POINT
@ -647,12 +654,13 @@ again: c = *fmt++;
nassigned++;
}
nread += p - buf;
nconversions++;
break;
#endif /* FLOATING_POINT */
}
}
input_failure:
return (nassigned ? nassigned : -1);
return (nconversions != 0 ? nassigned : EOF);
match_failure:
return (nassigned);
}