DVBV recording properties explorer plugin v.2.0 - Unicode bug

[majstang]

Bug-description:

Plugin fails to give correct analyse of Video compression in explorer properties on certain files. Instead of Video comression: MPEG-2 or H.264 it simply gives a null result.

 

Leads:

- DVBV recording properties v.1.0 did not fail once regarding the problem.

- No signal receiving problems (DVB-C).

- No errors in affected recordings (according to extended logging file and TS analyser).

- If using Mediainfos own GUI on the affected files the Video compression analyse is correct.

- Bug only strucks on files with special characters in filename.

- 1 out of 20 files with special characters in filename have the bug.

- If editing the file in for example VideoReDo the explorer plugin manages to correctly analyse the saved result.

- Explorer plugin is a third party GUI and uses mediainfo.dll. Mediainfodeveloper Jerome Martinez says:

 

Usually, wrong report from third party GUI is an error from MediaInfo (wrong analysis), so opening a bug ticket is normal, no problem.

 

Conclusions:

- Something in the Unicode implemenation in the DVBV recording properties v.2.0 explorer plugin is clearly wrong, triggering the bug when calling mediainfo API. The bug could also be in the mediainfo.dll itself as Jerome Martinez suggests. In any case the bug must be small since it does not affect all files with special characters and I suspect therefor quite hard to detect which "other" circumstances (API calls) triggers the bug.

 

To Christian or Griga:

- If you guys have any intentions to fix this two year old bug, please contact me and I will provide several sample files for you. Might be hard to solve this bug without them. Unfortunately they are quite large (can't shorten them down since the error then disappears). They vary between 2.3 - 5.3 GB.

EDIT: I found a cutter that preserves the problem, so i can send smaller samples if you are interested.

 

I suspect fixing the bug may need some collaboration with Jerome Martinez at mediainfo. I'm sure with his help it could be fixed in a jiffy

 

//Majstang

Edited May 3, 2014 by majstang

[majstang]

Fed up with this bug and nobody doing anything about it despite nagging about it for two years now. Decided to threw together my own mediainfo GUI and it works excellent to retrieve the video compression value on the files where your DVBViewer recording properties v.2.0 explorer plugin fails every time. Your mediainfo functions with Dllcalls must be wrongly constructed, especially the Unicode part.

 

 

;affected_file := "D:\TV\test\recording properties fails\20130828_14-19-03_Kanal 9_Tidsresenärerna - S18E07 10.ts"

;affected_file := "D:\TV\test\recording properties fails\20130830_14-14-02_Kanal 9_Tidsresenärerna - S18E09 10.ts"

;affected_file := "D:\TV\test\recording properties fails\20130903_14-19-03_Kanal 9_Tidsresenärerna - S19E01 14.ts"

;affected_file := "D:\TV\test\recording properties fails\20131010_14-19-02_Kanal 9_Tidsresenärerna - S12E04 13.ts"

;affected_file := "D:\TV\test\recording properties fails\20140429_21-59-04_TV12_Casino Royale - ★★★★★★★★☆☆.ts"

affected_file := "D:\TV\test\recording properties fails\20140501_21-59-02_TV12_American history X - ★★★★★★★★★☆.ts"

Mediainfo_location := "C:\Program Files\DVBViewer\MediaInfo.dll"

DllCall( "LoadLibrary", Str, Mediainfo_location )

hnd := MediaInfo_New()

MediaInfo_Open( hnd, affected_file )

videoformat := MediaInfo_Get( hnd, 1,0, "Format", 1 )

msgbox % videoformat

MediaInfo_New() {

Return DllCall( "mediainfo\MediaInfo" ( A_IsUnicode ? "" : "A" ) "_New" )

}

MediaInfo_Open( hnd, MediaFile ) {

Return DllCall( "mediainfo.dll\MediaInfo" ( A_IsUnicode ? "" : "A" ) "_Open", UInt,hnd

, Str,MediaFile, UInt )

}

MediaInfo_Get( hnd, StrK=0, StrN=0, Comm="", InfK=0, Srch=0 ) {

Return DllCall( "mediainfo.dll\MediaInfo" ( A_IsUnicode ? "" : "A" ) "_Get", UInt,hnd

, Int,StrK, Int,StrN, Str,Comm, Int,InfK, Int,Sech, Str )

}

MediaInfo_Close( hnd ) {

Return DllCall( "mediainfo\MediaInfo" ( A_IsUnicode ? "" : "A" ) "_Close", UInt,hnd )

}

 

Nevermind fixing the bug.

Thanks for nothing