After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 780406 - Valgrind warns about uninitialized data
Valgrind warns about uninitialized data
Status: RESOLVED DUPLICATE of bug 772949
Product: pygobject
Classification: Bindings
Component: introspection
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Nobody's working on this now (help wanted and appreciated)
Python bindings maintainers
Depends on:
Blocks:
 
 
Reported: 2017-03-22 15:26 UTC by Nicolas Dufresne (ndufresne)
Modified: 2017-03-23 18:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
pygi-info: Initialize GIArgument structure to 0 (1.85 KB, patch)
2017-03-22 15:26 UTC, Nicolas Dufresne (ndufresne)
none Details | Review

Description Nicolas Dufresne (ndufresne) 2017-03-22 15:26:33 UTC
I've met those message when running python/gi inside valgrind. I do think this is just noise, but still, have wrote a patch to silence it (patch coming).

==30861== Conditional jump or move depends on uninitialised value(s)
==30861==    at 0x4ED387A: PyLong_FromLong (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x16D62760: _pygi_argument_to_object (pygi-argument.c:1424)
==30861==    by 0x16D5BEC4: _wrap_g_constant_info_get_value (pygi-info.c:1724)
==30861==    by 0x4F5EB3C: PyEval_EvalFrameEx (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4F605E2: ??? (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4F606C2: PyEval_EvalCodeEx (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4ECABF7: ??? (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4E9FED6: PyObject_Call (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4EB6933: ??? (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4E9FED6: PyObject_Call (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4EA0AE1: PyObject_CallFunctionObjArgs (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4EF791E: ??? (in /usr/lib64/libpython3.5m.so.1.0)
Comment 1 Nicolas Dufresne (ndufresne) 2017-03-22 15:26:58 UTC
Created attachment 348510 [details] [review]
pygi-info: Initialize GIArgument structure to 0

This is purely cosmetic, at least I could not find any impact, but it
removes valgrind noise that this is being used uninitialized. An example
backtrace in valgrind looked like:

==30861== Conditional jump or move depends on uninitialised value(s)
==30861==    at 0x4ED387A: PyLong_FromLong (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x16D62760: _pygi_argument_to_object (pygi-argument.c:1424)
==30861==    by 0x16D5BEC4: _wrap_g_constant_info_get_value (pygi-info.c:1724)
==30861==    by 0x4F5EB3C: PyEval_EvalFrameEx (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4F605E2: ??? (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4F606C2: PyEval_EvalCodeEx (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4ECABF7: ??? (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4E9FED6: PyObject_Call (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4EB6933: ??? (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4E9FED6: PyObject_Call (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4EA0AE1: PyObject_CallFunctionObjArgs (in /usr/lib64/libpython3.5m.so.1.0)
==30861==    by 0x4EF791E: ??? (in /usr/lib64/libpython3.5m.so.1.0)
Comment 2 Christoph Reiter (lazka) 2017-03-22 15:40:59 UTC
Thanks :) See https://git.gnome.org/browse/pygobject/commit/?id=399939b55860211e0ab64bdfbfada2c86be8bf64
Comment 3 Christoph Reiter (lazka) 2017-03-23 15:03:07 UTC
I've pushed it to the 3-24 stable branch as well now

*** This bug has been marked as a duplicate of bug 772949 ***
Comment 4 Nicolas Dufresne (ndufresne) 2017-03-23 18:04:20 UTC
Thanks to you.