[Bug 44289] New: Possible access to unintended field in "wine/dlls/ comctl32/comboex.c" line 1033

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

            Bug ID: 44289
           Summary: Possible access to unintended field in
                    "wine/dlls/comctl32/comboex.c" line 1033
           Product: Wine
           Version: unspecified
          Hardware: x86
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: comctl32
          Assignee: wine-bugs(a)winehq.org
          Reporter: petrum(a)gmail.com
      Distribution: ---

While experimenting with a CodeSonar plugin we develop, we noticed a
potential bug in file "wine/dlls/comctl32/comboex.c" line 1033 function
COMBOEX_Create

SetWindowSubclass(infoPtr->hwndEdit, COMBOEX_EditWndProc, EDIT_SUBCLASSID,
                      (DWORD_PTR)hwnd);

infoPtr->font = (HFONT)SendMessageW(infoPtr->hwndCombo /*HERE*/, WM_GETFONT, 0,
0);

Shouldn't you access infoPtr->hwndEdit (instead of infoPtr->hwndCombo) at the
mentioned position in the second line?

Thanks,
Petru Florin Mihancea

-- 
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.

[Bug 44289] Possible access to unintended field in "wine/dlls/comctl32/comboex.c" line 1033

[WineHQ Bugzilla]

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

KRosUser <kyle.kcsoftwares(a)gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kyle.kcsoftwares(a)gmail.com

--- Comment #1 from KRosUser <kyle.kcsoftwares(a)gmail.com> ---
infoPtr->hwndCombo is still used in Wine 10.8 here :
https://github.com/wine-mirror/wine/blob/master/dlls/comctl32/comboex.c#L101...

Any confirmation it's a bug ?
Otherwise, to be CLOSED ?

-- 
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.