On Fri, Feb 24, 2017 at 11:17:44PM +0100, Greg Kurz wrote: > On Fri, 24 Feb 2017 17:17:30 +0000 > Stefan Hajnoczi <stefa...@redhat.com> wrote: > [...] > > > > This function doesn't handle absolute paths? It ignores leading '/' and > > > > therefore treats all paths as relative paths. > > > > > > > > > > Yes because any path coming from the client is supposed (*) to be > > > relative to the > > > shared directory and openat(2) says: > > > > Please change the function name since this isn't openat with nofollow > > behavior, it's a subset of openat that only takes relative paths with > > nofollow behavior. > > In the v2, this function is only called by local_open_nofollow() actually. > Maybe I should move the stripping of leading '/' characters there ?
As long as the function name is clear then I'm happy. If it has different semantics from openat() then it should have a different name (e.g. relative_openat_nofollow()). Stefan
signature.asc
Description: PGP signature