[Bug 53123] New: ListView: Ctrl+Shift+LMouse multiselect on LVS_OWNERDATA listviews erroneously sends LVN_ODSTATECHANGED

[WineHQ Bugzilla]

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

            Bug ID: 53123
           Summary: ListView: Ctrl+Shift+LMouse multiselect on
                    LVS_OWNERDATA listviews erroneously sends
                    LVN_ODSTATECHANGED
           Product: Wine
           Version: 7.10
          Hardware: x86-64
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: comctl32
          Assignee: wine-bugs at winehq.org
          Reporter: winehq-bugs at angelohaller.com
      Distribution: ---

Created attachment 72578
  --> https://bugs.winehq.org/attachment.cgi?id=72578
Example file that highlights the bug by crashing.

As in the title: ListViews that do not have LVS_OWNERDATA set as style falsely
do send an LVN_ODSTATECHANGED notification.

This notification is ONLY for LVS_OWNERDATA style ListViews and should not be
sent otherwise.

Steps to reproduce:
1. Run the attached `print-notify.exe`
2. Select a row by clicking on it
3. Hold down CTRL+SHIFT and select another row by clicking with the left mouse
button (selecting multiple rows)

The attached application crashes when using wine. Windows 7 does not.

This bug can only be triggered by left mouse clicks.

A patch is available as part of my `Fix LVS_OWNERDATA list view multi select.`
patch set. This report is solely to prove the bugs existence and increase the
likelihood of the patches getting accepted.

Source code available at:
https://raw.githubusercontent.com/szanni/win32-listview-tests/66163266e5433ea712a3dbbc6c3666465f0b4239/print-notify.c

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