file_write: Fix NULL pointer dereference.
[paraslash.git] / mp3_afh.c
index 7dda549..980649c 100644 (file)
--- a/mp3_afh.c
+++ b/mp3_afh.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2003-2008 Andre Noll <maan@systemlinux.org>
+ * Copyright (C) 2003-2010 Andre Noll <maan@systemlinux.org>
  *
  * Licensed under the GPL v2. For licencing details see COPYING.
  */
  *                      Johannes Overmann <overmann@iname.com>
  */
 
+#include <regex.h>
+
 #include "para.h"
 #include "error.h"
 #include "afh.h"
 #include "string.h"
-#include "afs.h"
-#include "server.h"
 
 /** \cond some defines and structs which are only used in this file */
 
@@ -71,6 +71,157 @@ static const int mp3info_bitrate[2][3][14] = {
 static const int frame_size_index[] = {24000, 72000, 72000};
 static const char *mode_text[] = {"stereo", "joint stereo", "dual channel", "mono", "invalid"};
 
+#ifdef HAVE_LIBID3TAG
+
+#include <id3tag.h>
+
+static char *get_latin1(id3_ucs4_t const *string)
+{
+       if (!string)
+               return NULL;
+       return (char *)id3_ucs4_latin1duplicate(string);
+}
+
+static char *get_stringlist(union id3_field *field)
+{
+       unsigned int k, nstrings = id3_field_getnstrings(field);
+       char *result = NULL;
+
+       for (k = 0; k < nstrings; k++) {
+               char *tmp = (char *)get_latin1(id3_field_getstrings(field, k));
+               if (result) {
+                       char *tmp2 = result;
+                       result = make_message("%s %s", tmp2, tmp);
+                       free(tmp);
+                       free(tmp2);
+               } else
+                       result = tmp;
+       }
+       return result;
+}
+
+static char *get_string(union id3_field *field)
+{
+       id3_ucs4_t const *string = id3_field_getfullstring(field);
+
+       return get_latin1(string);
+}
+
+#define FOR_EACH_FIELD(f, j, fr) for (j = 0; j < (fr)->nfields && \
+       (f = id3_frame_field((fr), j)); j++)
+
+static char *get_strings(struct id3_frame *fr)
+{
+       int j;
+       union id3_field *field;
+
+       FOR_EACH_FIELD(field, j, fr) {
+               enum id3_field_type type = id3_field_type(field);
+
+               if (type == ID3_FIELD_TYPE_STRINGLIST)
+                       return get_stringlist(field);
+               if (type == ID3_FIELD_TYPE_STRINGFULL)
+                       return get_string(field);
+       }
+       return NULL;
+}
+
+static void mp3_get_id3(__a_unused unsigned char *map,
+               __a_unused size_t numbytes, int fd, struct taginfo *tags)
+{
+       int i;
+       struct id3_tag *id3_t;
+       struct id3_file *id3_f = id3_file_fdopen(fd, ID3_FILE_MODE_READONLY);
+
+       if (!id3_f)
+               return;
+       id3_t = id3_file_tag(id3_f);
+       if (!id3_t) {
+               id3_file_close(id3_f);
+               return;
+       }
+       for (i = 0; i < id3_t->nframes; i++) {
+               struct id3_frame *fr = id3_t->frames[i];
+               if (!strcmp(fr->id, "TIT2")) {
+                       if (!tags->title)
+                               tags->title = get_strings(fr);
+                       continue;
+               }
+               if (!strcmp(fr->id, "TPE1")) {
+                       if (!tags->artist)
+                               tags->artist = get_strings(fr);
+                       continue;
+               }
+               if (!strcmp(fr->id, "TALB")) {
+                       if (!tags->album)
+                               tags->album = get_strings(fr);
+                       continue;
+               }
+               if (!strcmp(fr->id, "TDRC")) {
+                       if (!tags->year)
+                               tags->year = get_strings(fr);
+                       continue;
+               }
+               if (!strcmp(fr->id, "COMM")) {
+                       if (!tags->comment)
+                               tags->comment = get_strings(fr);
+                       continue;
+               }
+       }
+       id3_file_close(id3_f);
+}
+
+#else /* HAVE_LIBID3TAG */
+
+/*
+ * Remove trailing whitespace from the end of a string
+ */
+static char *unpad(char *string)
+{
+       char *pos = string + strlen(string) - 1;
+       while (para_isspace(pos[0]))
+               (pos--)[0] = 0;
+       return string;
+}
+
+static void mp3_get_id3(unsigned char *map, size_t numbytes, __a_unused int fd,
+       struct taginfo *tags)
+{
+       char title[31], artist[31], album[31], year[5], comment[31];
+       off_t fpos;
+
+       if (numbytes < 128 || strncmp("TAG", (char *)map + numbytes - 128, 3)) {
+               PARA_DEBUG_LOG("no id3 v1 tag\n");
+               return;
+       }
+       fpos = numbytes - 125;
+       memcpy(title, map + fpos, 30);
+       fpos += 30;
+       title[30] = '\0';
+       memcpy(artist, map + fpos, 30);
+       fpos += 30;
+       artist[30] = '\0';
+       memcpy(album, map + fpos, 30);
+       fpos += 30;
+       album[30] = '\0';
+       memcpy(year, map + fpos, 4);
+       fpos += 4;
+       year[4] = '\0';
+       memcpy(comment, map + fpos, 30);
+       comment[30] = '\0';
+       unpad(title);
+       unpad(artist);
+       unpad(album);
+       unpad(year);
+       unpad(comment);
+       tags->artist = para_strdup(artist);
+       tags->title = para_strdup(title);
+       tags->year = para_strdup(year);
+       tags->album = para_strdup(album);
+       tags->comment = para_strdup(comment);
+}
+#endif /* HAVE_LIBID3TAG */
+
 static int header_frequency(struct mp3header *h)
 {
        if (h->version > 2 || h->freq > 3)
@@ -117,20 +268,9 @@ static int frame_length(struct mp3header *header)
                + header->padding;
 }
 
