[LTP] [PATCH] open08: calling tst_get_bad_addr from LTP API
Jan Stancek
jstancek@redhat.com
Wed Jul 11 09:54:36 CEST 2018
----- Original Message -----
> On Wed, Jul 11, 2018 at 3:28 PM, Jan Stancek <jstancek@redhat.com> wrote:
> >
> >
> > ----- Original Message -----
> >> Maybe we misled by the code comments before, after thinking over, I
> >> guess the "outside your accessible address space" != "outside of
> >> process mapped space". So we don't need to do map/unmap to satisfy
> >> that.
> >
> > That may be the case, I changed it on purpose to match that comment.
> > Though as you pointed out PROT_NONE might be enough too.
> >
> > Is the motivation for your patch to re-use existing function
>
> Yes, firstly my motivation is to re-use the existing function. But then I
> realized that maybe the original author misunderst
> oo
> d the EFAULT error in open().
>
> So I find open(2) manual and get this:
> "EFAULT pathname points outside your *accessible* address space."
>
> Then I think maybe we need match the manual but not the code comments.
I'd say we match it already (in stronger form than PROT_NONE).
I don't object to re-using LTP function. Can you send v2
that updates also the comment please?
Regards,
Jan
More information about the ltp
mailing list