<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>6020</bug_id>
          
          <creation_ts>2014-03-20 14:49:08 +0000</creation_ts>
          <short_desc>should libmad be replaced with libmpg123 as the default mp3 decoder?</short_desc>
          <delta_ts>2016-06-23 14:24:31 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>7</classification_id>
          <classification>Build System, Metadata &amp; Runtime</classification>
          <product>OE-Core</product>
          <component>multimedia</component>
          <version>unspecified</version>
          <rep_platform>All</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>2.2</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Cristian Iorga">cristian.iorga</reporter>
          <assigned_to name="Alexander Kanavin">alex.kanavin</assigned_to>
          <cc>jessica.zhang</cc>
    
    <cc>jku</cc>
    
    <cc>meta.mr.watcher</cc>
    
    <cc>meta.watcher</cc>
    
    <cc>tanuk</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Don&apos;t know</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>41537</commentid>
    <comment_count>0</comment_count>
    <who name="Cristian Iorga">cristian.iorga</who>
    <bug_when>2014-03-20 14:49:08 +0000</bug_when>
    <thetext>MAD: MPEG Audio Decoder library and tools has not been updated from 2005 and should be replaced with mpg123 mp3 decoder in oe-core.
Reasons:
1. libmad has not been updated from 2005 which might be a cause for security related issues (software vulnerabilities);
2. libmad decoding performance is weak when compared with libmpg123 decoding perfomance;
3. MAD is not optimized for modern processors, while mpg123 is (Neon, SSE, AVX, etc).
4. mpg123 is still maintained/updated, which adds new features (like CPU optimizations), while also minimizing the potential for software vulnerabilities.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>41539</commentid>
    <comment_count>1</comment_count>
    <who name="Cristian Iorga">cristian.iorga</who>
    <bug_when>2014-03-20 14:53:30 +0000</bug_when>
    <thetext>References:

1. MP3 Decoder Libraries Compared
http://www.wezm.net/technical/2008/04/mp3-decoder-libraries-compared/
2. mpg123 - Fast console MPEG Audio Player and decoder library
http://mpg123.de/

mpg123 is ISO MPEG compliant, as attested by the tests run by the developer on each release.

mpg123 is already present in meta-openembedded/meta-multimedia layer.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>57499</commentid>
    <comment_count>2</comment_count>
    <who name="Tanu Kaskinen">tanuk</who>
    <bug_when>2015-12-31 13:09:47 +0000</bug_when>
    <thetext>What does &quot;the default mp3 decoder&quot; mean? Each application typically supports only one or the other, so we can&apos;t make a default choice on behalf of the applications. Is this just about the availability of libmpg123, that is, the goal here is to make it easier for applications to use libmpg123 by including it in oe-core?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60229</commentid>
    <comment_count>3</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2016-03-21 22:51:16 +0000</bug_when>
    <thetext>As far as oe-core is concerned, the only place libmad is used is in gstreamer1.0-plugins-ugly.  Need to see what the state of the libmpg123 plugin for GStreamer is.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60230</commentid>
    <comment_count>4</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2016-03-21 22:54:11 +0000</bug_when>
    <thetext>It&apos;s part of -ugly just as libmad.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61043</commentid>
    <comment_count>5</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2016-04-11 20:55:30 +0000</bug_when>
    <thetext>Reassigning to Alex as he&apos;s been doing multimedia work.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>63281</commentid>
    <comment_count>6</comment_count>
    <who name="Alexander Kanavin">alex.kanavin</who>
    <bug_when>2016-06-23 14:24:31 +0000</bug_when>
    <thetext>Patch is in master: d8afd0c9f2fc302aa6c3e88e452806d51b23e3cd</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>