<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-family:monospace"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Oct 4, 2021 at 8:07 AM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Richie,<br>
<br>
> In 32-bit (regardless of kernel bits) glibc and musl will usually<br>
> dereference the timespec pointers and try to read them. In some cases<br>
> this might be avoidable, but they must do it in others.<br>
<br>
> Passing invalid pointers is undefined in POSIX. In any case, AFAICT<br>
> libc would have to catch the signal in order to guarantee EFAULT is<br>
> returned.<br>
<br>
LGTM.<br>
<br>
Reviewed-by: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br>
<br>
> Possibly we shouldn't test things like this at all through libc<br>
> wrappers.<br>
Only for 32bit or also for 64 bit? Anyway, there has always been some cases<br>
where bad addr testing was problematic (e.g. non-intel arch).<br></blockquote><div><br></div><div><div style="font-family:monospace" class="gmail_default">I'd skip it for both, I recall that some implementations</div><div style="font-family:monospace" class="gmail_default">were crashing.<br></div></div><div><br></div><div>Acked-by: Jan Stancek <<a href="mailto:jstancek@redhat.com">jstancek@redhat.com</a>></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Kind regards,<br>
Petr<br>
<br>
-- <br>
Mailing list info: <a href="https://lists.linux.it/listinfo/ltp" rel="noreferrer" target="_blank">https://lists.linux.it/listinfo/ltp</a><br>
<br>
</blockquote></div></div>