Re: [apparmor] [Patch] Bug 888077 - aliases being partially applied

2013-07-08 Thread John Johansen
On 07/08/2013 08:03 PM, Seth Arnold wrote: > On Mon, Jul 08, 2013 at 02:06:42AM -0700, John Johansen wrote: >> Below is a mostly complete patch for >> https://bugs.launchpad.net/apparmor/+bug/888077 >> >> It is currently missing support for link and mount rules. This patch is >> done against the 2

[apparmor] Follow-up on DBus syntax

2013-07-08 Thread Tyler Hicks
After gathering everyone's opinions[1] on the DBus syntax, John's wider discussion[2] of IPC syntax, and various other conversations, I think we've come to a conclusion on what the DBus syntax should look like. I'll begin adjusting the existing parser patches (along with test cases, documentation,

[apparmor] Monthly IRC meeting

2013-07-08 Thread John Johansen
So its time for the monthly IRC meeting again, and my calendar reminder that is supposed to send an email, didn't yet again. The meeting is tomorrow (July 09, 2013) at 20:00 UTC in #apparmor on irc.oftc.net the agenda, such as it is can be found at http://wiki.apparmor.net/index.php/MeetingAge

Re: [apparmor] [Patch] Bug 888077 - aliases being partially applied

2013-07-08 Thread Seth Arnold
On Mon, Jul 08, 2013 at 02:06:42AM -0700, John Johansen wrote: > Below is a mostly complete patch for > https://bugs.launchpad.net/apparmor/+bug/888077 > > It is currently missing support for link and mount rules. This patch is > done against the 2.8 branch, and the question is, is this something

Re: [apparmor] [Branch ~apparmor-dev/apparmor/master] Rev 2136: Kshitij Gupta found and fixed a problem with Glob and Glob w/ Ext behavior

2013-07-08 Thread Seth Arnold
On Tue, Jul 09, 2013 at 01:35:03AM +0200, Christian Boltz wrote: > Besides some accidential whitespace changes (probably tabs vs. spaces), > I'm wondering about > > > +if (not $newpath ~~ $options) { > vs. > > +if (not $newpat

Re: [apparmor] [Branch ~apparmor-dev/apparmor/master] Rev 2136: Kshitij Gupta found and fixed a problem with Glob and Glob w/ Ext behavior

2013-07-08 Thread Christian Boltz
Hello, Am Freitag, 5. Juli 2013 schrieb nore...@launchpad.net: > revno: 2136 > committer: Seth Arnold > Kshitij Gupta found and fixed a problem with Glob and Glob w/ Ext > behavior in aa-logprof and aa-genprof that would add duplicate > entries to the interactive list of permissions. > >

Re: [apparmor] GSoC r13, r14, r15 review

2013-07-08 Thread Kshitij Gupta
>> Traceback (most recent call last): >> File "../apparmor/severity.py", line 76, in __init__ >> resource, severity = line.split() >> ValueError: need more than 1 value to unpack >> >> During handling of the above exception, another exception occurred: >> >> Traceback (most recent call last):

[apparmor] Fwd: GSoC r13, r14, r15 review

2013-07-08 Thread Kshitij Gupta
Hello, Those were a few lenghty commits, great to hear you worked on them. Thanks. :-) On Mon, Jul 8, 2013 at 5:40 AM, Christian Boltz wrote: > Hello, > > I just reviewed the changes from r13, r14 and r15. See the attached > file - I added several inline comments in the diff. > Response is embed

Re: [apparmor] [Patch] Bug 888077 - aliases being partially applied

2013-07-08 Thread John Johansen
On 07/08/2013 08:06 AM, Jamie Strandboge wrote: > On 07/08/2013 09:38 AM, John Johansen wrote: >> On 07/08/2013 07:15 AM, Jamie Strandboge wrote: >>> On 07/08/2013 04:06 AM, John Johansen wrote: Below is a mostly complete patch for https://bugs.launchpad.net/apparmor/+bug/888077 >>>

Re: [apparmor] [Patch] Bug 888077 - aliases being partially applied

2013-07-08 Thread Jamie Strandboge
On 07/08/2013 09:38 AM, John Johansen wrote: > On 07/08/2013 07:15 AM, Jamie Strandboge wrote: >> On 07/08/2013 04:06 AM, John Johansen wrote: >>> Below is a mostly complete patch for >>> https://bugs.launchpad.net/apparmor/+bug/888077 >>> >>> It is currently missing support for link and mount rul

Re: [apparmor] [Patch] Bug 888077 - aliases being partially applied

2013-07-08 Thread John Johansen
On 07/08/2013 07:15 AM, Jamie Strandboge wrote: > On 07/08/2013 04:06 AM, John Johansen wrote: >> Below is a mostly complete patch for >> https://bugs.launchpad.net/apparmor/+bug/888077 >> >> It is currently missing support for link and mount rules. This patch is >> done against the 2.8 branch, an

Re: [apparmor] [Patch] Bug 888077 - aliases being partially applied

2013-07-08 Thread John Johansen
On 07/08/2013 04:59 AM, Christian Boltz wrote: > Hello, > > Am Montag, 8. Juli 2013 schrieb John Johansen: >> Below is a mostly complete patch for >> https://bugs.launchpad.net/apparmor/+bug/888077 >> >> It is currently missing support for link and mount rules. This patch >> is done against the 2.

Re: [apparmor] [Patch] Bug 888077 - aliases being partially applied

2013-07-08 Thread Jamie Strandboge
On 07/08/2013 04:06 AM, John Johansen wrote: > Below is a mostly complete patch for > https://bugs.launchpad.net/apparmor/+bug/888077 > > It is currently missing support for link and mount rules. This patch is > done against the 2.8 branch, and the question is, is this something we > want in 2.8.

Re: [apparmor] [Patch] Bug 888077 - aliases being partially applied

2013-07-08 Thread Christian Boltz
Hello, Am Montag, 8. Juli 2013 schrieb John Johansen: > Below is a mostly complete patch for > https://bugs.launchpad.net/apparmor/+bug/888077 > > It is currently missing support for link and mount rules. This patch > is done against the 2.8 branch, and the question is, is this > something we wan

Re: [apparmor] [PATCH 1/3] Add new fontconfig configuration snippets path to the fonts abstraction.

2013-07-08 Thread Christian Boltz
Hello, Am Sonntag, 7. Juli 2013 schrieb Seth Arnold: > On Sat, Jul 06, 2013 at 10:47:44AM +0200, intrigeri+deb...@boum.org > wrote: > > At least on Debian, with recent versions of fontconfig-config > > (>= 2.10), files in /etc/fonts/conf.d/ are symlinks pointing to > > /usr/share/fontconfig/. > >

[apparmor] GSoC r13, r14, r15 review

2013-07-08 Thread Christian Boltz
Hello, I just reviewed the changes from r13, r14 and r15. See the attached file - I added several inline comments in the diff. Besides that, I got a nice traceback: # python3 severity_test.py E. == ERROR: testInvalid (__main_

[apparmor] [Patch] Bug 888077 - aliases being partially applied

2013-07-08 Thread John Johansen
Below is a mostly complete patch for https://bugs.launchpad.net/apparmor/+bug/888077 It is currently missing support for link and mount rules. This patch is done against the 2.8 branch, and the question is, is this something we want in 2.8.2/3. It is rather large, and I want to rework it some bef