[Bug 46342] New: kernel32/tests/ pipe.c shows uninitialized memory use in test_TransactNamedPipe()

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

            Bug ID: 46342
           Summary: kernel32/tests/pipe.c shows uninitialized memory use
                    in test_TransactNamedPipe()
           Product: Wine
           Version: 4.0-rc3
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase, valgrind
          Severity: normal
          Priority: P2
         Component: kernel32
          Assignee: wine-bugs(a)winehq.org
          Reporter: austinenglish(a)gmail.com
      Distribution: Gentoo

../../../tools/runtest -q -P wine -T ../../.. -M kernel32.dll -p
kernel32_test.exe.so pipe && touch pipe.ok
==8919== Syscall param writev(vector[...]) points to uninitialised byte(s)
==8919==    at 0x4356813: writev (writev.c:26)
==8919==    by 0x7BC75457: send_request (server.c:228)
==8919==    by 0x7BC76158: server_call_unlocked (server.c:288)
==8919==    by 0x7BC916EC: virtual_locked_server_call (virtual.c:2083)
==8919==    by 0x7BC41C1A: server_ioctl_file (file.c:1533)
==8919==    by 0x7BC447A5: NtFsControlFile (file.c:1733)
==8919==    by 0x7B470FA4: TransactNamedPipe (sync.c:1747)
==8919==    by 0x4B26CCC: _overlapped_transact_failure (pipe.c:3118)
==8919==    by 0x4B2748E: test_TransactNamedPipe (pipe.c:3346)
==8919==    by 0x4B27817: func_pipe (pipe.c:3735)
==8919==    by 0x4B6E8E4: run_test (test.h:617)
==8919==    by 0x4B6F322: main (test.h:701)
==8919==  Address 0x4eafd22 is on thread 1's stack
==8919==  in frame #8, created by test_TransactNamedPipe (pipe.c:3341)
==8919==  Uninitialised value was created by a stack allocation
==8919==    at 0x4B27446: test_TransactNamedPipe (pipe.c:3341)
==8919==

-- 
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 46342] kernel32/tests/ pipe.c shows uninitialized memory use in test_TransactNamedPipe()

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

Jacek Caban <jacek(a)codeweavers.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jacek(a)codeweavers.com

--- Comment #1 from Jacek Caban <jacek(a)codeweavers.com> ---
Created attachment 63209
  --> https://bugs.winehq.org/attachment.cgi?id=63209
fix

Does the attached patch help?

-- 
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 46342] kernel32/tests/ pipe.c shows uninitialized memory use in test_TransactNamedPipe()

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

--- Comment #2 from Austin English <austinenglish(a)gmail.com> ---
(In reply to Jacek Caban from comment #1)
> Created attachment 63209 [details]
> fix
>
> Does the attached patch help?

No, looks the same here (except line numbers shifted down by 5).

-- 
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 46342] kernel32/tests/pipe.c shows uninitialized memory use in test_TransactNamedPipe()

[WineHQ Bugzilla]

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

joaopa <jeremielapuree(a)yahoo.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jeremielapuree(a)yahoo.fr

--- Comment #3 from joaopa <jeremielapuree(a)yahoo.fr> ---
Created attachment 65596
  --> https://bugs.winehq.org/attachment.cgi?id=65596
console output

Looks like this bug is fixed in wine-4.19 (see the attached console output).
Can an administrator close this bug as FIXED?

-- 
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 46342] kernel32/tests/pipe.c shows uninitialized memory use in test_TransactNamedPipe()

[WineHQ Bugzilla]

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

--- Comment #4 from Austin English <austinenglish(a)gmail.com> ---
(In reply to joaopa from comment #3)
> Created attachment 65596 [details]
> console output
>
> Looks like this bug is fixed in wine-4.19 (see the attached console output).
> Can an administrator close this bug as FIXED?

I still see it:
==31951== 6 errors in context 2 of 15:
==31951== Syscall param writev(vector[...]) points to uninitialised byte(s)
==31951==    at 0x4368973: __writev (writev.c:26)
==31951==    by 0x4368973: writev (writev.c:24)
==31951==    by 0x7BC967C8: send_request (server.c:228)
==31951==    by 0x7BC97496: server_call_unlocked (server.c:288)
==31951==    by 0x7BCB1874: virtual_locked_server_call (virtual.c:2176)
==31951==    by 0x7BC6037A: server_ioctl_file (file.c:1532)
==31951==    by 0x7BC62F15: NtFsControlFile (file.c:1727)
==31951==    by 0x4A9F5A8: TransactNamedPipe (sync.c:1302)
==31951==    by 0x4C06104: _overlapped_transact_failure (pipe.c:3118)
==31951==    by 0x4C067ED: test_TransactNamedPipe (pipe.c:3347)
==31951==    by 0x4C06B39: func_pipe (pipe.c:4130)
==31951==    by 0x4C4A5E4: run_test (test.h:637)
==31951==    by 0x4C4AE6B: main (test.h:721)
==31951==  Address 0x4ebfdd2 is on thread 1's stack
==31951==  in frame #8, created by test_TransactNamedPipe (pipe.c:3342)
==31951==  Uninitialised value was created by a stack allocation
==31951==    at 0x4C067B7: test_TransactNamedPipe (pipe.c:3342)
==31951== 

wine-4.20-213-gddec23013e

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