common: check file existence before opening manifest

This fixes a problem when there is a broken symlink with the name
AndroidManifest.xml which will then lead to a crash when trying to open
it for trying to remove the debuggable flag.

diff --git a/fdroidserver/common.py b/fdroidserver/common.py
index 984f4636a4717c32eff192af2a5e3cc16e455cd6..c6fbf0d205453bb06931be64e987ae6a34e4ff33 100644 (file)
--- a/fdroidserver/common.py
+++ b/fdroidserver/common.py
@@ -1210,7 +1210,7 @@ def remove_debuggable_flags(root_dir):
     # Remove forced debuggable flags
     logging.debug("Removing debuggable flags from %s" % root_dir)
     for root, dirs, files in os.walk(root_dir):
-        if 'AndroidManifest.xml' in files:
+        if 'AndroidManifest.xml' in files and os.path.isfile(os.path.join(root, 'AndroidManifest.xml')):
             regsub_file(r'android:debuggable="[^"]*"',
                         '',
                         os.path.join(root, 'AndroidManifest.xml'))