On Thu, Jun 26, 2008 at 4:40 PM, David Given <dg@xxxxxxxxxxx> wrote: > /* Get the size of whatever the pointer points to */ > multiply = base->bit_size >> 3; > > This divides the bit size by 8 to get the size in bytes. However, this > doesn't take into account that bits_in_char might not be 8. I think this > should actually be: > > /* Get the size of whatever the pointer points to */ > multiply = base->bit_size / bits_in_char; I don't think so. The offset part of the ptr_add op is mean to be the absolute offset, not how many element of the pointer index. In other words, it is how many address number it need to add. Chris -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
