vpnor: partition: Add write permissions to writable files
The code preserves the file permissions when the partition files are
copied to the writeable area. Currently the file permissions are 664,
but if the squashfs was built with file permissions set to 444 to
make it explicit that the pnor partition files were read-only, the
files in the writable area would also have read only permissions.
Therefore, explicitly add write permissions to the partition files
that are copied to the writable area.
Tested:
- With current permissions (there's no change):
root@witherspoon-YL30UF74T02M:~# ls -l /var/lib/phosphor-software-manager/pnor/ro/HBD
-rw-rw-r-- 1 253760 401855 1179648 Sep 23 16:30 HBD
root@witherspoon-YL30UF74T02M:~#ls -l /var/lib/phosphor-software-manager/pnor/rw/HBD
-rw-rw-r-- 1 root root 1179648 Oct 9 18:38 HBD
- With read-only permissions:
root@witherspoon-YL30UF74T02M:~# ls -l /var/lib/phosphor-software-manager/pnor/ro/HBD
-r--r--r-- 1 root root 1179648 Oct 9 15:33 /var/lib/phosphor-software-manager/pnor/ro/HBD
root@witherspoon-YL30UF74T02M:~# ls -l /var/lib/phosphor-software-manager/pnor/rw/HBD
-rw-r--r-- 1 root root 1179648 Oct 9 18:43 /var/lib/phosphor-software-manager/pnor/rw/HBD
Change-Id: Ibc3f47bd3a4f645d99155d19624c7271df4d0470
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
diff --git a/vpnor/partition.cpp b/vpnor/partition.cpp
index 900004b..66c51d1 100644
--- a/vpnor/partition.cpp
+++ b/vpnor/partition.cpp
@@ -91,6 +91,7 @@
dst /= partition.data.name;
fs::copy_file(src, dst);
+ fs::permissions(dst, fs::perms::add_perms | fs::perms::owner_write);
return dst;
}