comparison open.cpp @ 596:6ad0a6e67d4c

Take advantage of those new handy _or_goto_error macros

Use them in various places where previously we either elided the error
checking (various lseek() calls) or used a combination of calls
(replaced by malloc_and_fill_or_goto_error()).

In the process, fix what is probably a bug (or else introduce one, but I
don't think so): audiodb_track_id_datum() computed the offset into the
timesTable wrongly, forgetting to multiply by 2.  (TODO: this should be
easily testable using the API).

Now all of LIBOBJS can be produced by my (Debian's) mingw32
cross-compiler, except for lshlib.o.

   size_t key_table_length = 0;
 
   if(adb->header->length > 0) {
     unsigned nfiles = adb->header->numFiles;
     key_table_length = align_page_up(nfiles * ADB_FILETABLE_ENTRY_SIZE);
-    key_table = (char *)malloc(key_table_length);
-    if(!key_table) {
-      goto error;
-    }
-    lseek_set_or_goto_error(adb->fd, adb->header->fileTableOffset);
-    read_or_goto_error(adb->fd, key_table, key_table_length);
+    malloc_and_fill_or_goto_error(char *, key_table, adb->header->fileTableOffset, key_table_length);
     for (unsigned int k = 0; k < nfiles; k++) {
       adb->keys->push_back(key_table + k*ADB_FILETABLE_ENTRY_SIZE);
       (*adb->keymap)[(key_table + k*ADB_FILETABLE_ENTRY_SIZE)] = k;
     }
     free(key_table);
   }
 
   return 0;
 
  error:
-  if(key_table) {
-    free(key_table);
-  }
+  maybe_free(key_table);
   return 1;
 }
 
 static int audiodb_collect_track_lengths(adb_t *adb) {
   uint32_t *track_table = 0;
   size_t track_table_length = 0;
   if(adb->header->length > 0) {
     unsigned nfiles = adb->header->numFiles;
     track_table_length = align_page_up(nfiles * ADB_TRACKTABLE_ENTRY_SIZE);
-    track_table = (uint32_t *) malloc(track_table_length);
-    if (!track_table) {
-      goto error;
-    }
-    lseek_set_or_goto_error(adb->fd, adb->header->trackTableOffset);
-    read_or_goto_error(adb->fd, track_table, track_table_length);
+    malloc_and_fill_or_goto_error(uint32_t *, track_table, adb->header->trackTableOffset, track_table_length);
     off_t offset = 0;
     for (unsigned int k = 0; k < nfiles; k++) {
       uint32_t track_length = track_table[k];
       adb->track_lengths->push_back(track_length);
       adb->track_offsets->push_back(offset);

   }
 
   return 0;
 
  error:
-  if(track_table) {
-    free(track_table);
-  }
+  maybe_free(track_table);
   return 1;
 }
 
 adb_t *audiodb_open(const char *path, int flags) {
   adb_t *adb = 0;

   adb->cached_lsh = 0;
   return adb;
 
  error:
   if(adb) {
-    if(adb->header) {
-      free(adb->header);
-    }
-    if(adb->path) {
-      free(adb->path);
-    }
+    maybe_free(adb->header);
+    maybe_free(adb->path);
     if(adb->keys) {
       delete adb->keys;
     }
     if(adb->keymap) {
       delete adb->keymap;