Handle spaces in exec, and set correct default zed.d

Signed-off-by: Jorgen Lundman <[email protected]>
openzfsonwindows · Nov 20, 2024 · f22537b · f22537b
1 parent 65ae459
commit f22537b
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/cmd/zed/os/windows/zed_exec.c b/cmd/zed/os/windows/zed_exec.c
@@ -203,7 +203,7 @@ launch_process(const char *path, const char *prog, char *env[])
 	fflush(stderr);
 
 	snprintf(cmd, sizeof (cmd),
-	    "powershell.exe -ExecutionPolicy Bypass -File %s", path);
+	    "powershell.exe -ExecutionPolicy Bypass -File \"%s\"", path);
 	/*
 	 * ENV:
 	 * Unix is an array of strings, will last pointer NULL.

diff --git a/cmd/zed/zed.h b/cmd/zed/zed.h
@@ -28,7 +28,11 @@
 /*
  * Absolute path for the default zed zedlet directory.
  */
+#ifdef _WIN32
+#define	ZED_ZEDLET_DIR		SYSCONFDIR "/zed.d"
+#else
 #define	ZED_ZEDLET_DIR		SYSCONFDIR "/zfs/zed.d"
+#endif
 
 /*
  * String prefix for ZED variables passed via environment variables.

diff --git a/include/os/windows/zfs/zfs_config.h b/include/os/windows/zfs/zfs_config.h
@@ -28,7 +28,7 @@
 /* Define to 1 to enabled dmu tx validation */
 /* #undef DEBUG_DMU_TX */
 
-#define	SYSCONFDIR "\\SystemRoot\\System32\\drivers"  // Windwosify me
+#define	SYSCONFDIR ZFSEXECDIR  // Windwosify me
 #define	PKGDATADIR "\\SystemRoot\\System32\\drivers"  // Windwosify me
 
 #define	TEXT_DOMAIN "zfs-windows-user"
@@ -154,7 +154,7 @@
 
 #define	ZFSEXECDIR "C:/Program Files/OpenZFS on Windows"
 
-#define	RUNSTATEDIR ZFSEXECDIR "/zed"
+#define	RUNSTATEDIR ZFSEXECDIR
 #define	_PATH_STDPATH ""
 #define	SBINDIR ZFSEXECDIR