ViewVC Help
View File | Revision Log | Show Annotations | Download File | Root Listing
root/radiance/ray/src/common/wordfile.c
(Generate patch)

Comparing ray/src/common/wordfile.c (file contents):
Revision 2.16 by greg, Mon Mar 21 19:06:08 2016 UTC vs.
Revision 2.20 by greg, Wed Apr 27 17:26:45 2016 UTC

# Line 3 | Line 3 | static const char      RCSid[] = "$Id$";
3   #endif
4   /*
5   * Load whitespace separated words from a file into an array.
6 * Assume the passed pointer array is big enough to hold them all.
6   *
7 < * External symbols declared in standard.h
7 > * External symbols declared in rtio.h
8   */
9  
10   #include "copyright.h"
11  
12   #include <ctype.h>
13   #include <string.h>
15 #include <stdio.h>
16 #include <sys/types.h>
17 #include <sys/stat.h>
18 #include <fcntl.h>
14  
15   #include "platform.h"
16 < #include "standard.h"
16 > #include "rtio.h"
17 > #include "rtmisc.h"
18  
19  
20 < #ifndef MAXFLEN
25 < #define MAXFLEN         204800  /* file must be smaller than this */
26 < #endif
20 > #define MAXWLEN         4096    /* words must be shorter than this */
21  
22  
23   int
# Line 33 | Line 27 | wordfile(                      /* get words from fname, put in words */
27          char    *fname
28   )
29   {
30 +        int     wrdcnt = 0;
31 +        int     n = 0;
32          int     fd;
33 <        char    buf[MAXFLEN];
38 <        int     n;
33 >        char    buf[MAXWLEN];
34                                          /* load file into buffer */
35 <        if (fname == NULL)
35 >        if (fname == NULL || !*fname)
36                  return(-1);                     /* no filename */
37 +        if (nargs <= 1)
38 +                return(-1);
39          if ((fd = open(fname, 0)) < 0)
40                  return(-1);                     /* open error */
41 <        n = read(fd, buf, MAXFLEN);
41 >        words[0] = NULL;
42 >        while (nargs > 1 && (n += read(fd, buf+n, MAXWLEN-n)) > 0) {
43 >                int     crem = 0;
44 >                if (n > MAXWLEN/2)              /* check for mid-word end */
45 >                        while (!isspace(buf[--n])) {
46 >                                if (n <= 0)     /* one long word! */
47 >                                        goto done;
48 >                                ++crem;
49 >                        }
50 >                buf[n] = '\0';                  /* terminate & parse */
51 >                n = wordstring(words, nargs, buf);
52 >                if (n < 0) {
53 >                        wrdcnt = -1;            /* memory error */
54 >                        break;
55 >                }
56 >                words += n; nargs -= n;
57 >                wrdcnt += n;
58 >                if ((n = crem) > 0)             /* move remainder */
59 >                        memmove(buf, buf+MAXWLEN-crem, crem);
60 >        }
61 > done:
62          close(fd);
63 <        if (n < 0)                              /* read error */
47 <                return(-1);
48 <        if (n == MAXFLEN)               /* file too big, take what we can */
49 <                while (!isspace(buf[--n]))
50 <                        if (n <= 0)             /* one long word! */
51 <                                return(-1);
52 <        buf[n] = '\0';                          /* terminate */
53 <        return(wordstring(words, nargs, buf));  /* wordstring does the rest */
63 >        return(wrdcnt);
64   }
65  
66  
# Line 79 | Line 89 | wordstring(                            /* allocate and load argument list */
89                  while (*++cp && !isspace(*cp))
90                          ;
91          }
92 +        *cp = '\0';                     /* terminates overflow */
93          *ap = NULL;
94          return(ap - avl);
95   }

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines