Yoshinori Sato <ysato@xxxxxxxxxxxxxxxxxxxx> wrote: > #define get_user(x, ptr) \ > ({ \ > + uintptr_t __uip = (uintptr_t)(ptr); \ const? > might_fault(); \ > - access_ok(VERIFY_READ, ptr, sizeof(*ptr)) ? \ > - __get_user(x, ptr) : \ > + access_ok(VERIFY_READ, __uip, sizeof(*ptr)) ? \ > + __get_user((x), (__typeof__(*ptr) *)__uip) : \ > -EFAULT; \ > }) Would it be better to use void* instead of uintptr_t? David -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html