On Tue, Dec 17, 2013 at 12:06:01PM +0100, Alexander Aring wrote: > On Tue, Dec 17, 2013 at 06:58:12AM -0400, Anderson Lizardo wrote: > > Hi Alexander, > > > > On Tue, Dec 17, 2013 at 6:32 AM, Alexander Aring <alex.aring@xxxxxxxxx> wrote: > > > Sometimes a nullpointer dereferencing occurs because of using a wrong > > > pointer arithmetic in udp_uncompression. > > > > > > This patch changes "**(hc06_ptr + 3)" to the right one "*(*hc06_ptr + > > > 3)". Dereferencing like "**(hc06_ptr + 3)" works in a random case only. > > > > And why not use hc06_ptr[0][3] ? IMHO it is more readable and the > > arithmetic is the same (as far as I know). > > > > mhh maybe we change it to *hc06_ptr[3] ? Otherwise we have always > something like [0][#] for access. > meant (*hc06_ptr)[3] here, but I have a better idea to add a function wich sets some data and increment it automatically. Will send v3 with that soon. - Alex -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
