Mark Wielaard <> changed:

           What    |Removed                     |Added
                 CC|                            |

--- Comment #4 from Mark Wielaard <> ---
(In reply to Keno Fischer from comment #3)
> > If I interpret this correctly, ARG3 is a C style zero-terminated string 
> > that the syscall
> > reads.  Is that correct?
> Essentially yes, though with the caveat that if there's no NUL after
> PAGE_SIZE-1 bytes, it'll still accept that as far as I know (as a
> PAGE_SIZE-1 sized string), since it uses strndup_user(str, PAGE_SIZE) on the
> kernel side of things.

That seems a funny corner case that I cannot imagine anybody relies on (the
caller would have to explicitly not pass a zero terminated string larger than
PAGE_SIZE, which is different on different arches). If someone does I think a
warning from valgrind seems somewhat justified. So just using PRE_MEM_RASCIIZ
here should be fine.

You are receiving this mail because:
You are watching all bug changes.

Reply via email to