[Groff-commit] groff ChangeLog src/preproc/pic/main.cpp

[Werner LEMBERG]

CVSROOT:        /cvsroot/groff
Module name:    groff
Changes by:     Werner LEMBERG <wl>     12/02/08 18:54:31

Modified files:
        .              : ChangeLog 
        src/preproc/pic: main.cpp 

Log message:
        * src/preproc/pic/main.cpp (do_file): Check for invalid characters.
        
        Problem reported by Doug McIlroy <address@hidden>.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/groff/ChangeLog?cvsroot=groff&r1=1.1323&r2=1.1324
http://cvs.savannah.gnu.org/viewcvs/groff/src/preproc/pic/main.cpp?cvsroot=groff&r1=1.7&r2=1.8

Patches:
Index: ChangeLog
===================================================================
RCS file: /cvsroot/groff/groff/ChangeLog,v
retrieving revision 1.1323
retrieving revision 1.1324
diff -u -b -r1.1323 -r1.1324
--- ChangeLog   25 Jan 2012 07:12:29 -0000      1.1323
+++ ChangeLog   8 Feb 2012 18:54:31 -0000       1.1324
@@ -1,3 +1,9 @@
+2012-02-08  Werner LEMBERG  <address@hidden>
+
+       * src/preproc/pic/main.cpp (do_file): Check for invalid characters.
+
+       Problem reported by Doug McIlroy <address@hidden>.
+
 2012-01-25  Ingo Schwarze  <address@hidden>
 
        [mdoc] * tmac/doc-syms: Fix meaning of XBD acronym.

Index: src/preproc/pic/main.cpp
===================================================================
RCS file: /cvsroot/groff/groff/src/preproc/pic/main.cpp,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -b -r1.7 -r1.8
--- src/preproc/pic/main.cpp    5 Jan 2009 20:11:08 -0000       1.7
+++ src/preproc/pic/main.cpp    8 Feb 2012 18:54:31 -0000       1.8
@@ -1,5 +1,5 @@
 // -*- C++ -*-
-/* Copyright (C) 1989-1992, 2000, 2001, 2002, 2003, 2006, 2009
+/* Copyright (C) 1989-1992, 2000, 2001, 2002, 2003, 2006, 2009, 2012
    Free Software Foundation, Inc.
      Written by James Clark (address@hidden)
 
@@ -316,6 +316,10 @@
   enum { START, MIDDLE, HAD_DOT, HAD_P, HAD_PS, HAD_l, HAD_lf } state = START;
   for (;;) {
     int c = getc(fp);
+    while (invalid_input_char(c)) {
+      error("invalid input character code %1", int(c));
+      c = getc(fp);
+    }
     if (c == EOF)
       break;
     switch (state) {