Re: [apparmor] [patch] Fix rttime default unit in RlimitRule

2015-06-26 Thread Steve Beattie
On Fri, Jun 26, 2015 at 11:15:04PM +0200, Christian Boltz wrote: > RlimitRule accidently used 'ms' (milliseconds) as default unit for > rttime rules, but rttime without unit means 'us' (microseconds). This > patch fixes this. > > Also add some tests with 'us' as unit, and two more to cover terribl

Re: [apparmor] [patch] Fix rttime default unit in RlimitRule

2015-06-26 Thread John Johansen
On 06/26/2015 03:15 PM, Christian Boltz wrote: > Hello, > > RlimitRule accidently used 'ms' (milliseconds) as default unit for > rttime rules, but rttime without unit means 'us' (microseconds). This > patch fixes this. > > Also add some tests with 'us' as unit, and two more to cover terribly > in

[apparmor] [patch] Fix rttime default unit in RlimitRule

2015-06-26 Thread Christian Boltz
Hello, RlimitRule accidently used 'ms' (milliseconds) as default unit for rttime rules, but rttime without unit means 'us' (microseconds). This patch fixes this. Also add some tests with 'us' as unit, and two more to cover terribly invalid corner cases (and to improve test coverage by 2 lines ;-)