[pulseaudio-discuss] Valgrind warnings, again

[Alexander E. Patrakov]

16.06.2016 13:43, Ahmed S. Darwish пишет:
> Hi :-)
>
> On Sun, Jun 12, 2016 at 09:39:17PM +0500, Alexander E. Patrakov wrote:
>> I have spent more time running pulseaudio (master + Tanu's patches) under
>> valgrind today.
>>
>> This set of warnings has already been reported, and they appear every time
>> PulseAudio is started:
>>
>> ==15442== Conditional jump or move depends on uninitialised value(s)
>> ==15442==    at 0x5C91288: shm_attach (shm.c:380)
>> ==15442==    by 0x5C91B68: pa_shm_cleanup (shm.c:453)
>> ==15442==    by 0x5C91D4C: sharedmem_create (shm.c:150)
>> ==15442==    by 0x5C91D4C: pa_shm_create_rw (shm.c:239)
>> ==15442==    by 0x5C82193: pa_mempool_new (memblock.c:848)
>> ==15442==    by 0xF245FF7: setup_srbchannel (protocol-native.c:2634)
>> ==15442==    by 0xF245FF7: command_auth (protocol-native.c:2864)
>> ==15442==    by 0x5C8989E: pa_pdispatch_run (pdispatch.c:346)
>> ==15442==    by 0xF2486C4: pstream_packet_callback (protocol-native.c:4989)
>> ==15442==    by 0x5C8C216: do_read (pstream.c:987)
>> ==15442==    by 0x5C8EEF3: do_pstream_read_write (pstream.c:227)
>> ==15442==    by 0x510040B: dispatch_pollfds (mainloop.c:655)
>> ==15442==    by 0x510040B: pa_mainloop_dispatch (mainloop.c:898)
>> ==15442==    by 0x510080B: pa_mainloop_iterate (mainloop.c:929)
>> ==15442==    by 0x51008AF: pa_mainloop_run (mainloop.c:944)
>> ==15442==
>
> Hmm .. the memfd patchset touched all the paths from do_pstream_read_write()
> upwards. Nonetheless, full valgrind checks was done back then without
> seeing any errors. [*]
>
> Do you see these warnings with memfds enabled or disabled?
>
> [*] https://goo.gl/Ls6gzJ (last row for the full valgrind command)

I did not enable memfd at runtime and did not disable it at compile time.

I think I'll bisect it later today or tomorrow.

-- 
Alexander E. Patrakov