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 <[email protected]> Reviewed-by: Shradha Gupta <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Wei Liu <[email protected]> Message-ID: <[email protected]>
1 parent bcc80de commit 91ae69c

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

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)