Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756842Ab0DEW7i (ORCPT ); Mon, 5 Apr 2010 18:59:38 -0400 Received: from aeryn.fluff.org.uk ([87.194.8.8]:14018 "EHLO kira.home.fluff.org" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1756717Ab0DEW7c (ORCPT ); Mon, 5 Apr 2010 18:59:32 -0400 Date: Mon, 5 Apr 2010 23:59:25 +0100 From: Ben Dooks To: Wolfram Sang Cc: kernel-janitors@vger.kernel.org, linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: i2c-drivers: Remove dangling pointers Message-ID: <20100405225925.GC32401@fluff.org.uk> References: <1269094385-16114-1-git-send-email-w.sang@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1269094385-16114-1-git-send-email-w.sang@pengutronix.de> X-Disclaimer: These are my own opinions, so there! User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 7986 Lines: 175 On Sat, Mar 20, 2010 at 03:12:41PM +0100, Wolfram Sang wrote: > Hi, > > here is a patch series fixing I2C drivers which free their private data > structure and leave the clientdata pointer dangling or do the cleanup in the > wrong order. All occurences were found using coccinelle [1] and the semantic > patch below. The matches have been reviewed, and sometimes adapted. > > The complete series goes to linux-i2c and kernel-janitors. The rest of the CCs > is handled per patch using get_maintainers.pl (which always adds LKML, so you > have the complete series there, too). > > Please review, comment, apply! > > Kind regards, > > Wolfram > > [1] http://coccinelle.lip6.fr/ >From the list of things here, there's nothing in here that I should be worried about (just checking) ? > === > > @@ > type T; > identifier client, data; > @@ > > // Check if function uses clientdata > ( > i2c_set_clientdata(client, data); > | > data = i2c_get_clientdata(client); > | > T data = i2c_get_clientdata(client); > ) > // Anything inbetween > ... > // Check if clientdata gets NULLed before data is freed > ( > i2c_set_clientdata(client, NULL); > ... > kfree(data); > | > + i2c_set_clientdata(client, NULL); > kfree(data); > ... > - i2c_set_clientdata(client, NULL); > | > + i2c_set_clientdata(client, NULL); > ? kfree(data); > ) > > === > > drivers/gpio/adp5588-gpio.c | 1 + > drivers/gpio/max732x.c | 1 + > drivers/gpio/pca953x.c | 1 + > drivers/gpio/pcf857x.c | 7 ++++--- > drivers/hwmon/ad7414.c | 2 ++ > drivers/hwmon/ad7418.c | 2 ++ > drivers/hwmon/adm1021.c | 2 ++ > drivers/hwmon/adm1025.c | 2 ++ > drivers/hwmon/adm1026.c | 2 ++ > drivers/hwmon/adm1029.c | 2 ++ > drivers/hwmon/adm1031.c | 2 ++ > drivers/hwmon/adm9240.c | 2 ++ > drivers/hwmon/ads7828.c | 5 ++++- > drivers/hwmon/adt7462.c | 2 ++ > drivers/hwmon/adt7470.c | 2 ++ > drivers/hwmon/adt7475.c | 2 ++ > drivers/hwmon/amc6821.c | 2 ++ > drivers/hwmon/asb100.c | 2 ++ > drivers/hwmon/atxp1.c | 2 ++ > drivers/hwmon/dme1737.c | 2 ++ > drivers/hwmon/ds1621.c | 2 ++ > drivers/hwmon/f75375s.c | 4 ++-- > drivers/hwmon/g760a.c | 4 ++-- > drivers/hwmon/gl518sm.c | 2 ++ > drivers/hwmon/gl520sm.c | 2 ++ > drivers/hwmon/lm63.c | 2 ++ > drivers/hwmon/lm77.c | 2 ++ > drivers/hwmon/lm78.c | 2 ++ > drivers/hwmon/lm80.c | 2 ++ > drivers/hwmon/lm83.c | 2 ++ > drivers/hwmon/lm85.c | 2 ++ > drivers/hwmon/lm87.c | 2 ++ > drivers/hwmon/lm90.c | 2 ++ > drivers/hwmon/lm92.c | 2 ++ > drivers/hwmon/lm93.c | 2 ++ > drivers/hwmon/lm95241.c | 1 + > drivers/hwmon/ltc4215.c | 2 ++ > drivers/hwmon/ltc4245.c | 2 ++ > drivers/hwmon/max1619.c | 2 ++ > drivers/hwmon/max6650.c | 2 ++ > drivers/hwmon/pcf8591.c | 6 +++++- > drivers/hwmon/smsc47m192.c | 2 ++ > drivers/hwmon/thmc50.c | 2 ++ > drivers/hwmon/tmp401.c | 1 + > drivers/hwmon/w83791d.c | 2 ++ > drivers/hwmon/w83792d.c | 2 ++ > drivers/hwmon/w83793.c | 1 + > drivers/hwmon/w83l785ts.c | 2 ++ > drivers/hwmon/w83l786ng.c | 2 ++ > drivers/input/keyboard/lm8323.c | 2 ++ > drivers/input/keyboard/qt2160.c | 2 +- > drivers/input/touchscreen/mcs5000_ts.c | 2 +- > drivers/input/touchscreen/tsc2007.c | 1 + > drivers/leds/leds-lp3944.c | 2 +- > drivers/leds/leds-pca9532.c | 4 ++-- > drivers/leds/leds-pca955x.c | 4 ++-- > drivers/macintosh/therm_adt746x.c | 2 ++ > drivers/media/radio/radio-tea5764.c | 2 ++ > drivers/media/radio/si470x/radio-si470x-i2c.c | 2 +- > drivers/media/video/cs5345.c | 1 + > drivers/media/video/cs53l32a.c | 1 + > drivers/media/video/ir-kbd-i2c.c | 2 ++ > drivers/media/video/tda9840.c | 1 + > drivers/media/video/tea6415c.c | 1 + > drivers/media/video/tea6420.c | 1 + > drivers/media/video/ths7303.c | 1 + > drivers/mfd/88pm860x-i2c.c | 1 + > drivers/mfd/ab3100-core.c | 2 ++ > drivers/mfd/da903x.c | 1 + > drivers/mfd/menelaus.c | 3 ++- > drivers/mfd/pcf50633-core.c | 1 + > drivers/mfd/tps65010.c | 2 +- > drivers/mfd/wm8350-i2c.c | 2 ++ > drivers/misc/ad525x_dpot.c | 2 +- > drivers/misc/eeprom/at24.c | 2 +- > drivers/misc/eeprom/eeprom.c | 6 +++++- > drivers/misc/eeprom/max6875.c | 2 ++ > drivers/misc/ics932s401.c | 2 ++ > drivers/misc/isl29003.c | 6 +++++- > drivers/misc/tsl2550.c | 6 +++++- > drivers/mtd/maps/pismo.c | 8 +++++++- > drivers/power/bq27x00_battery.c | 2 ++ > drivers/power/ds2782_battery.c | 4 ++-- > drivers/regulator/max1586.c | 2 +- > drivers/regulator/max8649.c | 3 ++- > drivers/regulator/max8660.c | 2 +- > drivers/rtc/rtc-ds1307.c | 2 ++ > drivers/rtc/rtc-fm3130.c | 2 ++ > drivers/rtc/rtc-m41t80.c | 2 ++ > drivers/rtc/rtc-pcf8563.c | 2 ++ > drivers/rtc/rtc-pcf8583.c | 2 ++ > drivers/rtc/rtc-rs5c372.c | 2 ++ > drivers/rtc/rtc-s35390a.c | 4 ++-- > drivers/staging/dream/synaptics_i2c_rmi.c | 2 ++ > drivers/staging/go7007/wis-saa7113.c | 1 + > drivers/staging/go7007/wis-saa7115.c | 1 + > drivers/staging/go7007/wis-tw9903.c | 1 + > drivers/staging/iio/adc/max1363_core.c | 2 ++ > drivers/staging/iio/light/tsl2563.c | 2 ++ > drivers/usb/otg/isp1301_omap.c | 5 ++++- > drivers/video/matrox/matroxfb_maven.c | 1 + > drivers/w1/masters/ds2482.c | 2 ++ > 102 files changed, 198 insertions(+), 33 deletions(-) > > -- > To unsubscribe from this list: send the line "unsubscribe linux-i2c" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Ben (ben@fluff.org, http://www.fluff.org/) 'a smiley only costs 4 bytes' -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/