[Bug 8532] JawsEditor 2.5/3.0 reports "Invalid imagesize" on startup ('IPicture::SaveAsFile' method too stubby/incorrect)

[wine-bugs at winehq.org]

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

Anastasius Focht <focht at gmx.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|http://www.jawspdf.com/down |http://www.jawspdf.com/down
                   |load/editor.html            |load/previous/#editor

--- Comment #8 from Anastasius Focht <focht at gmx.net> ---
Hello Dmitry,

thanks for the patch. Unfortunately it still doesn't work.

Although bitmap data is now provided by the help of 'OLEPictureImpl_Save' it
seems the app stream write method doesn't expect the 'StdPicture' header
(preamble + size) in preceding 'IStream_Write' call, causing a different
internal error/code path (still same external "Invalid imagesize" error
though).

It rather expects the raw bitmap data via 'IStream_Write(pStm, This->data,
This->datalen, &dummy)' immediately.

Source:
https://source.winehq.org/git/wine.git/blob/2f9987fd043d094c23430a1698f1a722be76b9bf:/dlls/oleaut32/olepicture.c#l1834

--- snip ---
...
1834         header[0] = (This->loadtime_magic != 0xdeadbeef) ?
This->loadtime_magic : 0x0000746c;
1835         header[1] = This->datalen;
1836         IStream_Write(pStm, header, 2 * sizeof(DWORD), &dummy);
1837         IStream_Write(pStm, This->data, This->datalen, &dummy);
1838         hResult = S_OK;
...
--- snip ---

I removed the 'StdPicture' header serialization call for testing purpose
(potentially breaking other things) and it allows the app to start.
Loading/viewing/annotating PDF works nicely.

This means we're on the right track but can't simply reuse
'OLEPictureImpl_Save' here.

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.