[Bug 26171] BVRP classic phone tools installer reports error 'Insufficient buffer' when trying to install print processor (AddPrintProcessorA stub should be symmetric to AddPrintProcessorW)

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=26171

Anastasius Focht <focht at gmx.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|BVRP classic phone tools    |BVRP classic phone tools
                   |fail to install             |installer reports error
                   |                            |'Insufficient buffer' when
                   |                            |trying to install print
                   |                            |processor
                   |                            |(AddPrintProcessorA stub
                   |                            |should be symmetric to
                   |                            |AddPrintProcessorW)
           Keywords|                            |Installer
                 CC|                            |focht at gmx.net
          Component|-unknown                    |spooler
                URL|ftp://ftp2.avanquest.com/ev |https://web.archive.org/web
                   |aluation/ClassicPhoneTools/ |/20200409184340/http://www.
                   |ClassicPhoneTools9ENG.exe   |tucows.com/download/windows
                   |                            |/files2/ClassicPhoneTools9E
                   |                            |NU.exe

--- Comment #6 from Anastasius Focht <focht at gmx.net> ---
Hello folks,

the problem with the message box being displayed is not critical. The installer
still succeeds in the end.

--- snip ---
...
002e:Call winspool.drv.AddPrintProcessorA(00000000,02724cc0 "Windows
x64",02724cd0 "WfxPrint2000.dll",02724d54 "WfxPrint2000") ret=02712af8
002e:fixme:winspool:AddPrintProcessorA ((null),"Windows
x64","WfxPrint2000.dll","WfxPrint2000"): stub
002e:Ret  winspool.drv.AddPrintProcessorA() retval=00000000 ret=02712af8
...
002e:Call user32.MessageBoxA(00000000,04fa9fd0 "Insufficient
buffer.\r\n",02724c40 "AddPrintProcessor2000",00000040) ret=02711b5c 
--- snip ---

https://source.winehq.org/git/wine.git/blob/a98ca88dd1139ad8e16e63fbb349c2fc5e9ad332:/dlls/winspool.drv/info.c#l5400

--- snip ---
5400
/*****************************************************************************
5401  *          AddPrintProcessorA  [WINSPOOL.@]
5402  */
5403 BOOL WINAPI AddPrintProcessorA(LPSTR pName, LPSTR pEnvironment, LPSTR
pPathName,
5404                                LPSTR pPrintProcessorName)
5405 {
5406     FIXME("(%s,%s,%s,%s): stub\n", debugstr_a(pName),
debugstr_a(pEnvironment),
5407           debugstr_a(pPathName), debugstr_a(pPrintProcessorName));
5408     return FALSE;
5409 }
--- snip ---

The crash with the app as seen in comment #0 backtrace is tracked as bug 44874
("Phonetools 9.x crashes on unimplemented function
KERNEL32.dll.RequestDeviceWakeup").

So the only point of this ticket would be faking success as
AddPrintProcessorW() already does but still being a stub. Bug 26006 is about
implementing it.

$ sha1sum ClassicPhoneTools9ENU.exe 
3630c8bcdd533320cacfbc99b9efee8e01eb59c3  ClassicPhoneTools9ENU.exe

$ du -sh ClassicPhoneTools9ENU.exe 
17M    ClassicPhoneTools9ENU.exe

$ wine --version
wine-5.5-373-g38e95ed222

Regards

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.