diff options
| author | Hans de Goede <j.w.r.degoede@hhs.nl> | 2006-06-04 20:23:01 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2006-06-22 11:10:34 -0700 |
| commit | a2392e0b907b9633c31be14ee75bb39fce348b01 (patch) | |
| tree | 7e393e1c53b7f26c87d871fd9eb401b0f4369d7d /drivers/i2c | |
| parent | f2b84bbcebfdbe4855bab532909eef6621999f9f (diff) | |
[PATCH] abituguru: Review fixes
Fixes to the Abit uGuru driver as requested in review by Jean Delvare:
- exactly calculate the sysfs_names array length using macro
- use snprintf when generating names to double check that the sysfs_names
array does not overflow.
- use ARRAY_SIZE and / or defines to determine number of loops in for loops
instead of using hardcoded values.
- In abituguru_probe(), refactor the error path leaving a single call to kfree
Signed-off-by: Hans de Goede <j.w.r.degoede@hhs.nl>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/i2c')
0 files changed, 0 insertions, 0 deletions