mpegaudioparse: Fix compilation with gcc 7
authorSebastian Dröge <sebastian@centricular.com>
Mon, 27 Feb 2017 19:01:23 +0000 (21:01 +0200)
committerSebastian Dröge <sebastian@centricular.com>
Mon, 27 Feb 2017 19:01:23 +0000 (21:01 +0200)
gstmpegaudioparse.c: In function ‘gst_mpeg_audio_parse_reset’:
gstmpegaudioparse.c:209:3: error: ‘memset’ used with length equal to number of elements without multiplication by element size [-Werror=memset-elt-size]
   memset (mp3parse->xing_seek_table_inverse, 0, 256);
   ^~~~~~
gstmpegaudioparse.c: In function ‘gst_mpeg_audio_parse_handle_first_frame’:
gstmpegaudioparse.c:951:7: error: ‘memset’ used with length equal to number of elements without multiplication by element size [-Werror=memset-elt-size]
       memset (mp3parse->xing_seek_table_inverse, 0, 256);
       ^~~~~~

gst/audioparsers/gstmpegaudioparse.c

index 1432973..6a9d7b9 100644 (file)
@@ -205,8 +205,9 @@ gst_mpeg_audio_parse_reset (GstMpegAudioParse * mp3parse)
   mp3parse->xing_total_time = 0;
   mp3parse->xing_bytes = 0;
   mp3parse->xing_vbr_scale = 0;
-  memset (mp3parse->xing_seek_table, 0, 100);
-  memset (mp3parse->xing_seek_table_inverse, 0, 256);
+  memset (mp3parse->xing_seek_table, 0, sizeof (mp3parse->xing_seek_table));
+  memset (mp3parse->xing_seek_table_inverse, 0,
+      sizeof (mp3parse->xing_seek_table_inverse));
 
   mp3parse->vbri_bitrate = 0;
   mp3parse->vbri_frames = 0;
@@ -947,8 +948,9 @@ gst_mpeg_audio_parse_handle_first_frame (GstMpegAudioParse * mp3parse,
     skip_toc:
       data += 100;
     } else {
-      memset (mp3parse->xing_seek_table, 0, 100);
-      memset (mp3parse->xing_seek_table_inverse, 0, 256);
+      memset (mp3parse->xing_seek_table, 0, sizeof (mp3parse->xing_seek_table));
+      memset (mp3parse->xing_seek_table_inverse, 0,
+          sizeof (mp3parse->xing_seek_table_inverse));
     }
 
     if (xing_flags & XING_VBR_SCALE_FLAG) {