From 9d96397cef3d75f2af96b7a31bf2aef910346b0c Mon Sep 17 00:00:00 2001 From: Hiltjo Posthuma Date: Thu, 23 Aug 2018 19:11:52 +0200 Subject: xml: remove TODO comments and add a note --- xml.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/xml.c b/xml.c index a34a12c..c3a0f7c 100644 --- a/xml.c +++ b/xml.c @@ -350,8 +350,9 @@ xml_parse(XMLParser *x) if (c == '!') { /* cdata and comments */ for (tagdatalen = 0; (c = x->getnext()) != EOF;) { - if (tagdatalen <= sizeof("[CDATA[") - 1) /* if (d < sizeof(x->data)) */ - x->data[tagdatalen++] = c; /* TODO: prevent overflow */ + /* NOTE: sizeof(x->data) must be atleast sizeof("[CDATA[") */ + if (tagdatalen <= sizeof("[CDATA[") - 1) + x->data[tagdatalen++] = c; if (c == '>') break; else if (c == '-' && tagdatalen == sizeof("--") - 1 && @@ -381,7 +382,7 @@ xml_parse(XMLParser *x) x->isshorttag = ispi; taglen = 1; while ((c = x->getnext()) != EOF) { - if (c == '/') /* TODO: simplify short tag? */ + if (c == '/') x->isshorttag = 1; /* short tag */ else if (c == '>' || isspace(c)) { x->tag[taglen] = '\0'; -- cgit v1.2.3