about summary refs log tree commit diff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>1998-09-18 18:51:34 +0000
committerUlrich Drepper <drepper@redhat.com>1998-09-18 18:51:34 +0000
commitc53a89d434f518f99d5168e9c8693ac63777042a (patch)
tree65f013cb4974e2ed4fb7144ff474a320fa95aac5
parent00bc5db059212a20afb42da40b38d7f145a46dfd (diff)
downloadglibc-c53a89d434f518f99d5168e9c8693ac63777042a.tar.gz
glibc-c53a89d434f518f99d5168e9c8693ac63777042a.tar.xz
glibc-c53a89d434f518f99d5168e9c8693ac63777042a.zip
Update.
	* stdio-common/vfscanf.c: Fix reading (nil) for %p.
-rw-r--r--ChangeLog2
-rw-r--r--stdio-common/vfscanf.c35
2 files changed, 32 insertions, 5 deletions
diff --git a/ChangeLog b/ChangeLog
index 7299d958f1..10b7215b99 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -25,6 +25,8 @@
 
 	* time/strftime.c [_LIBC] (ampm): Use tp->tm_hour not hour12.
 
+	* stdio-common/vfscanf.c: Fix reading (nil) for %p.
+
 1998-09-18  Mark Kettenis  <kettenis@phys.uva.nl>
 
 	* login/programs/pt_chown.c (more_help): Correct message that
diff --git a/stdio-common/vfscanf.c b/stdio-common/vfscanf.c
index f9bbb6929e..ec4a264d65 100644
--- a/stdio-common/vfscanf.c
+++ b/stdio-common/vfscanf.c
@@ -216,6 +216,8 @@ __vfscanf (FILE *s, const char *format, va_list argptr)
      possibly be matched even if in the input stream no character is
      available anymore.  */
   int skip_space = 0;
+  /* Nonzero if we are reading a pointer.  */
+  int read_pointer;
   /* Workspace.  */
   char *tw;			/* Temporary pointer.  */
   char *wp = NULL;		/* Workspace.  */
@@ -354,6 +356,9 @@ __vfscanf (FILE *s, const char *format, va_list argptr)
       /* This is the start of the conversion string. */
       flags = 0;
 
+      /* Not yet decided whether we read a pointer or not.  */
+      read_pointer = 0;
+
       /* Initialize state of modifiers.  */
       argpos = 0;
 
@@ -871,13 +876,32 @@ __vfscanf (FILE *s, const char *format, va_list argptr)
 	      c = inchar ();
 	    }
 
-	  /* The just read character is not part of the number anymore.  */
-	  ungetc (c, s);
-
 	  if (wpsize == 0 ||
 	      (wpsize == 1 && (wp[0] == '+' || wp[0] == '-')))
-	    /* There was no number.  */
-	    conv_error ();
+	    {
+	      /* There was no number.  If we are supposed to read a pointer
+		 we must recognize "(nil)" as well.  */
+	      if (wpsize == 0 && read_pointer && (width < 0 || width >= 0)
+		  && c == '('
+		  && tolower (inchar ()) == 'n'
+		  && tolower (inchar ()) == 'i'
+		  && tolower (inchar ()) == 'l'
+		  && inchar () == ')')
+		/* We must produce the value of a NULL pointer.  A single
+		   '0' digit is enough.  */
+		ADDW ('0');
+	      else
+		{
+		  /* The last read character is not part of the number
+		     anymore.  */
+		  ungetc (c, s);
+
+		  conv_error ();
+		}
+	    }
+	  else
+	    /* The just read character is not part of the number anymore.  */
+	    ungetc (c, s);
 
 	  /* Convert the number.  */
 	  ADDW ('\0');
@@ -1221,6 +1245,7 @@ __vfscanf (FILE *s, const char *format, va_list argptr)
 	  flags &= ~(SHORT|LONGDBL);
 	  flags |= LONG;
 	  number_signed = 0;
+	  read_pointer = 1;
 	  goto number;
 	}
     }