be more conservative on frame dimensions
Stefan Schuermans

Stefan Schuermans commited on 2012-05-06 17:21:56
Showing 1 changed files, with 4 additions and 4 deletions.

... ...
@@ -124,15 +124,15 @@ static void AppBbmParseHeader(AppBbmState *state)
124 124
   state->width    = ntohs(hdr.width);
125 125
   state->channels = ntohs(hdr.channels);
126 126
   state->maxval   = ntohs(hdr.maxval);
127
-  if (state->height < 1 || state->height > 1000 ||
128
-      state->width < 1 || state->width > 1000 ||
127
+  if (state->height < 1 || state->height > 256 ||
128
+      state->width < 1 || state->width > 256 ||
129 129
       state->channels < 1 || state->channels > 3 ||
130 130
       state->maxval < 1 || state->maxval > 255) {
131 131
     debug_app_bbm_printf("%hhu: invalid dimensions in BBM file", state->no);
132 132
     return; // retry with next file in next iteration
133 133
   }
134 134
   state->dataSz = state->height * state->width * state->channels;
135
-  if (state->dataSz > 1000) {
135
+  if (state->dataSz > 1024) {
136 136
     debug_app_bbm_printf("%hhu: BBM frame size too large", state->no);
137 137
     return; // retry with next file in next iteration
138 138
   }
... ...
@@ -291,7 +291,7 @@ void AppBbmRun(AppBbmState *state) // (extern)
291 291
   long delta = state->nextActMs - ms;
292 292
   if (delta > 0)
293 293
     return;
294
-  // if lagging behing more than 100ms, do not try to catch up -> advance time
294
+  // if lagging behind more than 100ms, do not try to catch up -> advance time
295 295
   if (delta < -100)
296 296
     state->nextActMs = ms;
297 297
 
298 298