On 10/26/2012 10:59 AM, Steve Beattie wrote:
> This patch fixes two issue with the simple test driver. The first is
> that child exec that actually ran the parser was located inside the
> eval statement. This meant that if the exec failed for some reason
> (like the parser didn't exist), the child wouldn't actually die,
> but would pop out of the eval and continue running through the loop
> of test profiles (while the parent process does the same). This meant
> that if the script ran on the full testsuite with a misconfiguration,
> it would explode creating O(n^2) processes, where n is the number of
> testcase files -- with over 25k testcases, that's a lot. The fis is to
> lift the child exec outside the eval{}, then an exec() failure causes
> the child process to die correctly.
>
> The second fix is that several of the testcases were added with the
> DESCRIPTION field added in lower case (i.e. #=Description blah blah).
> This fix makes the regex that pulls out the description not be
> case-sensitive.
>
> ---
Acked-by: John Johansen <john.johan...@canonical.com>
--
AppArmor mailing list
AppArmor@lists.ubuntu.com
Modify settings or unsubscribe at:
https://lists.ubuntu.com/mailman/listinfo/apparmor
