[Intel-gfx] [PATCH 1/3] drm: Don't create properties without names

Emil Velikov emil.l.velikov at gmail.com
Tue Mar 6 13:35:11 UTC 2018


Hi Ville,

On 2 March 2018 at 13:25, Ville Syrjala <ville.syrjala at linux.intel.com> wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>
> 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 at linux.intel.com>
> ---
>  drivers/gpu/drm/drm_property.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_property.c b/drivers/gpu/drm/drm_property.c
> index bae50e6b819d..fe8627fb7ae6 100644
> --- a/drivers/gpu/drm/drm_property.c
> +++ b/drivers/gpu/drm/drm_property.c
> @@ -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';
>
Shouldn't one also a) error (WARN + return NULL) out when name is NULL
and b) update the documentation?

-Emil


More information about the Intel-gfx mailing list