-/*
- * Remove trailing whitespace from the end of a string
- */
-static char *unpad(char *string)
-{
-       char *pos = string + strlen(string) - 1;
-       while (para_isspace(pos[0]))
-               (pos--)[0] = 0;
-       return string;
-}
-
 static int compare_headers(struct mp3header *h1,struct mp3header *h2)
 {
-       if ((*(uint*)h1) == (*(uint*)h2))
+       if ((*(unsigned int*)h1) == (*(unsigned int*)h2))
                return 1;
        if ((h1->version == h2->version) &&
                        (h1->layer == h2->layer) &&
@@ -222,49 +362,6 @@ static int mp3_seek_next_header(unsigned char *map, size_t numbytes, off_t *fpos
        return 0;
 }
 
-static char *mp3_get_id3(unsigned char *map, size_t numbytes)
-{
-       char title[31], artist[31], album[31], year[5], comment[31];
-       off_t fpos;
-
-       if (numbytes < 128 || strncmp("TAG", (char *)map + numbytes - 128, 3)) {
-               PARA_DEBUG_LOG("no id3 v1 tag\n");
-               return make_message("%s: (no id3 v1 tag)\n%s:\n",
-                       status_item_list[SI_TAGINFO1],
-                       status_item_list[SI_TAGINFO2]);
-       }
-       fpos = numbytes - 125;
-       memcpy(title, map + fpos, 30);
-       fpos += 30;
-       title[30] = '\0';
-       memcpy(artist, map + fpos, 30);
-       fpos += 30;
-       artist[30] = '\0';
-       memcpy(album, map + fpos, 30);
-       fpos += 30;
-       album[30] = '\0';
-       memcpy(year, map + fpos, 4);
-       fpos += 4;
-       year[4] = '\0';
-       memcpy(comment, map + fpos, 30);
-       comment[30] = '\0';
-       unpad(title);
-       unpad(artist);
-       unpad(album);
-       unpad(year);
-       unpad(comment);
-       return make_message("%s: %s, by %s\n" /* taginfo1 */
-               "%s: A: %s, Y: %s, C: %s\n", /* taginfo 2*/
-               status_item_list[SI_TAGINFO1],
-               *title? title : "(title tag not set)",
-               *artist? artist : "(artist tag not set)",
-               status_item_list[SI_TAGINFO2],
-               *album?  album : "(album tag not set)",
-               *year? year : "????",
-               *comment? comment : "(comment tag not set)"
-       );
-}
-
 static int find_valid_start(unsigned char *map, size_t numbytes, off_t *fpos,
        struct mp3header *header)
 {
@@ -284,27 +381,33 @@ static int find_valid_start(unsigned char *map, size_t numbytes, off_t *fpos,
        return frame_len;
 }
 
-static int mp3_read_info(unsigned char *map, size_t numbytes,
+static int mp3_read_info(unsigned char *map, size_t numbytes, int fd,
                struct afh_info *afhi)
 {
-       long freq_avg = 0, br_avg = 0;
-       int ret, len = 0, old_br = -1, vbr = 0;
+       uint64_t freq_sum = 0, br_sum = 0;
+       int fl = 0, ret, len = 0, old_br = -1, vbr = 0;
        struct timeval total_time = {0, 0};
        unsigned chunk_table_size = 1000; /* gets increased on demand */
        off_t fpos = 0;
        struct mp3header header;
-       char *taginfo;
 
        afhi->chunks_total = 0;
-       afhi->chunk_table = para_malloc(chunk_table_size * sizeof(size_t));
-       taginfo = mp3_get_id3(map, numbytes);
+       afhi->chunk_table = para_malloc(chunk_table_size * sizeof(uint32_t));
        while (1) {
-               unsigned long freq, br, fl;
+               int freq, br;
                struct timeval tmp, cct; /* current chunk time */
                fpos += len;
                len = find_valid_start(map, numbytes, &fpos, &header);
-               if (len <= 0)
+               if (len <= 0) {
+                       size_t end;
+                       ret = -E_MP3_INFO;
+                       if (!afhi->chunks_total)
+                               goto err_out;
+                       end = afhi->chunk_table[afhi->chunks_total - 1] + fl;
+                       afhi->chunk_table[afhi->chunks_total]
+                               = PARA_MIN(end, numbytes);
                        break;
+               }
                ret = header_frequency(&header);
                if (ret < 0)
                        continue;
@@ -322,49 +425,34 @@ static int mp3_read_info(unsigned char *map, size_t numbytes,
                tv_divide(br * 125, &tmp, &cct);
                tv_add(&cct, &total_time, &tmp);
                total_time = tmp;
-               //PARA_DEBUG_LOG("%s: br: %d, freq: %d, fl: %d, cct: %lu\n", __func__, br, freq, fl, cct.tv_usec);
                if (afhi->chunks_total >= chunk_table_size) {
                        chunk_table_size *= 2;
                        afhi->chunk_table = para_realloc(afhi->chunk_table,
-                               chunk_table_size * sizeof(size_t));
+                               chunk_table_size * sizeof(uint32_t));
                }
                afhi->chunk_table[afhi->chunks_total] = fpos;
-//             if (afhi->chunks_total < 10 || !(afhi->chunks_total % 1000))
-//                     PARA_INFO_LOG("chunk #%lu: %zd\n", afhi->chunks_total,
-//                             afhi->chunk_table[afhi->chunks_total]);
                afhi->chunks_total++;
-               if (afhi->chunks_total == 1) {
-                       freq_avg = freq;
-                       br_avg = br;
-                       old_br = br;
-                       continue;
-               }
-               freq_avg += ((long)freq - freq_avg) / ((long)afhi->chunks_total + 1);
-               br_avg += ((long)br - br_avg) / ((long)afhi->chunks_total + 1);
-               if (old_br != br)
+               freq_sum += freq;
+               br_sum += br;
+               if (afhi->chunks_total != 1 && old_br != br)
                        vbr = 1;
                old_br = br;
        }
        ret = -E_MP3_INFO;
-       if (!afhi->chunks_total || !freq_avg || !br_avg)
+       if (!freq_sum || !br_sum)
                goto err_out;
-       afhi->chunk_table[afhi->chunks_total] = numbytes - 1;
-       afhi->bitrate = br_avg;
-       afhi->frequency = freq_avg;
+       afhi->bitrate = br_sum / afhi->chunks_total;
+       afhi->frequency = freq_sum / afhi->chunks_total;
        afhi->channels = header_channels(&header);
        afhi->seconds_total = (tv2ms(&total_time) + 500) / 1000;
        tv_divide(afhi->chunks_total, &total_time, &afhi->chunk_tv);
        PARA_DEBUG_LOG("%lu chunks, each %lums\n", afhi->chunks_total,
                tv2ms(&afhi->chunk_tv));
-       tv_scale(3, &afhi->chunk_tv, &afhi->eof_tv);
-       PARA_DEBUG_LOG("eof timeout: %lu\n", tv2ms(&afhi->eof_tv));
-       afhi->info_string = make_message("%s: %cbr, %s\n%s",
-               status_item_list[SI_AUDIO_FILE_INFO], vbr? 'v' : 'c',
-               header_mode(&header), taginfo);
-       free(taginfo);
+       afhi->techinfo = make_message("%cbr, %s", vbr? 'v' : 'c',
+               header_mode(&header));
+       mp3_get_id3(map, numbytes, fd, &afhi->tags);
        return 1;
 err_out:
-       free(taginfo);
        PARA_ERROR_LOG("%s\n", para_strerror(-ret));
        free(afhi->chunk_table);
        return ret;
@@ -373,12 +461,12 @@ err_out:
 /*
  * Read mp3 information from audio file
  */
-int mp3_get_file_info(char *map, size_t numbytes,
+static int mp3_get_file_info(char *map, size_t numbytes, int fd,
                struct afh_info *afhi)
 {
        int ret;
 
-       ret = mp3_read_info((unsigned char *)map, numbytes, afhi);
+       ret = mp3_read_info((unsigned char *)map, numbytes, fd, afhi);
        if (ret < 0)
                return ret;
        if (afhi->seconds_total < 2 || !afhi->chunks_total)