drivers/gpu/drm/drm_sysfs.c: sysfs files error handling
authorDan Carpenter <error27@gmail.com>
Tue, 27 Apr 2010 21:11:05 +0000 (14:11 -0700)
committerDave Airlie <airlied@redhat.com>
Wed, 28 Apr 2010 08:42:52 +0000 (18:42 +1000)
commita1c4560d4d8909cc4feb6f9e875d0b92083e05cf
tree5fad4d68b80048f95068637c17888d700a47e09e
parent0031c41be5c529f8329e327b63cde92ba1284842
drivers/gpu/drm/drm_sysfs.c: sysfs files error handling

In the original code we used "j" as an iterator but we used "i" as an
index.

-               for (j = 0; j < i; j++)
-                       device_remove_file(&connector->kdev,
-                                          &connector_attrs[i]);

Smatch complained about that because "i" was potentially passed the end of
the array.  Which makes sense if we should be using "j" there.

I also thought that we should remove the files for &connector_attrs_opt1
but to do that I had to add separate iterators for &connector_attrs and
&connector_attrs_opt1.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@suse.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/drm_sysfs.c