Skip to content

Commit 91ae69c

Browse files
olafheringliuw
authored andcommitted
tools: hv: change permissions of NetworkManager configuration file
Align permissions of the resulting .nmconnection file, instead of the input file from hv_kvp_daemon. To avoid the tiny time frame where the output file is world-readable, use umask instead of chmod. Fixes: 42999c9 ("hv/hv_kvp_daemon:Support for keyfile based connection profile") Signed-off-by: Olaf Hering <olaf@aepfle.de> Reviewed-by: Shradha Gupta <shradhagupta@linux.microsoft.com> Link: https://lore.kernel.org/r/20241016143521.3735-1-olaf@aepfle.de Signed-off-by: Wei Liu <wei.liu@kernel.org> Message-ID: <20241016143521.3735-1-olaf@aepfle.de>
1 parent bcc80de commit 91ae69c

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

tools/hv/hv_set_ifconfig.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ echo "ONBOOT=yes" >> $1
8181

8282
cp $1 /etc/sysconfig/network-scripts/
8383

84-
chmod 600 $2
84+
umask 0177
8585
interface=$(echo $2 | awk -F - '{ print $2 }')
8686
filename="${2##*/}"
8787

0 commit comments

Comments
 (0)