drm: Don't create properties without names
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 2 Mar 2018 13:25:42 +0000 (15:25 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Tue, 6 Mar 2018 16:01:59 +0000 (18:01 +0200)
Creating a property that doesn't have a name makes no sense to me. Don't
allow it.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180302132544.12491-1-ville.syrjala@linux.intel.com
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/drm_property.c

index bae50e6..fe8627f 100644 (file)
@@ -99,10 +99,8 @@ struct drm_property *drm_property_create(struct drm_device *dev, int flags,
        property->num_values = num_values;
        INIT_LIST_HEAD(&property->enum_list);
 
-       if (name) {
-               strncpy(property->name, name, DRM_PROP_NAME_LEN);
-               property->name[DRM_PROP_NAME_LEN-1] = '\0';
-       }
+       strncpy(property->name, name, DRM_PROP_NAME_LEN);
+       property->name[DRM_PROP_NAME_LEN-1] = '\0';
 
        list_add_tail(&property->head, &dev->mode_config.property_list);