On Mon, Mar 11, 2013 at 02:04:51PM -0700, Kees Cook wrote: > Probably not. It just seemed like the existing comment was > insufficient after the removal of the redundant VERIFY_READ check that > happened recently. That's certainly true, the remaining comment is a little cryptic. Something more like: /* We need to check that we can read the entire relocation array, but * as we may need to update the presumed_offsets upon execution, we * need to check now for full write access. */ -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel