On Sun, 2016-05-01 at 18:34 +0100, Emil Velikov wrote:
> Ping ?
This faithfully preserves the original behavior, but...
> > +static void
> > +xf86CheckPrivs(const char *option, const char *arg, const char *path_file,
> > + const char *dfault)
> > +{
> > + if (xf86PrivsElevated() && !xf86PathIsSafe(arg)) {
> > + FatalError("\nInvalid argument for %s\n"
> > + "\tWith elevated privileges, the %s specified with %s
> > must\n"
> > + "\tinclude a relative path and must not contain any
> > \"..\"\n"
> > + "\telements.\n"
> > + "\tUsing default %s %s.\n\n",
> > + option, path_file, option, dfault, path_file);
> > + }
> > +}
This message is a lie, we do not in fact use the default, because
FatalError() means we never go any further. Clearly nobody can be
relying on that fallback working, so I'd just rip out the "default" bit
and leave it as a fatal error.
Also if I'm nitpicking, that error text is trash. Suggest "With
elevated privileges, %s must specify a relative path without any \"..\"
elements." % (option).
- ajax
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel
