staging: tidspbridge: fix potential array out of bounds write
authorOmar Ramirez Luna <omar.ramirez@copitl.com>
Thu, 10 Jan 2013 09:36:58 +0000 (03:36 -0600)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 18 Jan 2013 00:47:13 +0000 (16:47 -0800)
The name of the firmware (drv_datap->base_img) could potentially
become equal to 255 valid characters (size of exec_file), this
will result in an out of bounds write, given that the 255 chars
along with a '\0' terminator will be copied into an array of
255 chars.

Produce an error on this cases, because the driver expects the NULL
ending to be among the 255 char limit.

Reported-by: Chen Gang <gang.chen@asianux.com>
Signed-off-by: Omar Ramirez Luna <omar.ramirez@copitl.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/tidspbridge/rmgr/proc.c

index 5e43938ab7fa758170e34479992820cc361bddd4..ac016edc579fd2bb3027831177560dbad93216ca 100644 (file)
@@ -394,7 +394,7 @@ static int get_exec_file(struct cfg_devnode *dev_node_obj,
                if (!drv_datap || !drv_datap->base_img)
                        return -EFAULT;
 
-               if (strlen(drv_datap->base_img) > size)
+               if (strlen(drv_datap->base_img) >= size)
                        return -EINVAL;
 
                strcpy(exec_file, drv_datap->base_img);