xf86-video-amdgpu: Branch 'master'

Michel Dänzer daenzer at kemper.freedesktop.org
Thu Jun 15 03:21:40 UTC 2017


 src/drmmode_display.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 184d50e008b9b31f4dda1425e255af068b6ab068
Author: Adam Jackson <ajax at redhat.com>
Date:   Tue Jun 13 09:36:21 2017 -0400

    modesetting: Validate the atom for enum properties
    
    The client could have said anything here, and if what they said doesn't
    actually name an atom NameForAtom() will return NULL, and strcmp() will
    be unhappy about that.
    
    [copied from xserver d4995a3936ae283b9080fdaa0905daa669ebacfc]
    
    Signed-off-by: Adam Jackson <ajax at redhat.com>
    Reviewed-and-Tested-by: Michel Dänzer <michel.daenzer at amd.com>

diff --git a/src/drmmode_display.c b/src/drmmode_display.c
index f4bea0c..0d90041 100644
--- a/src/drmmode_display.c
+++ b/src/drmmode_display.c
@@ -1663,7 +1663,8 @@ drmmode_output_set_property(xf86OutputPtr output, Atom property,
 			    || value->size != 1)
 				return FALSE;
 			memcpy(&atom, value->data, 4);
-			name = NameForAtom(atom);
+			if (!(name = NameForAtom(atom)))
+				return FALSE;
 
 			/* search for matching name string, then set its value down */
 			for (j = 0; j < p->mode_prop->count_enums; j++) {


More information about the xorg-commit mailing list