search

lordmulder / LameXP

Audio Encoder Front-End
http://lordmulder.github.io/LameXP
Other
186 stars 18 forks source link

MediaInfo is crashing after upgrading to 4.16 RC3+ #65

Closed przemas2468 closed 6 years ago

przemas2468 commented 6 years ago

When I add a sound file *.flac with a cover, it pops up a window with the interruption of the MediaInfo process. In the <RC2 version there was no problem, but after the most recent update to RC4 it still exists. I attach screenshots and log to this error.

Screenshot 1 Screenshot 2 Log.txt

lordmulder commented 6 years ago

Hello.

Does it happen with any audio files? Or only with FLAC? Or only with this specific FLAC file?

You can try with my MediaInfo builds directly from the command-line:

mediainfo.exe --Language=raw --Full --Output=XML --Cover_Data=base64 <audio_file>

If this only happens with a specific file, can you please provide an example file?

Regards MuldeR

lordmulder commented 6 years ago

See here please: https://github.com/MediaArea/MediaInfo/issues/227

przemas2468 commented 6 years ago

I tested with different codecs (MP3, AAC, OPUS, Ogg, WAV and APE) and only FLAC crashes, from various sources of music that I have. I tried your MediaInfo build, which you provided and is similar error to what you reported in the repository of this program. But unfortunately not everything, because it turns out that LameXP has a problem with recognizing whether there is a cover and only MP3 and WAV correctly read this property. (I've attached next log to review)

Log 2.txt

lordmulder commented 6 years ago

As I wrote in the other issue (see link above), I was able to reproduce the FLAC crash here.

Also happens with the "official" MediaInfo binary.

So this seems to be an issue in latest MediaInfo. The developer of MediaInfo is investigating now.

Let's wait and see what happens...

lordmulder commented 6 years ago

Bug has been fixed in MediaInfo. Will update to latest MediaInfo soon...

lordmulder commented 6 years ago

Updated MediaInfo to the fixed version. Please try with the new LameXP build: LameXP-RC5.2018-04-19.Release-Static.Build-2126.exe

przemas2468 commented 6 years ago

OK, it looks like this problem has been solved. Very thanks for the releasing fix. :wink: