check GIF list files to be the same size so not crashing
authorGood Guy <[email protected]>
Wed, 26 Jan 2022 00:01:44 +0000 (17:01 -0700)
committerGood Guy <[email protected]>
Wed, 26 Jan 2022 00:01:44 +0000 (17:01 -0700)
cinelerra-5.1/cinelerra/file.C
cinelerra-5.1/cinelerra/file.inc
cinelerra-5.1/cinelerra/filebase.h
cinelerra-5.1/cinelerra/filegif.C
cinelerra-5.1/cinelerra/filegif.h
cinelerra-5.1/cinelerra/mwindow.C

index dbcd68899a5e150fe5e79d256fe0276fbefeb1f6..03941d9a0fceb32a38067e3d8bd82420c2034c92 100644 (file)
@@ -658,6 +658,17 @@ int File::open_file(Preferences *preferences,
        }
 
 
+// If file type is a list verify that all files match in dimensions.
+// Should be done only after the file open function has been performed
+// Reason: although this function checks if file exists or not but
+// it has no way of relaying this information back and if this function
+// is called before open_file the program may accidently interpret file
+// not found as file size don't match
+ if( !file->verify_file_list() ) {
+  delete file;  file = 0;
+  return FILE_SIZE_DONT_MATCH;
+ }
+
 
 // Set extra writing parameters to mandatory settings.
        if( wr ) {
index 11a5bc9af73b1e3f2b548380c7ea35b35e940ae3..c6228a28db546fc04be0d6f309aebf2b867f6d44 100644 (file)
@@ -53,6 +53,7 @@
 #define FILE_NOT_FOUND 1
 #define FILE_UNRECOGNIZED_CODEC 2
 #define FILE_IS_XML 3
+#define FILE_SIZE_DONT_MATCH 4
 
 // Numeric codes for each file format
 // They're stored in XML files, so they have to be fixed.
index 3fb8be42087ac8195bc442357355cbaeeb2e35c7..fb10df074494699be1df0f39fbf2aff6b49b8e2c 100644 (file)
@@ -132,6 +132,10 @@ public:
                int channel,
                int64_t len);
        void allocate_history(int len);
+// thie function will be used to verify if all files in a given
+// list are of same size or not. Each list type object should 
+// override this method with its own definition.
+        virtual int verify_file_list() { return 1; }
 
 // For static functions to access it
        Asset *asset;
index bb26762bef07094251db75deeaea1b000230bcc6..b80f741206c4b47a97e9b8020c0e049eccabce23 100644 (file)
@@ -31,6 +31,7 @@
 #include <unistd.h>
 #include <fcntl.h>
 #include <string.h>
+#include <ctype.h>
 
 //from "getarg.h"
 extern "C"
@@ -500,6 +501,65 @@ FrameWriterUnit* FileGIFList::new_writer_unit(FrameWriter *writer)
        return new GIFUnit(this, writer);
 }
 
+int FileGIFList::verify_file_list()
+{
+ // go through all .gif files in the list and
+ // verify their sizes match or not.
+ //printf("\nAsset Path: %s\n", asset->path);
+ FILE *stream = fopen(asset->path, "rb");
+ if (stream) {
+  char string[BCTEXTLEN];
+  int width, height, prev_width=-1, prev_height=-1;
+  // build the path prefix
+  char prefix[BCTEXTLEN], *bp = prefix, *cp = strrchr(asset->path, '/');
+  for( int i=0, n=!cp ? 0 : cp-asset->path; i<n; ++i ) *bp++ = asset->path[i];
+  *bp = 0;
+  // read entire input file
+  while( !feof(stream) && fgets(string, BCTEXTLEN, stream) ) {
+   int len = strlen(string);
+   if(!len || string[0] == '#' || string[0] == ' ' || isalnum(string[0])) continue;
+   if( string[len-1] == '\n' ) string[len-1] = 0;
+   // a possible .gif file path? fetch it
+   char path[BCTEXTLEN], *pp = path, *ep = pp + sizeof(path)-1;
+   if( string[0] == '.' && string[1] == '/' && prefix[0] )
+    pp += snprintf(pp, ep-pp, "%s/", prefix);
+   snprintf(pp, ep-pp, "%s", string);
+   // check if a valid file exists
+   if(!access(path, R_OK)) {
+    // check file header for size
+    FILE *gif_file_temp = fopen(path, "rb");
+    if (gif_file_temp) {
+     unsigned char test[16];
+     int ret = fread(test, 16, 1, gif_file_temp);
+     fclose(gif_file_temp);
+     if( ret < 1 ) continue;
+     // get height and width of gif file
+     width = test[6] | (test[7] << 8);
+     height = test[8] | (test[9] << 8);
+     // test with previous
+     if ( (prev_width == -1) && (prev_height == -1) ) {
+      prev_width = width;
+      prev_height = height;
+      continue;
+     }
+     else if ( (prev_width != width) || (prev_height != height) ) {
+      // this is the error case we are trying to avoid
+      fclose(stream);
+      return 0;
+     }
+    }
+
+   }
+  }
+  fclose(stream);
+  return 1;
+ }
+ // not sure if our function should be the one to raise not found error
+ perror(asset->path);
+ return 0;
+}
+
+
 GIFUnit::GIFUnit(FileGIFList *file, FrameWriter *writer)
  : FrameWriterUnit(writer)
 {
index 74cae2a0581e4cb7610d522a9ab0a178fd1cb946..98370de95b16fb45344fbaf481ae749ecbf8bf3a 100644 (file)
@@ -80,6 +80,8 @@ public:
        int use_path() { return 1; }
        int read_frame(VFrame *output, char *path);
        int write_frame(VFrame *frame, VFrame *data, FrameWriterUnit *unit);
+// Need to override this method from FILEBASE class
+       int verify_file_list();
        FrameWriterUnit* new_writer_unit(FrameWriter *writer);
 };
 
index e4d02d23b7e61596d56054b8ea36343c5c0f19b1..a502c15695818972bdcbcad6b75929480ba3310d 100644 (file)
@@ -2220,6 +2220,13 @@ if(debug) printf("MWindow::load_filenames %d\n", __LINE__);
                        }
                        result = 0;
                        break; }
+// File is a list and size of listed files don't match
+  case FILE_SIZE_DONT_MATCH: {
+   eprintf(_("File sizes don't match"));
+   sprintf(string, _("File sizes don't match"));
+   gui->show_message(string, theme->message_error);
+   gui->update_default_message();
+   break; }
 
                case FILE_NOT_FOUND: {
                        eprintf(_("Failed to open %s"), new_asset->path);