All of lore.kernel.org
 help / color / mirror / Atom feed
From: Petr Vorel <pvorel@suse.cz>
To: ltp@lists.linux.it
Subject: [LTP] [PATCH v2 2/6] net/dhcp: Use paths allowed by AppArmor for dnsmasq
Date: Tue, 11 Dec 2018 18:00:19 +0100	[thread overview]
Message-ID: <20181211170019.GA23604@dell5510> (raw)
In-Reply-To: <20181207123516.30867-3-pvorel@suse.cz>

Hi,

> Fixes for --log-facility and --dhcp-leasefile.
...

> +++ b/testcases/network/dhcp/dnsmasq_tests.sh
> @@ -8,10 +8,25 @@
>  dhcp_name="dnsmasq"

>  . dhcp_lib.sh
> +TST_SETUP="setup"
> +
> +log="/var/log/tst_dnsmasq.log"
This is going to be
log="/var/log/dnsmasq.tst.log"
As I updated it in upstream co match SELinux path [1].

Lease file isn't handled by SELinux (for AppArmor we use path, which is reserved
for lxc-net). maybe it'd make sense use /var/lib/dnsmasq/ directory, when
SELinux enabled (checked with tst_selinux_enabled() from 5th commit).

Kind regards,
Petr

[1] https://gitlab.com/apparmor/apparmor/commit/3ef8df6ac05057e46720b2eba099bad3416f763b

  reply	other threads:[~2018-12-11 17:00 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-07 12:35 [LTP] [PATCH v2 0/6] DHCP tests and AppArmor/SELinux improvements Petr Vorel
2018-12-07 12:35 ` [LTP] [PATCH v2 1/6] net/dhcp: Rename setup & cleanup functions Petr Vorel
2018-12-07 12:35 ` [LTP] [PATCH v2 2/6] net/dhcp: Use paths allowed by AppArmor for dnsmasq Petr Vorel
2018-12-11 17:00   ` Petr Vorel [this message]
2018-12-07 12:35 ` [LTP] [PATCH v2 3/6] net/dhcp: Move print_dhcp_log() into dhcp library Petr Vorel
2018-12-07 12:35 ` [LTP] [PATCH v2 4/6] ver_linux: Print AppArmor and SELinux status Petr Vorel
2018-12-07 12:35 ` [LTP] [PATCH v2 5/6] shell: Add tst_security.sh helper Petr Vorel
2018-12-12 11:54   ` Alexey Kodanev
2018-12-13 17:38     ` Petr Vorel
2018-12-07 12:35 ` [LTP] [PATCH v2 6/6] net: Add security warning flag to some net tests Petr Vorel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20181211170019.GA23604@dell5510 \
    --to=pvorel@suse.cz \
    --cc=ltp@lists.linux.it \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.