On Thu, Feb 14, 2013 at 10:26:43AM +0300, Dan Carpenter wrote: > There is a forever loop calling iio_channel_release() because the > "chan < " part of the "chan < ARRAY_SIZE()" is missing. This is in both > the error handling on probe and also in the remove function. > > The other thing is that it's possible for some of the elements of the > adc_bat->channel[chan] array to be an ERR_PTR(). I've changed them to > be NULL instead. We're still not allowed to pass NULLs to > iio_channel_release() so I've added a check. > > Finally, I removed an unused "chan = ARRAY_SIZE(gab_chan_name);" > statement as a small cleanup. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Applied, thanks a lot! Anton -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
