Re: cygrunsrv CWE-428

On 4/24/26 05:56, Corinna Vinschen wrote:
> Wow, that's weird.  No, you're not on the wrong branch or something.
>
> Given how long we don't use these reg keys anymore, you can see how many
> people are actually utilizing remote access :}

That makes me feel better. I thought I was going crazy.

> Would you mind terribly, to send a patch for the above problem as well,
> basically just changing the registry path?

Of course. That patch is attached here.

Thanks a lot to you and William for helping me with this.

Thanks,
James


-- 
Problem reports:      https://cygwin.com/problems.html
FAQ:                  https://cygwin.com/faq/
Documentation:        https://cygwin.com/docs.html
Unsubscribe info:     https://cygwin.com/ml/#unsubscribe-simple

diff --git a/cygrunsrv.h b/cygrunsrv.h
index 7839487..ee4b67a 100644
--- a/cygrunsrv.h
+++ b/cygrunsrv.h
@@ -43,8 +43,8 @@
 #define PARAM_TIMEOUT   "Timeout"
 #define PARAM_X_TIMEOUT "StopTimeout"
 
-#define CYG_ROOT	"SOFTWARE\\Cygnus Solutions\\Cygwin\\mounts v2\\/"
-#define CYG_ROOT_VAL	"native"
+#define CYG_ROOT	"SOFTWARE\\Cygwin\\setup"
+#define CYG_ROOT_VAL	"rootdir"
 
 #define DEF_STDIN_PATH	"/dev/null"
 #define DEF_LOG_PATH	"/var/log/"