[LTP] [PATCH 1/4] net/dhcp: Use paths allowed by AppArmor for dnsmasq
Petr Vorel
pvorel@suse.cz
Fri Oct 12 00:05:22 CEST 2018
Fixes for --log-facility and --dhcp-leasefile.
Path for log file expects AppArmor commit
025c7dc6 ("dnsmasq: Add permission to open log files").
NOTE: AppArmor optimization isn't needed for dhcpd.
Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
Changing path to /var/log require root, but we run most of network tests
under root anyway, at least for network namespaces.
I didn't add TST_NEEDS_ROOT=1, maybe I should.
Kind regards,
Petr
---
testcases/network/dhcp/dnsmasq_tests.sh | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/testcases/network/dhcp/dnsmasq_tests.sh b/testcases/network/dhcp/dnsmasq_tests.sh
index ad5885c84..43961f85f 100755
--- a/testcases/network/dhcp/dnsmasq_tests.sh
+++ b/testcases/network/dhcp/dnsmasq_tests.sh
@@ -9,9 +9,11 @@ dhcp_name="dnsmasq"
. dhcp_lib.sh
+log="/var/log/tst_dnsmasq.log"
+
common_opt="--no-hosts --no-resolv --dhcp-authoritative \
- --log-facility=./tst_dnsmasq.log --interface=$iface0 \
- --dhcp-leasefile=tst_dnsmasq.lease --port=0 --conf-file= "
+ --log-facility=$log --interface=$iface0 \
+ --dhcp-leasefile=/var/lib/misc/dnsmasq.tst.leases --port=0 --conf-file= "
start_dhcp()
{
@@ -33,12 +35,12 @@ start_dhcp6()
cleanup_dhcp()
{
- rm -f tst_dnsmasq.log
+ rm -f $log
}
print_dhcp_log()
{
- cat tst_dnsmasq.log
+ cat $log
}
print_dhcp_version()
--
2.19.0
More information about the ltp
mailing list