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

Comparing ray/src/common/readoct.c (file contents):
Revision 1.13 by greg, Mon Aug 5 08:58:52 1991 UTC vs.
Revision 2.3 by greg, Thu Mar 12 12:18:18 1992 UTC

# Line 18 | Line 18 | static char SCCSid[] = "$SunId$ LBL";
18  
19   #include  "otypes.h"
20  
21 – extern double  atof();
21   static double  getflt();
22   static long  getint();
23   static char  *getstr();
24 + static int  getobj(), octerror();
25   static OCTREE  getfullnode(), gettree();
26  
27   static char  *infn;                     /* input file name */
28   static FILE  *infp;                     /* input file stream */
29 + static int  objsize;                    /* size of stored OBJECT's */
30   static OBJECT  objorig;                 /* zeroeth object */
31   static short  otypmap[NUMOTYPE+8];      /* object type map */
32  
# Line 42 | Line 43 | char  *ofn[];
43          int  nf;
44          OBJECT  fnobjects;
45          register int  i;
46 +        long  m;
47          
48          if (fname == NULL) {
49                  infn = "standard input";
# Line 58 | Line 60 | char  *ofn[];
60          if (checkheader(infp, OCTFMT, load&IO_INFO ? stdout : NULL) < 0)
61                  octerror(USER, "not an octree");
62                                          /* check format */
63 <        if (getint(2) != OCTMAGIC)
63 >        if ((objsize = getint(2)-OCTMAGIC) <= 0 ||
64 >                        objsize > MAXOBJSIZ || objsize > sizeof(long))
65                  octerror(USER, "incompatible octree format");
66                                          /* get boundaries */
67          if (load & IO_BOUNDS) {
# Line 81 | Line 84 | char  *ofn[];
84          if (load & IO_FILES)
85                  ofn[nf] = NULL;
86                                          /* get number of objects */
87 <        fnobjects = getint(sizeof(OBJECT));
87 >        fnobjects = m = getint(objsize);
88 >        if (fnobjects != m)
89 >                octerror(USER, "too many objects");
90  
91          if (load & IO_TREE) {
92                                                  /* get the octree */
93                  scene->cutree = gettree();
94 <                                                /* get the scene */
95 <                if (nf == 0 && load & IO_SCENE) {
94 >                if (load & IO_SCENE)            /* get the scene */
95 >                    if (nf == 0) {
96                          for (i = 0; *getstr(sbuf); i++)
97                                  if ((otypmap[i] = otype(sbuf)) < 0) {
98                                          sprintf(errmsg, "unknown type \"%s\"",
# Line 96 | Line 101 | char  *ofn[];
101                                  }
102                          while (getobj() != OVOID)
103                                  ;
104 <                } else if (load & IO_SCENE) {           /* consistency checks */
105 <                                                /* check object count */
104 >                    } else {                    /* consistency checks */
105 >                                        /* check object count */
106                          if (nobjects != objorig+fnobjects)
107                                  octerror(USER, "bad object count; octree stale?");
108 <                                                /* check for non-surfaces */
108 >                                        /* check for non-surfaces */
109                          if (nonsurfinset(objorig, fnobjects))
110 <                                octerror(USER, "non-surface in set; octree stale?");
111 <                }
110 >                                octerror(USER, "modifier in tree; octree stale?");
111 >                    }
112          }
113          fclose(infp);
114          return(nf);
# Line 131 | Line 136 | getfullnode()                  /* get a set, return fullnode */
136   {
137          OBJECT  set[MAXSET+1];
138          register int  i;
139 +        register long  m;
140  
141 <        set[0] = getint(sizeof(OBJECT));
136 <        if (set[0] > MAXSET)
141 >        if ((set[0] = getint(objsize)) > MAXSET)
142                  octerror(USER, "bad set in getfullnode");
143 <        for (i = 1; i <= set[0]; i++)
144 <                set[i] = getint(sizeof(OBJECT)) + objorig;
143 >        for (i = 1; i <= set[0]; i++) {
144 >                m = getint(objsize) + objorig;
145 >                if ((set[i] = m) != m)
146 >                        octerror(USER, "too many objects");
147 >        }
148          return(fullnode(set));
149   }      
150  
# Line 205 | Line 213 | getobj()                               /* get next object */
213          char  sbuf[MAXSTR];
214          int  obj;
215          register int  i;
216 +        register long  m;
217          register OBJREC  *objp;
218  
219          i = getint(1);
# Line 215 | Line 224 | getobj()                               /* get next object */
224          objp = objptr(obj);
225          if ((objp->otype = otypmap[i]) < 0)
226                  octerror(USER, "reference to unknown type");
227 <        if ((objp->omod = getint(sizeof(OBJECT))) != OVOID)
228 <                objp->omod += objorig;
227 >        if ((m = getint(objsize)) != OVOID) {
228 >                m += objorig;
229 >                if ((OBJECT)m != m)
230 >                        octerror(USER, "too many objects");
231 >        }
232 >        objp->omod = m;
233          objp->oname = savqstr(getstr(sbuf));
234          if (objp->oargs.nsargs = getint(2)) {
235                  objp->oargs.sarg = (char **)bmalloc
# Line 239 | Line 252 | getobj()                               /* get next object */
252                  objp->oargs.iarg = NULL;
253   #endif
254          if (objp->oargs.nfargs = getint(2)) {
255 <                objp->oargs.farg = (double *)bmalloc
256 <                                (objp->oargs.nfargs*sizeof(double));
255 >                objp->oargs.farg = (FLOAT *)bmalloc
256 >                                (objp->oargs.nfargs*sizeof(FLOAT));
257                  if (objp->oargs.farg == NULL)
258                          goto memerr;
259                  for (i = 0; i < objp->oargs.nfargs; i++)

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines