https://bugs.winehq.org/show_bug.cgi?id=38020
Bug ID: 38020 Summary: Foxit Reader 6.12 crashes on unimplemented function msasn1.dll.ASN1_CreateModule during validation of digital signatures Product: Wine Version: 1.7.35 Hardware: x86 OS: Linux Status: NEW Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs@winehq.org Reporter: focht@gmx.net Distribution: ---
Hello folks,
split off from bug 35613
Prerequisite: 'winetricks -q mfc42'
--- snip --- err:module:import_dll Library MFC42u.DLL (which is needed by L"C:\Program Files\Foxit Software\Foxit Reader\plugins\jrsys\x86\jrsysMSCryptoDll.dll") not found --- snip ---
When doing the validation:
--- snip --- $ pwd /home/focht/.wine/drive_c/Program Files/Foxit Software/Foxit Reader
$ wine ./Foxit\ Reader.exe ... wine: Call from 0x7b83b527 to unimplemented function msasn1.dll.ASN1_CreateModule, aborting fixme:ntdll:RtlNtStatusToDosErrorNoTeb no mapping for 80000100 --- snip ---
An ASN1 API implementation can be found in the 'Windows Portable Runtime' (FreeRDP spin-off project, Apache License).
$ sha1sum FoxitReader612.1224_enu_Setup.exe 3c4a6587af9c09b760fcc60038ad88c964404060 FoxitReader612.1224_enu_Setup.exe
$ du -sh FoxitReader612.1224_enu_Setup.exe 32M FoxitReader612.1224_enu_Setup.exe
$ wine --version wine-1.7.35-89-gbad99c9
Regards
https://bugs.winehq.org/show_bug.cgi?id=38020
Anastasius Focht focht@gmx.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download URL| |http://www.oldapps.com/foxi | |t_reader.php?old_foxit_read | |er=13798
https://bugs.winehq.org/show_bug.cgi?id=38020
--- Comment #1 from Austin English austinenglish@gmail.com --- Created attachment 50811 --> https://bugs.winehq.org/attachment.cgi?id=50811 stub
Does this help?
https://bugs.winehq.org/show_bug.cgi?id=38020
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |austinenglish@gmail.com
https://bugs.winehq.org/show_bug.cgi?id=38020
--- Comment #2 from Anastasius Focht focht@gmx.net --- Hello Austin,
no, that can't work if you don't update the .spec file.
With the .spec file fixed it runs into next stub:
--- snip --- ... fixme:shell:IPersistFile_fnSaveCompleted (0x1bf8c8)->(L"C:\users\focht\Recent\SampleSignedPDFDocument.pdf (3).lnk"): stub ... fixme:msasn:ASN1_CreateModule 65536, 1024, 1000, 4, 0x765e9234, 0x765e9244, 0x765e9254, 1985909348, 1953919600 stub! fixme:msasn:ASN1_CreateModule 65536, 1024, 1000, 22, 0x765e8fe0, 0x765e9038, 0x765e9090, 1985908968, 1886938736 stub! fixme:msasn:ASN1_CreateModule 65536, 1024, 1000, 22, 0x765e8e18, 0x765e8e70, 0x765e8ec8, 1985908512, 1853384304 stub! fixme:msasn:ASN1_CreateModule 65536, 1024, 1000, 64, 0x765e7160, 0x765e7260, 0x765e7360, 1985901664, 959460728 stub! fixme:msasn:ASN1_CreateModule 65536, 1024, 1000, 48, 0x765e8998, 0x765e8a58, 0x765e8b18, 1985907672, 1935895408 stub! wine: Call from 0x7b83b54f to unimplemented function msasn1.dll.ASN1_Decode, aborting --- snip ---
Sample signed PDF download: http://blogs.adobe.com/security/SampleSignedPDFDocument.pdf
Regards
https://bugs.winehq.org/show_bug.cgi?id=38020
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #50811|0 |1 is obsolete| |
--- Comment #3 from Austin English austinenglish@gmail.com --- Created attachment 50832 --> https://bugs.winehq.org/attachment.cgi?id=50832 patches
I contacted the author of the FreeRDP asn1 implementation, who has agreed to relicense that code under LPGL2.1 for wine.
Unfortuantely, only one function that was implemented was used by Foxit. With that and the other patches, there are no crashes for me.
https://bugs.winehq.org/show_bug.cgi?id=38020
André H. nerv@dawncrow.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |nerv@dawncrow.de
--- Comment #4 from André H. nerv@dawncrow.de --- (In reply to Austin English from comment #3)
Created attachment 50832 [details] patches
I contacted the author of the FreeRDP asn1 implementation, who has agreed to relicense that code under LPGL2.1 for wine.
Unfortuantely, only one function that was implemented was used by Foxit. With that and the other patches, there are no crashes for me.
Good job!
https://bugs.winehq.org/show_bug.cgi?id=38020
--- Comment #5 from Austin English austinenglish@gmail.com --- (In reply to André H. from comment #4)
(In reply to Austin English from comment #3)
Created attachment 50832 [details] patches
I contacted the author of the FreeRDP asn1 implementation, who has agreed to relicense that code under LPGL2.1 for wine.
Unfortuantely, only one function that was implemented was used by Foxit. With that and the other patches, there are no crashes for me.
Good job!
https://source.winehq.org/patches/data/115834
https://bugs.winehq.org/show_bug.cgi?id=38020
Anastasius Focht focht@gmx.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch
https://bugs.winehq.org/show_bug.cgi?id=38020
Alistair Leslie-Hughes leslie_alistair@hotmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |leslie_alistair@hotmail.com Staged patchset| |https://github.com/wine-sta | |ging/wine-staging/tree/mast | |er/patches/msasn1-ASN1_Crea | |teModule Status|NEW |STAGED
https://bugs.winehq.org/show_bug.cgi?id=38020
Alistair Leslie-Hughes leslie_alistair@hotmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|STAGED |NEW
https://bugs.winehq.org/show_bug.cgi?id=38020
Anastasius Focht focht@gmx.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Component|-unknown |msasn1
--- Comment #6 from Anastasius Focht focht@gmx.net --- Hello folks,
revisiting, still present.
https://source.winehq.org/git/wine.git/blob/HEAD:/dlls/msasn1/msasn1.spec
$ sha1sum FoxitReader612.1224_enu_Setup.exe 3c4a6587af9c09b760fcc60038ad88c964404060 FoxitReader612.1224_enu_Setup.exe
$ du -sh FoxitReader612.1224_enu_Setup.exe 32M FoxitReader612.1224_enu_Setup.exe
$ wine --version wine-4.3-136-g56f34c7489
Regards
https://bugs.winehq.org/show_bug.cgi?id=38020
Vijay Kamuju infyquest@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |infyquest@gmail.com
--- Comment #7 from Vijay Kamuju infyquest@gmail.com --- sent new patch series for stub implementation. https://source.winehq.org/patches/data/184526 https://source.winehq.org/patches/data/184531 https://source.winehq.org/patches/data/184528
https://bugs.winehq.org/show_bug.cgi?id=38020
--- Comment #8 from Vijay Kamuju infyquest@gmail.com --- The stubs are committed, sent in new patches for implementation of CreateModule https://source.winehq.org/patches/data/184796 https://source.winehq.org/patches/data/184799
https://bugs.winehq.org/show_bug.cgi?id=38020
--- Comment #9 from Vijay Kamuju infyquest@gmail.com --- CreateModule is implemented, can we close this bug? I tried the Foxit version stated here it does not have the relevant plugins. It does not replicate the issue. I will try to get a stub implemented for the second part of the bug. I will try to find an older version of the Foxit.
https://bugs.winehq.org/show_bug.cgi?id=38020
Vijay Kamuju infyquest@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Fixed by SHA1| |48aefd2a0409d8c2a8474eb1109 | |92ac2532b6a78 Resolution|--- |FIXED
--- Comment #10 from Vijay Kamuju infyquest@gmail.com --- The second part is also implemented as a stub to avoid the crash, hence we can close this bug. Fixed by https://source.winehq.org/git/wine.git/commitdiff/48aefd2a0409d8c2a8474eb110... https://source.winehq.org/git/wine.git/commitdiff/84096382fc4ca80f5fc59908bf... https://source.winehq.org/git/wine.git/commitdiff/d2f3f3aaa831fa9c5b14dac824...
https://bugs.winehq.org/show_bug.cgi?id=38020
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #11 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 5.10.
https://bugs.winehq.org/show_bug.cgi?id=38020
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |5.0.x
https://bugs.winehq.org/show_bug.cgi?id=38020
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|5.0.x |---
--- Comment #12 from Michael Stefaniuc mstefani@winehq.org --- Removing the 5.0.x milestone from bug fixes included in 5.0.3.
https://bugs.winehq.org/show_bug.cgi?id=38020
Anastasius Focht focht@gmx.net changed:
What |Removed |Added ---------------------------------------------------------------------------- URL|http://www.oldapps.com/foxi |https://web.archive.org/web |t_reader.php?old_foxit_read |/20150411041055/http://down |er=13798 |load.oldapps.com/FoxItReade | |r/FoxitReader612.1224_enu_S | |etup.exe