[Bug 44290] New: Possible access to unintended field in "wine/dlls/winmm/ mmio.c" line 788

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

            Bug ID: 44290
           Summary: Possible access to unintended field in
                    "wine/dlls/winmm/mmio.c" line 788
           Product: Wine
           Version: unspecified
          Hardware: x86
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: winmm&mci
          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/winmm/mmio.c"" line 788 function mmioRead

size = MMIO_GrabNextBuffer(wm, TRUE);
if (size <= 0) break;
if (size > cch) size = cch;
memcpy(pch, wm->info.pchBuffer, size);
wm->info.pchNext += size; //HERE
pch += size;
cch -= size;
count += size;

Shouldn't wm->info.pchBuffer be incremented at the mentioned line (instead of 
wm->info.pchNext)?

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 44290] Possible access to unintended field in "wine/dlls/winmm/mmio.c " line 788

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

Alexandre Julliard <julliard(a)winehq.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |INVALID

--- Comment #1 from Alexandre Julliard <julliard(a)winehq.org> ---
No, pchNext is correct.

-- 
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 44290] Possible access to unintended field in "wine/dlls/winmm/mmio.c " line 788

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

André H. <nerv(a)dawncrow.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nerv(a)dawncrow.de
             Status|RESOLVED                    |CLOSED

--- Comment #2 from André H. <nerv(a)dawncrow.de> ---
closing invalid

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