[PATCH 3/3] drm: property: Improve four size determinations

[Markus Elfring]

From: Markus Elfring <elfr...@users.sourceforge.net>
Date: Tue, 26 Dec 2023 09:45:36 +0100

Replace the specification of data structures by pointer dereferences
as the parameter for the operator “sizeof” to make the corresponding size
determination a bit safer according to the Linux coding style convention.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfr...@users.sourceforge.net>
---
 drivers/gpu/drm/drm_property.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/drm_property.c b/drivers/gpu/drm/drm_property.c
index ea365f00e890..1fe54cbf1c83 100644
--- a/drivers/gpu/drm/drm_property.c
+++ b/drivers/gpu/drm/drm_property.c
@@ -107,7 +107,7 @@ struct drm_property *drm_property_create(struct drm_device 
*dev,
        if (WARN_ON(strlen(name) >= DRM_PROP_NAME_LEN))
                return NULL;

-       property = kzalloc(sizeof(struct drm_property), GFP_KERNEL);
+       property = kzalloc(sizeof(*property), GFP_KERNEL);
        if (!property)
                return NULL;

@@ -418,7 +418,7 @@ int drm_property_add_enum(struct drm_property *property,
        if (WARN_ON(index >= property->num_values))
                return -EINVAL;

-       prop_enum = kzalloc(sizeof(struct drm_property_enum), GFP_KERNEL);
+       prop_enum = kzalloc(sizeof(*prop_enum), GFP_KERNEL);
        if (!prop_enum)
                return -ENOMEM;

@@ -560,10 +560,10 @@ drm_property_create_blob(struct drm_device *dev, size_t 
length,
        struct drm_property_blob *blob;
        int ret;

-       if (!length || length > INT_MAX - sizeof(struct drm_property_blob))
+       if (!length || length > INT_MAX - sizeof(*blob))
                return ERR_PTR(-EINVAL);

-       blob = kvzalloc(sizeof(struct drm_property_blob)+length, GFP_KERNEL);
+       blob = kvzalloc(sizeof(*blob) + length, GFP_KERNEL);
        if (!blob)
                return ERR_PTR(-ENOMEM);

--
2.43.0