2011/7/23 Antti Palosaari <crope@xxxxxx>: > On 07/23/2011 02:31 AM, Antti Palosaari wrote: >> >> On 07/23/2011 02:01 AM, HoP wrote: >>> >>> 2011/7/23 Antti Palosaari<crope@xxxxxx>: >>>> >>>> But now I see what you mean. msg2[1] is set as garbage fields in case of >>>> incoming msg len is 1. True, but it does not harm since it is not >>>> used in >>>> that case. >>> >>> In case of write, cxd2820r_tuner_i2c_xfer() gets msg[] parameter >>> with only one element, true? If so, then my patch is correct. >> >> Yes it is true but nonsense. It is also wrong to make always msg2 as two >> element array too, but those are just simpler and generates most likely >> some code less. Could you see it can cause problem in some case? > > Now I thought it more, could it crash if it point out of memory area? I see you finally understood what I wanted to do :-) I'm surprised that it not crashed already. I thought I have to missed something. Honza -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
