Bug 11788 - easymp3gain gives error message "List index (-1) out of bounds Press OK to ignore and risk data corruption"
Summary: easymp3gain gives error message "List index (-1) out of bounds Press OK to ig...
Status: RESOLVED WONTFIX
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Florian Hubold
QA Contact:
URL:
Whiteboard:
Keywords: Triaged
Depends on:
Blocks:
 
Reported: 2013-11-27 05:36 CET by Curtis Hildebrand
Modified: 2018-02-07 18:15 CET (History)
2 users (show)

See Also:
Source RPM: easymp3gain-0.5.0-5.mga4.src.rpm
CVE:
Status comment:


Attachments

Description Curtis Hildebrand 2013-11-27 05:36:17 CET
Everytime I add files to EasyMp3Gain I receive this message.  Usually the program goes grey, but is still responsive and the error message pops up after a few clicks.  When I run the mp3gain routine it works as expected.

Error message is:
"List index (-1) out of bounds 
Press OK to ignore and risk data corruption
Press Cancel to kill the program


Steps to Reproduce:
1.  Add files => program goes grey
2.  Click the Analyze icon => files are analyzed as expected.
3.  Click Apply Track Gain => error message pops up yet mp3gain is applied as exptected.


Reproducible: 

Steps to Reproduce:
Manuel Hiebel 2013-11-27 10:02:23 CET

Keywords: (none) => Triaged
Assignee: bugsquad => mageia

Comment 1 Damien Lallement 2013-12-24 02:39:30 CET
I can't reproduce. Can you give more information please?
Comment 2 Curtis Hildebrand 2014-02-08 06:22:02 CET
Hmmm.  I tried with a new user under GNOME and I got the same result.  I'll keep digging.
Comment 3 Francisco Barcenas 2015-05-08 00:41:33 CEST
I have the same problem..

CC: (none) => barcenas.francisco

Comment 4 Rémi Verschelde 2015-05-08 00:58:46 CEST
Assigning to new maintainer.

Assignee: mageia => doktor5000
Summary: Gives error message "List index (-1) out of bounds Press OK to ignore and risk data corruption" => easymp3gain gives error message "List index (-1) out of bounds Press OK to ignore and risk data corruption"

Comment 5 Francisco Barcenas 2015-05-08 01:19:18 CEST
It seems that it has to do something with the file name.  All the ones that those are occurring with either have a _ or a / or # sign in the file or directory name.

There should be a recommendation to NORMALIZE the file/directory names with..???

What could/should you use?
Comment 6 Florian Hubold 2015-05-11 17:05:58 CEST
Actually I don't use it anymore due to this, got this pretty often when using it to apply gain to the files I keep on my mp3 player. This is what I use now to increase each file to 99dB:

find /some/path/with/mp3/files -type f -iname "*.mp3" -print0 | xargs -0 mp3gain -d 10 -c -T


There are two possibilities for this bug:

- it can be workarounded by opening options => advanced => "number of sublevels when adding files" and increasing this value to something that fits your setup.
- it's a typical off-by-one error in the code, and somebody needs to provide a patch. easymp3gain itself has not been maintained since 2010, so I'd prefer dropping it as mp3gain itself is a little more flexible and you don't need to fiddle with the GUI. 

And I can't provide a patch for the code.


(In reply to Francisco Barcenas from comment #5)
> It seems that it has to do something with the file name.  All the ones that
> those are occurring with either have a _ or a / or # sign in the file or
> directory name.
> 
> There should be a recommendation to NORMALIZE the file/directory names
> with..???
> 
> What could/should you use?

You certainly cannot have / as part of a filename.
For renaming, krename is pretty good, rename itself as CLI-only is also good, there's also pyrenamer or gprename, although the two latter ones are not packaged for Mageia yet.
In any case, that would only be an ugly workaround.

CC: (none) => doktor5000
Hardware: x86_64 => All

Comment 7 Florian Hubold 2018-02-07 18:15:21 CET
Closing this one as there's no fix available.

Status: NEW => RESOLVED
Resolution: (none) => WONTFIX


Note You need to log in before you can comment on or make changes to this bug.