From 49b6bf70e49022f81b3995b75b30d3cfc087ebc0 Mon Sep 17 00:00:00 2001
From: Connor Tumbleson <connor.tumbleson@gmail.com>
Date: Tue, 23 Sep 2014 07:56:28 -0500
Subject: [PATCH] [skip] check result of .setExecutable()

---
 .../main/java/brut/androlib/res/AndrolibResources.java    | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/brut.apktool/apktool-lib/src/main/java/brut/androlib/res/AndrolibResources.java b/brut.apktool/apktool-lib/src/main/java/brut/androlib/res/AndrolibResources.java
index bc28b39f..ed3ccb56 100644
--- a/brut.apktool/apktool-lib/src/main/java/brut/androlib/res/AndrolibResources.java
+++ b/brut.apktool/apktool-lib/src/main/java/brut/androlib/res/AndrolibResources.java
@@ -818,8 +818,12 @@ final public class AndrolibResources {
         } catch (BrutException ex) {
             throw new AndrolibException(ex);
         }
-        mAaptBinary.setExecutable(true);
-        return mAaptBinary;
+        if (mAaptBinary.setExecutable(true)) {
+            return mAaptBinary;
+        }
+
+        System.err.println("Can't set aapt binary as executable");
+        throw new AndrolibException("Can't set aapt binary as executable");
     }
 
     public File getAndroidResourcesFile() throws AndrolibException {