Merge "Fix an overflow found by ASAN."

2014-10-20 21:48:43 +00:00
parent 09fd5a1873 d395f79133
commit e40974d928
1 changed files with 1 additions and 1 deletions
--- a/tools/aapt/Resource.cpp
+++ b/tools/aapt/Resource.cpp
@@ -553,7 +553,7 @@ static int validateAttr(const String8& path, const ResTable& table,
                    String8(parser.getElementName(&len)).string(), attr);
            return ATTR_LEADING_SPACES;
        }
-        if (str[len-1] == ' ') {
+        if (len != 0 && str[len-1] == ' ') {
            fprintf(stderr, "%s:%d: Tag <%s> attribute %s can not end with a space.\n",
                    path.string(), parser.getLineNumber(),
                    String8(parser.getElementName(&len)).string(), attr);