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

Tyler Hicks tyhicks at canonical.com
Sat Dec 6 00:22:06 UTC 2014


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>
---
 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 c7f8f33..7ef74e7 100644
--- a/parser/parser_main.c
+++ b/parser/parser_main.c
@@ -773,7 +773,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.0




More information about the AppArmor mailing list