[apparmor] [PATCH v2 08/42] With the auto cleanup changes we can now directly return the value if we have a cache hit.

[Tyler Hicks]

On 2015-03-06 15:48:24, Tyler Hicks wrote:
> From: John Johansen <john.johansen at canonical.com>
> 
> Signed-off-by: John Johansen <john.johansen at canonical.com>
> [tyhicks: Forward ported patch to trunk]
> Signed-off-by: Tyler Hicks <tyhicks at canonical.com>
> ---

Re-reviewed this one and it still looks good.

Tyler

>  parser/parser_main.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/parser/parser_main.c b/parser/parser_main.c
> index 342ee14..6850086 100644
> --- a/parser/parser_main.c
> +++ b/parser/parser_main.c
> @@ -792,7 +792,7 @@ int process_profile(int option, const char *profilename)
>  				PERROR("Cache hit: %s\n", cachename);
>  			retval = process_binary(option, cachename);
>  			if (!retval || skip_bad_cache_rebuild)
> -				goto out;
> +				return retval;
>  		}
>  		if (write_cache) {
>  			/* Otherwise, set up to save a cached copy */
> -- 
> 2.1.4
> 
> 
> -- 
> AppArmor mailing list
> AppArmor at lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20150324/deed00cc/attachment.pgp>