On Thu, Mar 19, 2015 at 09:47:34PM +0100, Christian Boltz wrote:
> this patch changes serialize_parse_profile_start() to use 
> parse_profile_start() instead of using duplicated code.
> 
> The behaviour is mostly kept, with the exception that the function is 
> more strict now and raises exceptions instead of ignoring errors.
> 
> In practise, this won't change anything because the profiles are parsed
> with parse_profile() (which calls parse_profile_start()) - and that 
> already errors out.
> 
> The tests are updated to match the more strict behaviour.
> 
> 
> The next step would be to drop serialize_parse_profile_start() 
> completely, but this isn't urgent and can/should be done when we have 
> test coverage for serialize_profile_from_old_profile() one day ;-)
> 
> As usual, this patch is for trunk and 2.9.
> 
> 
> [ 27-simplify-serialize_parse_profile_data.diff ]

Acked-by: Steve Beattie <[email protected]> for trunk. Thanks.

-- 
Steve Beattie
<[email protected]>
http://NxNW.org/~steve/

Attachment: signature.asc
Description: Digital signature

-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to