Compare name length to avoid false positives in media_get_entity_by_name
authorSergio Aguirre <saaguirre@ti.com>
Wed, 25 Apr 2012 13:57:13 +0000 (08:57 -0500)
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Sun, 29 Apr 2012 17:30:50 +0000 (19:30 +0200)
If two subdevice have names that only differ by a suffix (such as "OMAP4
ISS ISP IPIPE" and "OMAP4 ISS ISP IPIPEIF") the media_get_entity_by_name
function might return a pointer to the entity with the longest name when
called with the shortest name. Fix this by verifying that the candidate
entity name length is equal to the requested name length.

Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
src/mediactl.c
src/tools.h

index 5b8c587..bc6a713 100644 (file)
@@ -63,10 +63,17 @@ struct media_entity *media_get_entity_by_name(struct media_device *media,
 {
        unsigned int i;
 
 {
        unsigned int i;
 
+       /* A match is impossible if the entity name is longer than the maximum
+        * size we can get from the kernel.
+        */
+       if (length >= FIELD_SIZEOF(struct media_entity_desc, name))
+               return NULL;
+
        for (i = 0; i < media->entities_count; ++i) {
                struct media_entity *entity = &media->entities[i];
 
        for (i = 0; i < media->entities_count; ++i) {
                struct media_entity *entity = &media->entities[i];
 
-               if (strncmp(entity->info.name, name, length) == 0)
+               if (strncmp(entity->info.name, name, length) == 0 &&
+                   entity->info.name[length] == '\0')
                        return entity;
        }
 
                        return entity;
        }
 
index e56edb2..de06cb3 100644 (file)
@@ -23,6 +23,7 @@
 #define __TOOLS_H__
 
 #define ARRAY_SIZE(array)      (sizeof(array) / sizeof((array)[0]))
 #define __TOOLS_H__
 
 #define ARRAY_SIZE(array)      (sizeof(array) / sizeof((array)[0]))
+#define FIELD_SIZEOF(t, f)     (sizeof(((t*)0)->f))
 
 #endif /* __TOOLS_H__ */
 
 
 #endif /* __TOOLS_H__ */