On 10/21/2012 11:52 AM, Julia Lawall wrote: > From: Julia Lawall <julia.law...@lip6.fr> > > The function __iio_add_event_config_attrs is only called once, by the > function iio_device_register_eventset. If the call fails, > iio_device_register_eventset calls __iio_remove_event_config_attrs. There > is thus no need for __iio_add_event_config_attrs to also call > __iio_remove_event_config_attrs on failure. > > A simplified version of the semantic match that finds this problem is as > follows: (http://coccinelle.lip6.fr/) > > // <smpl> > @r@ > identifier f,free,a; > parameter list[n] ps; > type T; > expression e; > @@ > > f(ps,T a,...) { > ... when any > when != a = e > if(...) { ... free(a); ... return ...; } > ... when any > } > > @@ > identifier r.f,r.free; > expression x,a; > expression list[r.n] xs; > @@ > > * x = f(xs,a,...); > if (...) { ... free(a); ... return ...; } > // </smpl> > > Signed-off-by: Julia Lawall <julia.law...@lip6.fr>
Thanks - merged to fixes-togreg branch of iio.git Jonathan > > --- > __iio_remove_event_config_attrs kfrees the elements of a list, but doesn't > actually remove them from the list. Perhaps for safety this should be > cleaned up as well. Not tested. > > drivers/iio/industrialio-event.c | 7 +------ > 1 file changed, 1 insertion(+), 6 deletions(-) > > diff --git a/drivers/iio/industrialio-event.c > b/drivers/iio/industrialio-event.c > index fa6543b..78570c7 100644 > --- a/drivers/iio/industrialio-event.c > +++ b/drivers/iio/industrialio-event.c > @@ -350,15 +350,10 @@ static inline int __iio_add_event_config_attrs(struct > iio_dev *indio_dev) > ret = iio_device_add_event_sysfs(indio_dev, > &indio_dev->channels[j]); > if (ret < 0) > - goto error_clear_attrs; > + return ret; > attrcount += ret; > } > return attrcount; > - > -error_clear_attrs: > - __iio_remove_event_config_attrs(indio_dev); > - > - return ret; > } > > static bool iio_check_for_dynamic_events(struct iio_dev *indio_dev) > > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
