summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorGravatar Nikias Bassen2017-01-01 21:23:15 +0100
committerGravatar Nikias Bassen2017-01-01 21:23:15 +0100
commitc2ea55aa6cac25934c08a5408e6ad23ab69e824a (patch)
treed04ac91ca6919cbb310d4263bd50b9e0e687aa44 /src
parentbe2b1690a3ecb97620f01669b1b8ac556181db48 (diff)
downloadlibplist-c2ea55aa6cac25934c08a5408e6ad23ab69e824a.tar.gz
libplist-c2ea55aa6cac25934c08a5408e6ad23ab69e824a.tar.bz2
xplist: Fix numerical character entity conversion
Diffstat (limited to 'src')
-rw-r--r--src/xplist.c8
1 files changed, 6 insertions, 2 deletions
diff --git a/src/xplist.c b/src/xplist.c
index 68fd3ab..11b7400 100644
--- a/src/xplist.c
+++ b/src/xplist.c
@@ -639,6 +639,7 @@ static int unescape_entities(char *str, size_t *length)
}
if (str+i >= entp+1) {
int entlen = str+i - entp;
+ int bytelen = 1;
if (!strncmp(entp, "amp", 3)) {
/* the '&' is already there */
} else if (!strncmp(entp, "apos", 4)) {
@@ -682,17 +683,20 @@ static int unescape_entities(char *str, size_t *length)
*(entp+1) = (char)(0x80 + ((val >> 6) & 0x3F));
*(entp+2) = (char)(0x80 + (val & 0x3F));
entp+=3;
+ bytelen = 4;
} else if (val >= 0x800) {
/* three bytes */
*(entp-1) = (char)(0xE0 + ((val >> 12) & 0xF));
*(entp+0) = (char)(0x80 + ((val >> 6) & 0x3F));
*(entp+1) = (char)(0x80 + (val & 0x3F));
entp+=2;
+ bytelen = 3;
} else if (val >= 0x80) {
/* two bytes */
*(entp-1) = (char)(0xC0 + ((val >> 6) & 0x1F));
*(entp+0) = (char)(0x80 + (val & 0x3F));
entp++;
+ bytelen = 2;
} else {
/* one byte */
*(entp-1) = (char)(val & 0x7F);
@@ -702,8 +706,8 @@ static int unescape_entities(char *str, size_t *length)
return -1;
}
memmove(entp, str+i+1, len - i);
- i -= entlen;
- len -= (entlen+1);
+ i -= entlen+1 - bytelen;
+ len -= entlen+2 - bytelen;
continue;
}
}