diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2017-08-09 11:07:24 +0100 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2018-02-14 09:41:49 +0000 |
commit | 1d227a47223d2ff5c6745c0d82e96e70d456d5de (patch) | |
tree | 412650c1cc6026154ce2cc3979b73969c485fce8 /lib/igt_kms.c | |
parent | 6d51cf112374ba73f7391b5d846bb83e81b060d3 (diff) |
lib/kms: Clear unused fields for getproperty ioctl
==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@chris-wilson.co.uk>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Diffstat (limited to 'lib/igt_kms.c')
-rw-r--r-- | lib/igt_kms.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/lib/igt_kms.c b/lib/igt_kms.c index eb57f4a9..ecc7a799 100644 --- a/lib/igt_kms.c +++ b/lib/igt_kms.c @@ -1108,11 +1108,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; |