Skip site navigation (1)Skip section navigation (2) 
Date:      Thu, 18 May 2017 01:32:50 +0000 (UTC)
From:      Sepherosa Ziehau <sephe@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-11@freebsd.org
Subject:   svn commit: r318433 - stable/11/contrib/hyperv/tools
Message-ID:  <201705180132.v4I1WoL0043234@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help 
Author: sephe
Date: Thu May 18 01:32:50 2017
New Revision: 318433
URL: https://svnweb.freebsd.org/changeset/base/318433

Log:
  MFC 317783
      hyperv/kvp: Fix pool direcrory and file permission
  
      PR:		209385
      Sponsored by:	Microsoft

Modified:
  stable/11/contrib/hyperv/tools/hv_kvp_daemon.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/contrib/hyperv/tools/hv_kvp_daemon.c
==============================================================================
--- stable/11/contrib/hyperv/tools/hv_kvp_daemon.c	Thu May 18 00:32:05 2017	(r318432)
+++ stable/11/contrib/hyperv/tools/hv_kvp_daemon.c	Thu May 18 01:32:50 2017	(r318433)
@@ -61,6 +61,10 @@ typedef uint16_t	__u16;
 typedef uint32_t	__u32;
 typedef uint64_t	__u64;
 
+#define POOL_FILE_MODE	(S_IRUSR | S_IWUSR)
+#define POOL_DIR_MODE	(POOL_FILE_MODE | S_IXUSR)
+#define POOL_DIR	"/var/db/hyperv/pool"
+
 /*
  * ENUM Data
  */
@@ -285,11 +289,12 @@ kvp_file_init(void)
 	int i;
 	int alloc_unit = sizeof(struct kvp_record) * ENTRIES_PER_BLOCK;
 
-	if (mkdir("/var/db/hyperv/pool", S_IRUSR | S_IWUSR | S_IROTH) < 0 &&
+	if (mkdir(POOL_DIR, POOL_DIR_MODE) < 0 &&
 	    (errno != EEXIST && errno != EISDIR)) {
 		KVP_LOG(LOG_ERR, " Failed to create /var/db/hyperv/pool\n");
 		exit(EXIT_FAILURE);
 	}
+	chmod(POOL_DIR, POOL_DIR_MODE); /* fix old mistake */
 
 	for (i = 0; i < HV_KVP_POOL_COUNT; i++)
 	{
@@ -297,11 +302,12 @@ kvp_file_init(void)
 		records_read = 0;
 		num_blocks = 1;
 		snprintf(fname, MAX_FILE_NAME, "/var/db/hyperv/pool/.kvp_pool_%d", i);
-		fd = open(fname, O_RDWR | O_CREAT, S_IRUSR | S_IWUSR | S_IROTH);
+		fd = open(fname, O_RDWR | O_CREAT, POOL_FILE_MODE);
 
 		if (fd == -1) {
 			return (1);
 		}
+		fchmod(fd, POOL_FILE_MODE); /* fix old mistake */
 
 
 		filep = fopen(fname, "r");

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201705180132.v4I1WoL0043234>