[Bug 28423] ImmGetDescriptionW() from imm32 crashes with certain parameters

[wine-bugs at winehq.org]

http://bugs.winehq.org/show_bug.cgi?id=28423

Aric Stewart <aric at codeweavers.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |aric at codeweavers.com

--- Comment #15 from Aric Stewart <aric at codeweavers.com> 2013-09-17 13:12:34 CDT ---
(In reply to comment #14)
> Created attachment 40573 [details]
> Patch to fix the issue in wine 1.5.6
> 
> The attached patch will fix the problem but as I could not test it fully in
> windows I'm not submitting it yet. None of the machines in testbot returns
> valid IMEs to test (tests are skipped).

Ok I am testing on Japanese windows XP.

ret = ImmGetDescriptionW(hkl, NULL, 100);  -> Causes and exception
ret = ImmGetDescriptionA(hkl, NULL, 100); -> Causes an exception


As for Vista and Windows 7, ImmGetDescription is broken there. It just does not
function.

http://blogs.msdn.com/b/michkap/archive/2008/12/04/9173920.aspx

I can try installing a custom old style IME and see but the above windows XP
behavior seems to imply to me that crashing in the case of HKL, NULL, 100 is
correct.


Back to windows XP;

    ret = ImmGetDescriptionW(NULL, NULL, 100);
    ok(!ret, "ImmGetDescriptionW failed, expected 0 received %d.\n", ret);

That works without issue.


that that implys that your 
+  if (!hKL) return 0;

is correct, but that
+  if (!lpszDescription) return 0;

is incorrect.

If there is any other testing I can do please ask.

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
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.