Re: ntdll: add a warning about running wine as root (try 2)

[Vincent Povirk]

The patch does not apply to current master.

Vincent Povirk



On Mon, Feb 2, 2009 at 8:01 PM, Austin English <austinenglish(a)gmail.com> wrote:
> Been seeing it quite a bit recently in the forums. While the patches
> to break using sudo/su have helped a bit, seems some users are
> convinced they need to run wine as root. This adds a non fatal
> message, so no functionality is lost for those that _do_ need to run
> as root.
>
> Try 2:
> Made warning a bit more verbose. Changed geteuid to getuid, since we
> already check for that function, and geteuid is a bit is superfluous,
> since sudo should already be prevented.
>
> --
> -Austin

ntdll: add a warning about running wine as root (try 2)

[Austin English]

On Mon, Feb 2, 2009 at 8:27 PM, Vincent Povirk
<madewokherd+8cd9(a)gmail.com> wrote:
> The patch does not apply to current master.
>
> Vincent Povirk
>
> On Mon, Feb 2, 2009 at 8:01 PM, Austin English <austinenglish(a)gmail.com> wrote:
> > Been seeing it quite a bit recently in the forums. While the patches
> > to break using sudo/su have helped a bit, seems some users are
> > convinced they need to run wine as root. This adds a non fatal
> > message, so no functionality is lost for those that _do_ need to run
> > as root.
> >
> > Try 2:
> > Made warning a bit more verbose. Changed geteuid to getuid, since we
> > already check for that function, and geteuid is a bit is superfluous,
> > since sudo should already be prevented.
> >
> > --
> > -Austin

Thanks. Killed my hard drive this morning, so edited previous diff
rather than making a new one. Resending a fresh one now.

-- 
-Austin