[PATCH] setup: allow running as non-admin

Christopher Faylor cgf-use-the-mailinglist-please@cygwin.com
Thu Nov 7 00:23:00 GMT 2013


On Thu, Nov 07, 2013 at 10:52:00AM +1100, Shaddy Baddah wrote:
>Hi,
>
>As discussed, please find the patch to allow setup to be run as a
>non-admin user include clear text.
>
>-- 
>Regards,
>Shaddy
>

Only a very minor comment:

>+
>+        // Note, this is necessary to avoid an infinite loop.
>+        // The understanding is that pre-Vista, the runas verb will not

This indentation looks wrong.

>+		// result in a privilege elevated process. Therefore we need to
>+		// indicate to the forked process that it should be happy with
>+		// whatever privileges it is run with.

cgf



More information about the Cygwin-apps mailing list