[Intel-gfx] [PATCH igt 10/10] lib/kms: Clear unused fields for getproperty ioctl
Chris Wilson
chris at chris-wilson.co.uk
Mon Dec 4 09:27:31 UTC 2017
==24749== Syscall param ioctl(generic) points to uninitialised byte(s)
==24749== at 0x6A8ADC7: ioctl (syscall-template.S:84)
==24749== by 0x5067687: drmIoctl (in /opt/xorg/lib64/libdrm.so.2.4.0)
==24749== by 0x138531: kmstest_set_connector_dpms (igt_kms.c:1022)
==24749== by 0x112937: set_dpms (kms_flip.c:263)
==24749== by 0x112937: run_test_step (kms_flip.c:776)
==24749== by 0x112937: event_loop (kms_flip.c:1138)
==24749== by 0x115468: run_test_on_crtc_set (kms_flip.c:1378)
==24749== by 0x115468: run_test (kms_flip.c:1450)
==24749== by 0x1111CF: main (kms_flip.c:1673)
==24749== Address 0x1ffefff2a0 is on thread 1's stack
==24749== in frame #2, created by kmstest_set_connector_dpms (igt_kms.c:1012)
Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
---
lib/igt_kms.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index 125ecb19..0c3ebdb2 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -1107,11 +1107,10 @@ void kmstest_set_connector_dpms(int fd, drmModeConnector *connector, int mode)
bool found_it = false;
for (i = 0; i < connector->count_props; i++) {
- struct drm_mode_get_property prop;
+ struct drm_mode_get_property prop = {
+ .prop_id = connector->props[i],
+ };
- prop.prop_id = connector->props[i];
- prop.count_values = 0;
- prop.count_enum_blobs = 0;
if (drmIoctl(fd, DRM_IOCTL_MODE_GETPROPERTY, &prop))
continue;
--
2.15.1
More information about the Intel-gfx
mailing list