[XMLLITE]
authorAmine Khaldi <amine.khaldi@reactos.org>
Tue, 26 Mar 2013 19:02:38 +0000 (19:02 +0000)
committerAmine Khaldi <amine.khaldi@reactos.org>
Tue, 26 Mar 2013 19:02:38 +0000 (19:02 +0000)
* Sync with Wine 1.5.26.

svn path=/trunk/; revision=58609

reactos/dll/win32/xmllite/reader.c
reactos/dll/win32/xmllite/xmllite_main.c
reactos/media/doc/README.WINE

index 4a13ea3..9dbc96e 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * IXmlReader implementation
  *
- * Copyright 2010, 2012 Nikolay Sivov
+ * Copyright 2010, 2012-2013 Nikolay Sivov
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ -48,11 +48,56 @@ typedef enum
     XmlEncoding_Unknown
 } xml_encoding;
 
+typedef enum
+{
+    XmlReadInState_Initial,
+    XmlReadInState_XmlDecl,
+    XmlReadInState_Misc_DTD,
+    XmlReadInState_DTD,
+    XmlReadInState_DTD_Misc,
+    XmlReadInState_Element,
+    XmlReadInState_Content,
+    XmlReadInState_MiscEnd
+} XmlReaderInternalState;
+
+/* This state denotes where parsing was interrupted by input problem.
+   Reader resumes parsing using this information. */
+typedef enum
+{
+    XmlReadResumeState_Initial,
+    XmlReadResumeState_PITarget,
+    XmlReadResumeState_PIBody,
+    XmlReadResumeState_CDATA,
+    XmlReadResumeState_Comment,
+    XmlReadResumeState_STag
+} XmlReaderResumeState;
+
+/* saved pointer index to resume from particular input position */
+typedef enum
+{
+    XmlReadResume_Name,  /* PITarget, name for NCName, prefix for QName */
+    XmlReadResume_Local, /* local for QName */
+    XmlReadResume_Body,  /* PI body, comment text, CDATA text */
+    XmlReadResume_Last
+} XmlReaderResume;
+
+typedef enum
+{
+    StringValue_LocalName,
+    StringValue_QualifiedName,
+    StringValue_Value,
+    StringValue_Last
+} XmlReaderStringValue;
+
 static const WCHAR utf16W[] = {'U','T','F','-','1','6',0};
 static const WCHAR utf8W[] = {'U','T','F','-','8',0};
 
 static const WCHAR dblquoteW[] = {'\"',0};
 static const WCHAR quoteW[] = {'\'',0};
+static const WCHAR ltW[] = {'<',0};
+static const WCHAR gtW[] = {'>',0};
+static const WCHAR commentW[] = {'<','!','-','-',0};
+static const WCHAR piW[] = {'<','?',0};
 
 struct xml_encoding_data
 {
@@ -76,7 +121,7 @@ typedef struct
 
 typedef struct input_buffer input_buffer;
 
-typedef struct _xmlreaderinput
+typedef struct
 {
     IXmlReaderInput IXmlReaderInput_iface;
     LONG ref;
@@ -91,14 +136,29 @@ typedef struct _xmlreaderinput
        optimizations possible with IStream aren't implemented */
     ISequentialStream *stream;
     input_buffer *buffer;
+    unsigned int pending : 1;
 } xmlreaderinput;
 
+static const struct IUnknownVtbl xmlreaderinputvtbl;
+
+/* Structure to hold parsed string of specific length.
+
+   Reader stores node value as 'start' pointer, on request
+   a null-terminated version of it is allocated.
+
+   To init a strval variable use reader_init_strval(),
+   to set strval as a reader value use reader_set_strval().
+ */
 typedef struct
 {
-    const WCHAR *str;
-    UINT len;
+    WCHAR *start; /* input position where value starts */
+    UINT len;     /* length in WCHARs, altered after ReadValueChunk */
+    WCHAR *str;   /* allocated null-terminated string */
 } strval;
 
+static WCHAR emptyW[] = {0};
+static const strval strval_empty = {emptyW, 0, emptyW};
+
 struct attribute
 {
     struct list entry;
@@ -106,19 +166,31 @@ struct attribute
     strval value;
 };
 
-typedef struct _xmlreader
+struct element
+{
+    struct list entry;
+    strval qname;
+};
+
+typedef struct
 {
     IXmlReader IXmlReader_iface;
     LONG ref;
     xmlreaderinput *input;
     IMalloc *imalloc;
     XmlReadState state;
+    XmlReaderInternalState instate;
+    XmlReaderResumeState resumestate;
     XmlNodeType nodetype;
     DtdProcessing dtdmode;
     UINT line, pos;           /* reader position in XML stream */
     struct list attrs; /* attributes list for current node */
     struct attribute *attr; /* current attribute */
     UINT attr_count;
+    struct list elements;
+    strval strvalues[StringValue_Last];
+    UINT depth;
+    WCHAR *resume[XmlReadResume_Last]; /* pointers used to resume reader */
 } xmlreader;
 
 struct input_buffer
@@ -174,6 +246,21 @@ static inline void reader_free(xmlreader *reader, void *mem)
     m_free(reader->imalloc, mem);
 }
 
+static HRESULT reader_strvaldup(xmlreader *reader, const strval *src, strval *dest)
+{
+    *dest = *src;
+
+    if (src->str != strval_empty.str)
+    {
+        dest->str = reader_alloc(reader, (dest->len+1)*sizeof(WCHAR));
+        if (!dest->str) return E_OUTOFMEMORY;
+        memcpy(dest->str, src->str, dest->len*sizeof(WCHAR));
+        dest->str[dest->len] = 0;
+    }
+
+    return S_OK;
+}
+
 /* reader input memory allocation functions */
 static inline void *readerinput_alloc(xmlreaderinput *input, size_t len)
 {
@@ -233,6 +320,134 @@ static HRESULT reader_add_attr(xmlreader *reader, strval *localname, strval *val
     return S_OK;
 }
 
+/* This one frees stored string value if needed */
+static void reader_free_strvalued(xmlreader *reader, strval *v)
+{
+    if (v->str != strval_empty.str)
+    {
+        reader_free(reader, v->str);
+        *v = strval_empty;
+    }
+}
+
+static inline void reader_init_strvalue(WCHAR *str, UINT len, strval *v)
+{
+    v->start = v->str = str;
+    v->len = len;
+}
+
+static void reader_free_strvalue(xmlreader *reader, XmlReaderStringValue type)
+{
+    reader_free_strvalued(reader, &reader->strvalues[type]);
+}
+
+static void reader_free_strvalues(xmlreader *reader)
+{
+    int type;
+    for (type = 0; type < StringValue_Last; type++)
+        reader_free_strvalue(reader, type);
+}
+
+/* This helper should only be used to test if strings are the same,
+   it doesn't try to sort. */
+static inline int strval_eq(const strval *str1, const strval *str2)
+{
+    if (str1->len != str2->len) return 0;
+    return !memcmp(str1->str, str2->str, str1->len*sizeof(WCHAR));
+}
+
+static void reader_clear_elements(xmlreader *reader)
+{
+    struct element *elem, *elem2;
+    LIST_FOR_EACH_ENTRY_SAFE(elem, elem2, &reader->elements, struct element, entry)
+    {
+        reader_free_strvalued(reader, &elem->qname);
+        reader_free(reader, elem);
+    }
+    list_init(&reader->elements);
+}
+
+static HRESULT reader_inc_depth(xmlreader *reader)
+{
+    /* FIXME: handle XmlReaderProperty_MaxElementDepth property */
+    reader->depth++;
+    return S_OK;
+}
+
+static HRESULT reader_push_element(xmlreader *reader, strval *qname)
+{
+    struct element *elem;
+    HRESULT hr;
+
+    elem = reader_alloc(reader, sizeof(*elem));
+    if (!elem) return E_OUTOFMEMORY;
+
+    hr = reader_strvaldup(reader, qname, &elem->qname);
+    if (FAILED(hr)) return hr;
+
+    if (!list_empty(&reader->elements))
+    {
+        hr = reader_inc_depth(reader);
+        if (FAILED(hr)) return hr;
+    }
+
+    list_add_head(&reader->elements, &elem->entry);
+    return hr;
+}
+
+static void reader_pop_element(xmlreader *reader)
+{
+    struct element *elem = LIST_ENTRY(list_head(&reader->elements), struct element, entry);
+
+    if (elem)
+    {
+        list_remove(&elem->entry);
+        reader_free_strvalued(reader, &elem->qname);
+        reader_free(reader, elem);
+    }
+}
+
+/* Always make a copy, cause strings are supposed to be null terminated. Null pointer for 'value'
+   means node value is to be determined. */
+static void reader_set_strvalue(xmlreader *reader, XmlReaderStringValue type, const strval *value)
+{
+    strval *v = &reader->strvalues[type];
+
+    reader_free_strvalue(reader, type);
+    if (!value)
+    {
+        v->str = NULL;
+        v->start = NULL;
+        v->len = 0;
+        return;
+    }
+
+    if (value->str == strval_empty.str)
+        *v = *value;
+    else
+    {
+        if (type == StringValue_Value)
+        {
+            /* defer allocation for value string */
+            v->str = NULL;
+            v->start = value->start;
+            v->len = value->len;
+        }
+        else
+        {
+            v->str = reader_alloc(reader, (value->len + 1)*sizeof(WCHAR));
+            memcpy(v->str, value->start, value->len*sizeof(WCHAR));
+            v->str[value->len] = 0;
+            v->len = value->len;
+        }
+    }
+}
+
+static inline int is_reader_pending(xmlreader *reader)
+{
+    return reader->input->pending;
+}
+
 static HRESULT init_encoded_buffer(xmlreaderinput *input, encoded_buffer *buffer)
 {
     const int initial_len = 0x2000;
@@ -356,10 +571,12 @@ static HRESULT readerinput_query_for_stream(xmlreaderinput *readerinput)
 static HRESULT readerinput_growraw(xmlreaderinput *readerinput)
 {
     encoded_buffer *buffer = &readerinput->buffer->encoded;
-    ULONG len = buffer->allocated - buffer->written, read;
+    /* to make sure aligned length won't exceed allocated length */
+    ULONG len = buffer->allocated - buffer->written - 4;
+    ULONG read;
     HRESULT hr;
 
-    /* always try to get aligned to 4 bytes, so the only case we can get partialy read characters is
+    /* always try to get aligned to 4 bytes, so the only case we can get partially read characters is
        variable width encodings like UTF-8 */
     len = (len + 3) & ~3;
     /* try to use allocated space or grow */
@@ -370,9 +587,11 @@ static HRESULT readerinput_growraw(xmlreaderinput *readerinput)
         len = buffer->allocated - buffer->written;
     }
 
+    read = 0;
     hr = ISequentialStream_Read(readerinput->stream, buffer->data + buffer->written, len, &read);
-    if (FAILED(hr)) return hr;
     TRACE("requested %d, read %d, ret 0x%08x\n", len, read, hr);
+    readerinput->pending = hr == E_PENDING;
+    if (FAILED(hr)) return hr;
     buffer->written += read;
 
     return hr;
@@ -392,23 +611,48 @@ static void readerinput_grow(xmlreaderinput *readerinput, int length)
     }
 }
 
+static inline int readerinput_is_utf8(xmlreaderinput *readerinput)
+{
+    static char startA[] = {'<','?'};
+    static char commentA[] = {'<','!'};
+    encoded_buffer *buffer = &readerinput->buffer->encoded;
+    unsigned char *ptr = (unsigned char*)buffer->data;
+
+    return !memcmp(buffer->data, startA, sizeof(startA)) ||
+           !memcmp(buffer->data, commentA, sizeof(commentA)) ||
+           /* test start byte */
+           (ptr[0] == '<' &&
+            (
+             (ptr[1] && (ptr[1] <= 0x7f)) ||
+             (buffer->data[1] >> 5) == 0x6  || /* 2 bytes */
+             (buffer->data[1] >> 4) == 0xe  || /* 3 bytes */
+             (buffer->data[1] >> 3) == 0x1e)   /* 4 bytes */
+           );
+}
+
 static HRESULT readerinput_detectencoding(xmlreaderinput *readerinput, xml_encoding *enc)
 {
     encoded_buffer *buffer = &readerinput->buffer->encoded;
-    static char startA[] = {'<','?','x','m'};
     static WCHAR startW[] = {'<','?'};
+    static WCHAR commentW[] = {'<','!'};
     static char utf8bom[] = {0xef,0xbb,0xbf};
     static char utf16lebom[] = {0xff,0xfe};
 
     *enc = XmlEncoding_Unknown;
 
-    if (buffer->written <= 3) return MX_E_INPUTEND;
+    if (buffer->written <= 3)
+    {
+        HRESULT hr = readerinput_growraw(readerinput);
+        if (FAILED(hr)) return hr;
+        if (buffer->written <= 3) return MX_E_INPUTEND;
+    }
 
     /* try start symbols if we have enough data to do that, input buffer should contain
        first chunk already */
-    if (!memcmp(buffer->data, startA, sizeof(startA)))
+    if (readerinput_is_utf8(readerinput))
         *enc = XmlEncoding_UTF8;
-    else if (!memcmp(buffer->data, startW, sizeof(startW)))
+    else if (!memcmp(buffer->data, startW, sizeof(startW)) ||
+             !memcmp(buffer->data, commentW, sizeof(commentW)))
         *enc = XmlEncoding_UTF16;
     /* try with BOM now */
     else if (!memcmp(buffer->data, utf8bom, sizeof(utf8bom)))
@@ -440,20 +684,41 @@ static int readerinput_get_utf8_convlen(xmlreaderinput *readerinput)
     return len;
 }
 
-/* returns byte length of complete char sequence for specified code page, */
-static int readerinput_get_convlen(xmlreaderinput *readerinput, UINT cp)
+/* Returns byte length of complete char sequence for buffer code page,
+   it's relative to current buffer position which is currently used for BOM handling
+   only. */
+static int readerinput_get_convlen(xmlreaderinput *readerinput)
 {
     encoded_buffer *buffer = &readerinput->buffer->encoded;
-    int len = buffer->written;
+    int len;
 
-    if (cp == CP_UTF8)
+    if (readerinput->buffer->code_page == CP_UTF8)
         len = readerinput_get_utf8_convlen(readerinput);
     else
         len = buffer->written;
 
+    TRACE("%d\n", len - (int)(buffer->cur - buffer->data));
     return len - (buffer->cur - buffer->data);
 }
 
+/* It's possible that raw buffer has some leftovers from last conversion - some char
+   sequence that doesn't represent a full code point. Length argument should be calculated with
+   readerinput_get_convlen(), if it's -1 it will be calculated here. */
+static void readerinput_shrinkraw(xmlreaderinput *readerinput, int len)
+{
+    encoded_buffer *buffer = &readerinput->buffer->encoded;
+
+    if (len == -1)
+        len = readerinput_get_convlen(readerinput);
+
+    memmove(buffer->data, buffer->cur + (buffer->written - len), len);
+    /* everything below cur is lost too */
+    buffer->written -= len + (buffer->cur - buffer->data);
+    /* after this point we don't need cur pointer really,
+       it's used only to mark where actual data begins when first chunk is read */
+    buffer->cur = buffer->data;
+}
+
 /* note that raw buffer content is kept */
 static void readerinput_switchencoding(xmlreaderinput *readerinput, xml_encoding enc)
 {
@@ -461,12 +726,14 @@ static void readerinput_switchencoding(xmlreaderinput *readerinput, xml_encoding
     encoded_buffer *dest = &readerinput->buffer->utf16;
     int len, dest_len;
     HRESULT hr;
+    WCHAR *ptr;
     UINT cp;
 
     hr = get_code_page(enc, &cp);
     if (FAILED(hr)) return;
 
-    len = readerinput_get_convlen(readerinput, cp);
+    readerinput->buffer->code_page = cp;
+    len = readerinput_get_convlen(readerinput);
 
     TRACE("switching to cp %d\n", cp);
 
@@ -475,35 +742,81 @@ static void readerinput_switchencoding(xmlreaderinput *readerinput, xml_encoding
     {
         readerinput_grow(readerinput, len);
         memcpy(dest->data, src->cur, len);
-        readerinput->buffer->code_page = cp;
+        dest->written += len*sizeof(WCHAR);
         return;
     }
 
     dest_len = MultiByteToWideChar(cp, 0, src->cur, len, NULL, 0);
     readerinput_grow(readerinput, dest_len);
-    MultiByteToWideChar(cp, 0, src->cur, len, (WCHAR*)dest->data, dest_len);
-    dest->data[dest_len] = 0;
-    readerinput->buffer->code_page = cp;
+    ptr = (WCHAR*)dest->data;
+    MultiByteToWideChar(cp, 0, src->cur, len, ptr, dest_len);
+    ptr[dest_len] = 0;
+    dest->written += dest_len*sizeof(WCHAR);
 }
 
-static inline const WCHAR *reader_get_cur(xmlreader *reader)
+/* shrinks parsed data a buffer begins with */
+static void reader_shrink(xmlreader *reader)
 {
-    return (WCHAR*)reader->input->buffer->utf16.cur;
+    encoded_buffer *buffer = &reader->input->buffer->utf16;
+
+    /* avoid to move too often using threshold shrink length */
+    if (buffer->cur - buffer->data > buffer->written / 2)
+    {
+        buffer->written -= buffer->cur - buffer->data;
+        memmove(buffer->data, buffer->cur, buffer->written);
+        buffer->cur = buffer->data;
+        *(WCHAR*)&buffer->cur[buffer->written] = 0;
+    }
 }
 
-static int reader_cmp(xmlreader *reader, const WCHAR *str)
+/* This is a normal way for reader to get new data converted from raw buffer to utf16 buffer.
+   It won't attempt to shrink but will grow destination buffer if needed */
+static HRESULT reader_more(xmlreader *reader)
 {
-    const WCHAR *ptr = reader_get_cur(reader);
-    int i = 0;
+    xmlreaderinput *readerinput = reader->input;
+    encoded_buffer *src = &readerinput->buffer->encoded;
+    encoded_buffer *dest = &readerinput->buffer->utf16;
+    UINT cp = readerinput->buffer->code_page;
+    int len, dest_len;
+    HRESULT hr;
+    WCHAR *ptr;
 
-    return strncmpW(str, ptr, strlenW(str));
+    /* get some raw data from stream first */
+    hr = readerinput_growraw(readerinput);
+    len = readerinput_get_convlen(readerinput);
 
-    while (str[i]) {
-        if (ptr[i] != str[i]) return 0;
-        i++;
+    /* just copy for UTF-16 case */
+    if (cp == ~0)
+    {
+        readerinput_grow(readerinput, len);
+        memcpy(dest->data, src->cur, len);
+        dest->written += len*sizeof(WCHAR);
+        return hr;
     }
 
-    return 1;
+    dest_len = MultiByteToWideChar(cp, 0, src->cur, len, NULL, 0);
+    readerinput_grow(readerinput, dest_len);
+    ptr = (WCHAR*)dest->data;
+    MultiByteToWideChar(cp, 0, src->cur, len, ptr, dest_len);
+    ptr[dest_len] = 0;
+    dest->written += dest_len*sizeof(WCHAR);
+    /* get rid of processed data */
+    readerinput_shrinkraw(readerinput, len);
+
+    return hr;
+}
+
+static inline WCHAR *reader_get_cur(xmlreader *reader)
+{
+    WCHAR *ptr = (WCHAR*)reader->input->buffer->utf16.cur;
+    if (!*ptr) reader_more(reader);
+    return ptr;
+}
+
+static int reader_cmp(xmlreader *reader, const WCHAR *str)
+{
+    const WCHAR *ptr = reader_get_cur(reader);
+    return strncmpW(str, ptr, strlenW(str));
 }
 
 /* moves cursor n WCHARs forward */
@@ -519,13 +832,18 @@ static void reader_skipn(xmlreader *reader, int n)
     }
 }
 
+static inline int is_wchar_space(WCHAR ch)
+{
+    return ch == ' ' || ch == '\t' || ch == '\r' || ch == '\n';
+}
+
 /* [3] S ::= (#x20 | #x9 | #xD | #xA)+ */
 static int reader_skipspaces(xmlreader *reader)
 {
     encoded_buffer *buffer = &reader->input->buffer->utf16;
     const WCHAR *ptr = reader_get_cur(reader), *start = ptr;
 
-    while (*ptr == ' ' || *ptr == '\t' || *ptr == '\r' || *ptr == '\n')
+    while (is_wchar_space(*ptr))
     {
         buffer->cur += sizeof(WCHAR);
         if (*ptr == '\r')
@@ -546,7 +864,7 @@ static int reader_skipspaces(xmlreader *reader)
 /* [26] VersionNum ::= '1.' [0-9]+ */
 static HRESULT reader_parse_versionnum(xmlreader *reader, strval *val)
 {
-    const WCHAR *ptr, *ptr2, *start = reader_get_cur(reader);
+    WCHAR *ptr, *ptr2, *start = reader_get_cur(reader);
     static const WCHAR onedotW[] = {'1','.',0};
 
     if (reader_cmp(reader, onedotW)) return WC_E_XMLDECL;
@@ -559,8 +877,7 @@ static HRESULT reader_parse_versionnum(xmlreader *reader, strval *val)
 
     if (ptr2 == ptr) return WC_E_DIGIT;
     TRACE("version=%s\n", debugstr_wn(start, ptr-start));
-    val->str = start;
-    val->len = ptr-start;
+    reader_init_strvalue(start, ptr-start, val);
     reader_skipn(reader, ptr-ptr2);
     return S_OK;
 }
@@ -587,8 +904,7 @@ static HRESULT reader_parse_versioninfo(xmlreader *reader)
     if (!reader_skipspaces(reader)) return WC_E_WHITESPACE;
 
     if (reader_cmp(reader, versionW)) return WC_E_XMLDECL;
-    name.str = reader_get_cur(reader);
-    name.len = 7;
+    reader_init_strvalue(reader_get_cur(reader), 7, &name);
     /* skip 'version' */
     reader_skipn(reader, 7);
 
@@ -625,7 +941,7 @@ static inline int is_wchar_encname(WCHAR ch)
 /* [81] EncName ::= [A-Za-z] ([A-Za-z0-9._] | '-')* */
 static HRESULT reader_parse_encname(xmlreader *reader, strval *val)
 {
-    const WCHAR *start = reader_get_cur(reader), *ptr;
+    WCHAR *start = reader_get_cur(reader), *ptr;
     xml_encoding enc;
     int len;
 
@@ -691,15 +1007,14 @@ static HRESULT reader_parse_sddecl(xmlreader *reader)
     static const WCHAR standaloneW[] = {'s','t','a','n','d','a','l','o','n','e',0};
     static const WCHAR yesW[] = {'y','e','s',0};
     static const WCHAR noW[] = {'n','o',0};
-    const WCHAR *start, *ptr;
+    WCHAR *start, *ptr;
     strval name, val;
     HRESULT hr;
 
     if (!reader_skipspaces(reader)) return WC_E_WHITESPACE;
 
     if (reader_cmp(reader, standaloneW)) return S_FALSE;
-    name.str = reader_get_cur(reader);
-    name.len = 10;
+    reader_init_strvalue(reader_get_cur(reader), 10, &name);
     /* skip 'standalone' */
     reader_skipn(reader, 10);
 
@@ -733,11 +1048,11 @@ static HRESULT reader_parse_sddecl(xmlreader *reader)
 /* [23] XMLDecl ::= '<?xml' VersionInfo EncodingDecl? SDDecl? S? '?>' */
 static HRESULT reader_parse_xmldecl(xmlreader *reader)
 {
-    static const WCHAR xmldeclW[] = {'<','?','x','m','l',0};
+    static const WCHAR xmldeclW[] = {'<','?','x','m','l',' ',0};
     static const WCHAR declcloseW[] = {'?','>',0};
     HRESULT hr;
 
-    /* check if we have "<?xml" */
+    /* check if we have "<?xml " */
     if (reader_cmp(reader, xmldeclW)) return S_FALSE;
 
     reader_skipn(reader, 5);
@@ -757,111 +1072,1046 @@ static HRESULT reader_parse_xmldecl(xmlreader *reader)
     if (reader_cmp(reader, declcloseW)) return WC_E_XMLDECL;
     reader_skipn(reader, 2);
 
+    reader->nodetype = XmlNodeType_XmlDeclaration;
+    reader_set_strvalue(reader, StringValue_LocalName, &strval_empty);
+    reader_set_strvalue(reader, StringValue_QualifiedName, &strval_empty);
+    reader_set_strvalue(reader, StringValue_Value, &strval_empty);
+
     return S_OK;
 }
 
-static HRESULT WINAPI xmlreader_QueryInterface(IXmlReader *iface, REFIID riid, void** ppvObject)
+/* [15] Comment ::= '<!--' ((Char - '-') | ('-' (Char - '-')))* '-->' */
+static HRESULT reader_parse_comment(xmlreader *reader)
 {
-    xmlreader *This = impl_from_IXmlReader(iface);
-
-    TRACE("%p %s %p\n", This, debugstr_guid(riid), ppvObject);
+    WCHAR *start, *ptr;
 
-    if (IsEqualGUID(riid, &IID_IUnknown) ||
-        IsEqualGUID(riid, &IID_IXmlReader))
+    if (reader->resume[XmlReadResume_Body])
     {
-        *ppvObject = iface;
+        start = reader->resume[XmlReadResume_Body];
+        ptr = reader_get_cur(reader);
     }
     else
     {
-        FIXME("interface %s not implemented\n", debugstr_guid(riid));
-        return E_NOINTERFACE;
+        /* skip '<!--' */
+        reader_skipn(reader, 4);
+        reader_shrink(reader);
+        ptr = start = reader_get_cur(reader);
+        reader->nodetype = XmlNodeType_Comment;
+        reader->resume[XmlReadResume_Body] = start;
+        reader->resumestate = XmlReadResumeState_Comment;
+        reader_set_strvalue(reader, StringValue_LocalName, NULL);
+        reader_set_strvalue(reader, StringValue_QualifiedName, NULL);
+        reader_set_strvalue(reader, StringValue_Value, NULL);
     }
 
-    IXmlReader_AddRef(iface);
+    /* will exit when there's no more data, it won't attempt to
+       read more from stream */
+    while (*ptr)
+    {
+        if (ptr[0] == '-')
+        {
+            if (ptr[1] == '-')
+            {
+                if (ptr[2] == '>')
+                {
+                    strval value;
+
+                    TRACE("%s\n", debugstr_wn(start, ptr-start));
+                    /* skip '-->' */
+                    reader_skipn(reader, 3);
+                    reader_init_strvalue(start, ptr-start, &value);
+                    reader_set_strvalue(reader, StringValue_LocalName, &strval_empty);
+                    reader_set_strvalue(reader, StringValue_QualifiedName, &strval_empty);
+                    reader_set_strvalue(reader, StringValue_Value, &value);
+                    reader->resume[XmlReadResume_Body] = NULL;
+                    reader->resumestate = XmlReadResumeState_Initial;
+                    return S_OK;
+                }
+                else
+                    return WC_E_COMMENT;
+            }
+            else
+                ptr++;
+        }
+        else
+        {
+            reader_skipn(reader, 1);
+            ptr++;
+        }
+    }
 
     return S_OK;
 }
 
-static ULONG WINAPI xmlreader_AddRef(IXmlReader *iface)
-{
-    xmlreader *This = impl_from_IXmlReader(iface);
-    ULONG ref = InterlockedIncrement(&This->ref);
-    TRACE("(%p)->(%d)\n", This, ref);
-    return ref;
-}
-
-static ULONG WINAPI xmlreader_Release(IXmlReader *iface)
-{
-    xmlreader *This = impl_from_IXmlReader(iface);
-    LONG ref = InterlockedDecrement(&This->ref);
+/* [2] Char ::= #x9 | #xA | #xD | [#x20-#xD7FF] | [#xE000-#xFFFD] | [#x10000-#x10FFFF] */
+static inline int is_char(WCHAR ch)
+{
+    return (ch == '\t') || (ch == '\r') || (ch == '\n') ||
+           (ch >= 0x20 && ch <= 0xd7ff) ||
+           (ch >= 0xd800 && ch <= 0xdbff) || /* high surrogate */
+           (ch >= 0xdc00 && ch <= 0xdfff) || /* low surrogate */
+           (ch >= 0xe000 && ch <= 0xfffd);
+}
+
+/* [13] PubidChar ::= #x20 | #xD | #xA | [a-zA-Z0-9] | [-'()+,./:=?;!*#@$_%] */
+static inline int is_pubchar(WCHAR ch)
+{
+    return (ch == ' ') ||
+           (ch >= 'a' && ch <= 'z') ||
+           (ch >= 'A' && ch <= 'Z') ||
+           (ch >= '0' && ch <= '9') ||
+           (ch >= '-' && ch <= ';') || /* '()*+,-./:; */
+           (ch == '=') || (ch == '?') ||
+           (ch == '@') || (ch == '!') ||
+           (ch >= '#' && ch <= '%') || /* #$% */
+           (ch == '_') || (ch == '\r') || (ch == '\n');
+}
+
+static inline int is_namestartchar(WCHAR ch)
+{
+    return (ch == ':') || (ch >= 'A' && ch <= 'Z') ||
+           (ch == '_') || (ch >= 'a' && ch <= 'z') ||
+           (ch >= 0xc0   && ch <= 0xd6)   ||
+           (ch >= 0xd8   && ch <= 0xf6)   ||
+           (ch >= 0xf8   && ch <= 0x2ff)  ||
+           (ch >= 0x370  && ch <= 0x37d)  ||
+           (ch >= 0x37f  && ch <= 0x1fff) ||
+           (ch >= 0x200c && ch <= 0x200d) ||
+           (ch >= 0x2070 && ch <= 0x218f) ||
+           (ch >= 0x2c00 && ch <= 0x2fef) ||
+           (ch >= 0x3001 && ch <= 0xd7ff) ||
+           (ch >= 0xd800 && ch <= 0xdbff) || /* high surrogate */
+           (ch >= 0xdc00 && ch <= 0xdfff) || /* low surrogate */
+           (ch >= 0xf900 && ch <= 0xfdcf) ||
+           (ch >= 0xfdf0 && ch <= 0xfffd);
+}
+
+/* [4 NS] NCName ::= Name - (Char* ':' Char*) */
+static inline int is_ncnamechar(WCHAR ch)
+{
+    return (ch >= 'A' && ch <= 'Z') ||
+           (ch == '_') || (ch >= 'a' && ch <= 'z') ||
+           (ch == '-') || (ch == '.') ||
+           (ch >= '0'    && ch <= '9')    ||
+           (ch == 0xb7)                   ||
+           (ch >= 0xc0   && ch <= 0xd6)   ||
+           (ch >= 0xd8   && ch <= 0xf6)   ||
+           (ch >= 0xf8   && ch <= 0x2ff)  ||
+           (ch >= 0x300  && ch <= 0x36f)  ||
+           (ch >= 0x370  && ch <= 0x37d)  ||
+           (ch >= 0x37f  && ch <= 0x1fff) ||
+           (ch >= 0x200c && ch <= 0x200d) ||
+           (ch >= 0x203f && ch <= 0x2040) ||
+           (ch >= 0x2070 && ch <= 0x218f) ||
+           (ch >= 0x2c00 && ch <= 0x2fef) ||
+           (ch >= 0x3001 && ch <= 0xd7ff) ||
+           (ch >= 0xd800 && ch <= 0xdbff) || /* high surrogate */
+           (ch >= 0xdc00 && ch <= 0xdfff) || /* low surrogate */
+           (ch >= 0xf900 && ch <= 0xfdcf) ||
+           (ch >= 0xfdf0 && ch <= 0xfffd);
+}
+
+static inline int is_namechar(WCHAR ch)
+{
+    return (ch == ':') || is_ncnamechar(ch);
+}
+
+/* [4] NameStartChar ::= ":" | [A-Z] | "_" | [a-z] | [#xC0-#xD6] | [#xD8-#xF6] | [#xF8-#x2FF] | [#x370-#x37D] |
+                            [#x37F-#x1FFF] | [#x200C-#x200D] | [#x2070-#x218F] | [#x2C00-#x2FEF] | [#x3001-#xD7FF] |
+                            [#xF900-#xFDCF] | [#xFDF0-#xFFFD] | [#x10000-#xEFFFF]
+   [4a] NameChar ::= NameStartChar | "-" | "." | [0-9] | #xB7 | [#x0300-#x036F] | [#x203F-#x2040]
+   [5]  Name     ::= NameStartChar (NameChar)* */
+static HRESULT reader_parse_name(xmlreader *reader, strval *name)
+{
+    WCHAR *ptr, *start;
+
+    if (reader->resume[XmlReadResume_Name])
+    {
+        start = reader->resume[XmlReadResume_Name];
+        ptr = reader_get_cur(reader);
+    }
+    else
+    {
+        ptr = start = reader_get_cur(reader);
+        if (!is_namestartchar(*ptr)) return WC_E_NAMECHARACTER;
+    }
 
-    TRACE("(%p)->(%d)\n", This, ref);
+    while (is_namechar(*ptr))
+    {
+        reader_skipn(reader, 1);
+        ptr = reader_get_cur(reader);
+    }
 
-    if (ref == 0)
+    if (is_reader_pending(reader))
     {
-        IMalloc *imalloc = This->imalloc;
-        if (This->input) IUnknown_Release(&This->input->IXmlReaderInput_iface);
-        reader_clear_attrs(This);
-        reader_free(This, This);
-        if (imalloc) IMalloc_Release(imalloc);
+        reader->resume[XmlReadResume_Name] = start;
+        return E_PENDING;
     }
+    else
+        reader->resume[XmlReadResume_Name] = NULL;
 
-    return ref;
+    TRACE("name %s:%d\n", debugstr_wn(start, ptr-start), (int)(ptr-start));
+    reader_init_strvalue(start, ptr-start, name);
+
+    return S_OK;
 }
 
-static HRESULT WINAPI xmlreader_SetInput(IXmlReader* iface, IUnknown *input)
+/* [17] PITarget ::= Name - (('X' | 'x') ('M' | 'm') ('L' | 'l')) */
+static HRESULT reader_parse_pitarget(xmlreader *reader, strval *target)
 {
-    xmlreader *This = impl_from_IXmlReader(iface);
+    static const WCHAR xmlW[] = {'x','m','l'};
+    strval name;
     HRESULT hr;
+    UINT i;
 
-    TRACE("(%p %p)\n", This, input);
+    hr = reader_parse_name(reader, &name);
+    if (FAILED(hr)) return is_reader_pending(reader) ? E_PENDING : WC_E_PI;
 
-    if (This->input)
+    /* now that we got name check for illegal content */
+    if (name.len == 3 && !strncmpiW(name.str, xmlW, 3))
+        return WC_E_LEADINGXML;
+
+    /* PITarget can't be a qualified name */
+    for (i = 0; i < name.len; i++)
+        if (name.str[i] == ':')
+            return i ? NC_E_NAMECOLON : WC_E_PI;
+
+    TRACE("pitarget %s:%d\n", debugstr_wn(name.str, name.len), name.len);
+    *target = name;
+    return S_OK;
+}
+
+/* [16] PI ::= '<?' PITarget (S (Char* - (Char* '?>' Char*)))? '?>' */
+static HRESULT reader_parse_pi(xmlreader *reader)
+{
+    WCHAR *ptr, *start;
+    strval target;
+    HRESULT hr;
+
+    switch (reader->resumestate)
     {
-        readerinput_release_stream(This->input);
-        IUnknown_Release(&This->input->IXmlReaderInput_iface);
-        This->input = NULL;
+    case XmlReadResumeState_Initial:
+        /* skip '<?' */
+        reader_skipn(reader, 2);
+        reader_shrink(reader);
+        reader->resumestate = XmlReadResumeState_PITarget;
+    case XmlReadResumeState_PITarget:
+        hr = reader_parse_pitarget(reader, &target);
+        if (FAILED(hr)) return hr;
+        reader->resumestate = XmlReadResumeState_PIBody;
+    default:
+        ;
     }
 
-    This->line = This->pos = 0;
-
-    /* just reset current input */
-    if (!input)
+    ptr = reader_get_cur(reader);
+    /* exit earlier if there's no content */
+    if (ptr[0] == '?' && ptr[1] == '>')
     {
-        This->state = XmlReadState_Initial;
+        /* skip '?>' */
+        reader_skipn(reader, 2);
+        reader->nodetype = XmlNodeType_ProcessingInstruction;
+        reader->resumestate = XmlReadResumeState_Initial;
+        reader_set_strvalue(reader, StringValue_LocalName, &target);
+        reader_set_strvalue(reader, StringValue_QualifiedName, &target);
+        reader_set_strvalue(reader, StringValue_Value, &strval_empty);
         return S_OK;
     }
 
-    /* now try IXmlReaderInput, ISequentialStream, IStream */
-    hr = IUnknown_QueryInterface(input, &IID_IXmlReaderInput, (void**)&This->input);
-    if (hr != S_OK)
+    if (!reader->resume[XmlReadResume_Body])
     {
-        IXmlReaderInput *readerinput;
-
-        /* create IXmlReaderInput basing on supplied interface */
-        hr = CreateXmlReaderInputWithEncodingName(input,
-                                         NULL, NULL, FALSE, NULL, &readerinput);
-        if (hr != S_OK) return hr;
-        This->input = impl_from_IXmlReaderInput(readerinput);
+        /* now at least a single space char should be there */
+        if (!is_wchar_space(*ptr)) return WC_E_WHITESPACE;
+        reader_skipspaces(reader);
+        ptr = start = reader_get_cur(reader);
+        reader->resume[XmlReadResume_Body] = start;
+    }
+    else
+    {
+        start = reader->resume[XmlReadResume_Body];
+        ptr = reader_get_cur(reader);
     }
 
-    /* set stream for supplied IXmlReaderInput */
-    hr = readerinput_query_for_stream(This->input);
-    if (hr == S_OK)
-        This->state = XmlReadState_Initial;
+    while (*ptr)
+    {
+        if (ptr[0] == '?')
+        {
+            if (ptr[1] == '>')
+            {
+                strval value;
+
+                TRACE("%s\n", debugstr_wn(start, ptr-start));
+                /* skip '?>' */
+                reader_skipn(reader, 2);
+                reader->nodetype = XmlNodeType_ProcessingInstruction;
+                reader->resumestate = XmlReadResumeState_Initial;
+                reader->resume[XmlReadResume_Body] = NULL;
+                reader_init_strvalue(start, ptr-start, &value);
+                reader_set_strvalue(reader, StringValue_LocalName, &target);
+                reader_set_strvalue(reader, StringValue_QualifiedName, &target);
+                reader_set_strvalue(reader, StringValue_Value, &value);
+                return S_OK;
+            }
+            else
+            {
+                ptr++;
+                reader_more(reader);
+            }
+        }
+        else
+        {
+            reader_skipn(reader, 1);
+            ptr = reader_get_cur(reader);
+        }
+    }
 
-    return hr;
+    return S_OK;
 }
 
-static HRESULT WINAPI xmlreader_GetProperty(IXmlReader* iface, UINT property, LONG_PTR *value)
+/* This one is used to parse significant whitespace nodes, like in Misc production */
+static HRESULT reader_parse_whitespace(xmlreader *reader)
 {
-    xmlreader *This = impl_from_IXmlReader(iface);
+    WCHAR *start, *ptr;
 
-    TRACE("(%p %u %p)\n", This, property, value);
+    reader_shrink(reader);
+    start = reader_get_cur(reader);
 
-    if (!value) return E_INVALIDARG;
+    reader_skipspaces(reader);
+    ptr = reader_get_cur(reader);
+    TRACE("%s\n", debugstr_wn(start, ptr-start));
 
-    switch (property)
+    reader->nodetype = XmlNodeType_Whitespace;
+    reader_set_strvalue(reader, StringValue_LocalName, &strval_empty);
+    reader_set_strvalue(reader, StringValue_QualifiedName, &strval_empty);
+    reader_set_strvalue(reader, StringValue_Value, &strval_empty);
+    return S_OK;
+}
+
+/* [27] Misc ::= Comment | PI | S */
+static HRESULT reader_parse_misc(xmlreader *reader)
+{
+    HRESULT hr = S_FALSE;
+
+    if (reader->resumestate != XmlReadResumeState_Initial)
+    {
+        hr = reader_more(reader);
+        if (FAILED(hr)) return hr;
+
+        /* finish current node */
+        switch (reader->resumestate)
+        {
+        case XmlReadResumeState_PITarget:
+        case XmlReadResumeState_PIBody:
+            return reader_parse_pi(reader);
+        case XmlReadResumeState_Comment:
+            return reader_parse_comment(reader);
+        default:
+            ERR("unknown resume state %d\n", reader->resumestate);
+        }
+    }
+
+    while (1)
+    {
+        const WCHAR *cur = reader_get_cur(reader);
+
+        if (is_wchar_space(*cur))
+            hr = reader_parse_whitespace(reader);
+        else if (!reader_cmp(reader, commentW))
+            hr = reader_parse_comment(reader);
+        else if (!reader_cmp(reader, piW))
+            hr = reader_parse_pi(reader);
+        else
+            break;
+
+        if (hr != S_FALSE) return hr;
+    }
+
+    return hr;
+}
+
+/* [11] SystemLiteral ::= ('"' [^"]* '"') | ("'" [^']* "'") */
+static HRESULT reader_parse_sys_literal(xmlreader *reader, strval *literal)
+{
+    WCHAR *start = reader_get_cur(reader), *cur, quote;
+
+    if (*start != '"' && *start != '\'') return WC_E_QUOTE;
+
+    quote = *start;
+    reader_skipn(reader, 1);
+
+    cur = start = reader_get_cur(reader);
+    while (is_char(*cur) && *cur != quote)
+    {
+        reader_skipn(reader, 1);
+        cur = reader_get_cur(reader);
+    }
+    if (*cur == quote) reader_skipn(reader, 1);
+
+    literal->str = start;
+    literal->len = cur-start;
+    TRACE("%s\n", debugstr_wn(start, cur-start));
+    return S_OK;
+}
+
+/* [12] PubidLiteral ::= '"' PubidChar* '"' | "'" (PubidChar - "'")* "'"
+   [13] PubidChar ::= #x20 | #xD | #xA | [a-zA-Z0-9] | [-'()+,./:=?;!*#@$_%] */
+static HRESULT reader_parse_pub_literal(xmlreader *reader, strval *literal)
+{
+    WCHAR *start = reader_get_cur(reader), *cur, quote;
+
+    if (*start != '"' && *start != '\'') return WC_E_QUOTE;
+
+    quote = *start;
+    reader_skipn(reader, 1);
+
+    cur = start;
+    while (is_pubchar(*cur) && *cur != quote)
+    {
+        reader_skipn(reader, 1);
+        cur = reader_get_cur(reader);
+    }
+
+    reader_init_strvalue(start, cur-start, literal);
+    TRACE("%s\n", debugstr_wn(start, cur-start));
+    return S_OK;
+}
+
+/* [75] ExternalID ::= 'SYSTEM' S SystemLiteral | 'PUBLIC' S PubidLiteral S SystemLiteral */
+static HRESULT reader_parse_externalid(xmlreader *reader)
+{
+    static WCHAR systemW[] = {'S','Y','S','T','E','M',0};
+    static WCHAR publicW[] = {'P','U','B','L','I','C',0};
+    strval name;
+    HRESULT hr;
+    int cnt;
+
+    if (reader_cmp(reader, systemW))
+    {
+        if (reader_cmp(reader, publicW))
+            return S_FALSE;
+        else
+        {
+            strval pub;
+
+            /* public id */
+            reader_skipn(reader, 6);
+            cnt = reader_skipspaces(reader);
+            if (!cnt) return WC_E_WHITESPACE;
+
+            hr = reader_parse_pub_literal(reader, &pub);
+            if (FAILED(hr)) return hr;
+
+            reader_init_strvalue(publicW, strlenW(publicW), &name);
+            return reader_add_attr(reader, &name, &pub);
+        }
+    }
+    else
+    {
+        strval sys;
+
+        /* system id */
+        reader_skipn(reader, 6);
+        cnt = reader_skipspaces(reader);
+        if (!cnt) return WC_E_WHITESPACE;
+
+        hr = reader_parse_sys_literal(reader, &sys);
+        if (FAILED(hr)) return hr;
+
+        reader_init_strvalue(systemW, strlenW(systemW), &name);
+        return reader_add_attr(reader, &name, &sys);
+    }
+
+    return hr;
+}
+
+/* [28] doctypedecl ::= '<!DOCTYPE' S Name (S ExternalID)? S? ('[' intSubset ']' S?)? '>' */
+static HRESULT reader_parse_dtd(xmlreader *reader)
+{
+    static const WCHAR doctypeW[] = {'<','!','D','O','C','T','Y','P','E',0};
+    strval name;
+    WCHAR *cur;
+    HRESULT hr;
+
+    /* check if we have "<!DOCTYPE" */
+    if (reader_cmp(reader, doctypeW)) return S_FALSE;
+    reader_shrink(reader);
+
+    /* DTD processing is not allowed by default */
+    if (reader->dtdmode == DtdProcessing_Prohibit) return WC_E_DTDPROHIBITED;
+
+    reader_skipn(reader, 9);
+    if (!reader_skipspaces(reader)) return WC_E_WHITESPACE;
+
+    /* name */
+    hr = reader_parse_name(reader, &name);
+    if (FAILED(hr)) return WC_E_DECLDOCTYPE;
+
+    reader_skipspaces(reader);
+
+    hr = reader_parse_externalid(reader);
+    if (FAILED(hr)) return hr;
+
+    reader_skipspaces(reader);
+
+    cur = reader_get_cur(reader);
+    if (*cur != '>')
+    {
+        FIXME("internal subset parsing not implemented\n");
+        return E_NOTIMPL;
+    }
+
+    /* skip '>' */
+    reader_skipn(reader, 1);
+
+    reader->nodetype = XmlNodeType_DocumentType;
+    reader_set_strvalue(reader, StringValue_LocalName, &name);
+    reader_set_strvalue(reader, StringValue_QualifiedName, &name);
+
+    return S_OK;
+}
+
+/* [11 NS] LocalPart ::= NCName */
+static HRESULT reader_parse_local(xmlreader *reader, strval *local)
+{
+    WCHAR *ptr, *start;
+
+    if (reader->resume[XmlReadResume_Local])
+    {
+        start = reader->resume[XmlReadResume_Local];
+        ptr = reader_get_cur(reader);
+    }
+    else
+    {
+        ptr = start = reader_get_cur(reader);
+    }
+
+    while (is_ncnamechar(*ptr))
+    {
+        reader_skipn(reader, 1);
+        ptr = reader_get_cur(reader);
+    }
+
+    if (is_reader_pending(reader))
+    {
+         reader->resume[XmlReadResume_Local] = start;
+         return E_PENDING;
+    }
+    else
+         reader->resume[XmlReadResume_Local] = NULL;
+
+    reader_init_strvalue(start, ptr-start, local);
+
+    return S_OK;
+}
+
+/* [7 NS]  QName ::= PrefixedName | UnprefixedName
+   [8 NS]  PrefixedName ::= Prefix ':' LocalPart
+   [9 NS]  UnprefixedName ::= LocalPart
+   [10 NS] Prefix ::= NCName */
+static HRESULT reader_parse_qname(xmlreader *reader, strval *prefix, strval *local, strval *qname)
+{
+    WCHAR *ptr, *start;
+    HRESULT hr;
+
+    if (reader->resume[XmlReadResume_Name])
+    {
+        start = reader->resume[XmlReadResume_Name];
+        ptr = reader_get_cur(reader);
+    }
+    else
+    {
+        ptr = start = reader_get_cur(reader);
+        reader->resume[XmlReadResume_Name] = start;
+        if (!is_ncnamechar(*ptr)) return NC_E_QNAMECHARACTER;
+    }
+
+    if (reader->resume[XmlReadResume_Local])
+    {
+        hr = reader_parse_local(reader, local);
+        if (FAILED(hr)) return hr;
+
+        reader_init_strvalue(reader->resume[XmlReadResume_Name],
+                             local->start - reader->resume[XmlReadResume_Name] - 1,
+                             prefix);
+    }
+    else
+    {
+        /* skip prefix part */
+        while (is_ncnamechar(*ptr))
+        {
+            reader_skipn(reader, 1);
+            ptr = reader_get_cur(reader);
+        }
+
+        if (is_reader_pending(reader)) return E_PENDING;
+
+        /* got a qualified name */
+        if (*ptr == ':')
+        {
+            reader_init_strvalue(start, ptr-start, prefix);
+
+            /* skip ':' */
+            reader_skipn(reader, 1);
+            hr = reader_parse_local(reader, local);
+            if (FAILED(hr)) return hr;
+        }
+        else
+        {
+            reader_init_strvalue(reader->resume[XmlReadResume_Name], ptr-reader->resume[XmlReadResume_Name], local);
+            reader_init_strvalue(NULL, 0, prefix);
+        }
+    }
+
+    reader_init_strvalue(start, ptr-start, local);
+
+    if (prefix->len)
+        TRACE("qname %s:%s\n", debugstr_wn(prefix->start, prefix->len), debugstr_wn(local->start, local->len));
+    else
+        TRACE("ncname %s\n", debugstr_wn(local->start, local->len));
+
+    reader_init_strvalue(prefix->start ? prefix->start : local->start,
+                        /* count ':' too */
+                        (prefix->len ? prefix->len + 1 : 0) + local->len,
+                         qname);
+
+    reader->resume[XmlReadResume_Name] = NULL;
+    reader->resume[XmlReadResume_Local] = NULL;
+
+    return S_OK;
+}
+
+/* [12 NS] STag ::= '<' QName (S Attribute)* S? '>'
+   [14 NS] EmptyElemTag ::= '<' QName (S Attribute)* S? '/>' */
+static HRESULT reader_parse_stag(xmlreader *reader, strval *prefix, strval *local, strval *qname, int *empty)
+{
+    static const WCHAR endW[] = {'/','>',0};
+    HRESULT hr;
+
+    hr = reader_parse_qname(reader, prefix, local, qname);
+    if (FAILED(hr)) return hr;
+
+    reader_skipspaces(reader);
+
+    /* empty element */
+    if ((*empty = !reader_cmp(reader, endW)))
+    {
+        /* skip '/>' */
+        reader_skipn(reader, 2);
+        return S_OK;
+    }
+
+    /* got a start tag */
+    if (!reader_cmp(reader, gtW))
+    {
+        /* skip '>' */
+        reader_skipn(reader, 1);
+        return reader_push_element(reader, qname);
+    }
+
+    FIXME("only empty elements/start tags without attribute list supported\n");
+    return E_NOTIMPL;
+}
+
+/* [39] element ::= EmptyElemTag | STag content ETag */
+static HRESULT reader_parse_element(xmlreader *reader)
+{
+    HRESULT hr;
+
+    switch (reader->resumestate)
+    {
+    case XmlReadResumeState_Initial:
+        /* check if we are really on element */
+        if (reader_cmp(reader, ltW)) return S_FALSE;
+
+        /* skip '<' */
+        reader_skipn(reader, 1);
+
+        reader_shrink(reader);
+        reader->resumestate = XmlReadResumeState_STag;
+    case XmlReadResumeState_STag:
+    {
+        strval qname, prefix, local;
+        int empty = 0;
+
+        /* this handles empty elements too */
+        hr = reader_parse_stag(reader, &prefix, &local, &qname, &empty);
+        if (FAILED(hr)) return hr;
+
+        /* FIXME: need to check for defined namespace to reject invalid prefix,
+           currently reject all prefixes */
+        if (prefix.len) return NC_E_UNDECLAREDPREFIX;
+
+        /* if we got empty element and stack is empty go straight to Misc */
+        if (empty && list_empty(&reader->elements))
+            reader->instate = XmlReadInState_MiscEnd;
+        else
+            reader->instate = XmlReadInState_Content;
+
+        reader->nodetype = XmlNodeType_Element;
+        reader->resumestate = XmlReadResumeState_Initial;
+        reader_set_strvalue(reader, StringValue_LocalName, &local);
+        reader_set_strvalue(reader, StringValue_QualifiedName, &qname);
+        break;
+    }
+    default:
+        hr = E_FAIL;
+    }
+
+    return hr;
+}
+
+/* [13 NS] ETag ::= '</' QName S? '>' */
+static HRESULT reader_parse_endtag(xmlreader *reader)
+{
+    strval prefix, local, qname;
+    struct element *elem;
+    HRESULT hr;
+
+    /* skip '</' */
+    reader_skipn(reader, 2);
+
+    hr = reader_parse_qname(reader, &prefix, &local, &qname);
+    if (FAILED(hr)) return hr;
+
+    reader_skipspaces(reader);
+
+    if (reader_cmp(reader, gtW)) return WC_E_GREATERTHAN;
+
+    /* skip '>' */
+    reader_skipn(reader, 1);
+
+    /* Element stack should never be empty at this point, cause we shouldn't get to
+       content parsing if it's empty. */
+    elem = LIST_ENTRY(list_head(&reader->elements), struct element, entry);
+    if (!strval_eq(&elem->qname, &qname)) return WC_E_ELEMENTMATCH;
+
+    reader_pop_element(reader);
+
+    reader->nodetype = XmlNodeType_EndElement;
+    reader_set_strvalue(reader, StringValue_LocalName, &local);
+    reader_set_strvalue(reader, StringValue_QualifiedName, &qname);
+
+    return S_OK;
+}
+
+/* [18] CDSect ::= CDStart CData CDEnd
+   [19] CDStart ::= '<![CDATA['
+   [20] CData ::= (Char* - (Char* ']]>' Char*))
+   [21] CDEnd ::= ']]>' */
+static HRESULT reader_parse_cdata(xmlreader *reader)
+{
+    WCHAR *start, *ptr;
+
+    if (reader->resume[XmlReadResume_Body])
+    {
+        start = reader->resume[XmlReadResume_Body];
+        ptr = reader_get_cur(reader);
+    }
+    else
+    {
+        /* skip markup '<![CDATA[' */
+        reader_skipn(reader, 9);
+        reader_shrink(reader);
+        ptr = start = reader_get_cur(reader);
+        reader->nodetype = XmlNodeType_CDATA;
+        reader->resume[XmlReadResume_Body] = start;
+        reader->resumestate = XmlReadResumeState_CDATA;
+        reader_set_strvalue(reader, StringValue_LocalName, NULL);
+        reader_set_strvalue(reader, StringValue_QualifiedName, NULL);
+        reader_set_strvalue(reader, StringValue_Value, NULL);
+    }
+
+    while (*ptr)
+    {
+        if (*ptr == ']' && *(ptr+1) == ']' && *(ptr+2) == '>')
+        {
+            strval value;
+
+            TRACE("%s\n", debugstr_wn(start, ptr-start));
+            /* skip ']]>' */
+            reader_skipn(reader, 3);
+            reader_init_strvalue(start, ptr-start, &value);
+            reader_set_strvalue(reader, StringValue_LocalName, &strval_empty);
+            reader_set_strvalue(reader, StringValue_QualifiedName, &strval_empty);
+            reader_set_strvalue(reader, StringValue_Value, &value);
+            reader->resume[XmlReadResume_Body] = NULL;
+            reader->resumestate = XmlReadResumeState_Initial;
+            return S_OK;
+        }
+        else
+        {
+            reader_skipn(reader, 1);
+            ptr++;
+        }
+    }
+
+    return S_OK;
+}
+
+/* [66] CharRef ::= '&#' [0-9]+ ';' | '&#x' [0-9a-fA-F]+ ';'
+   [67]        Reference ::= EntityRef | CharRef
+   [68]        EntityRef ::= '&' Name ';' */
+static HRESULT reader_parse_reference(xmlreader *reader)
+{
+    FIXME("References not supported\n");
+    return E_NOTIMPL;
+}
+
+/* [14] CharData ::= [^<&]* - ([^<&]* ']]>' [^<&]*) */
+static HRESULT reader_parse_chardata(xmlreader *reader)
+{
+    FIXME("CharData not supported\n");
+    return E_NOTIMPL;
+}
+
+/* [43] content ::= CharData? ((element | Reference | CDSect | PI | Comment) CharData?)* */
+static HRESULT reader_parse_content(xmlreader *reader)
+{
+    static const WCHAR cdstartW[] = {'<','!','[','C','D','A','T','A','[',0};
+    static const WCHAR etagW[] = {'<','/',0};
+    static const WCHAR ampW[] = {'&',0};
+
+    if (reader->resumestate != XmlReadResumeState_Initial)
+    {
+        switch (reader->resumestate)
+        {
+        case XmlReadResumeState_CDATA:
+            return reader_parse_cdata(reader);
+        case XmlReadResumeState_Comment:
+            return reader_parse_comment(reader);
+        case XmlReadResumeState_PIBody:
+        case XmlReadResumeState_PITarget:
+            return reader_parse_pi(reader);
+        default:
+            ERR("unknown resume state %d\n", reader->resumestate);
+        }
+    }
+
+    reader_shrink(reader);
+
+    /* handle end tag here, it indicates end of content as well */
+    if (!reader_cmp(reader, etagW))
+        return reader_parse_endtag(reader);
+
+    if (!reader_cmp(reader, commentW))
+        return reader_parse_comment(reader);
+
+    if (!reader_cmp(reader, piW))
+        return reader_parse_pi(reader);
+
+    if (!reader_cmp(reader, cdstartW))
+        return reader_parse_cdata(reader);
+
+    if (!reader_cmp(reader, ampW))
+        return reader_parse_reference(reader);
+
+    if (!reader_cmp(reader, ltW))
+        return reader_parse_element(reader);
+
+    /* what's left must be CharData */
+    return reader_parse_chardata(reader);
+}
+
+static HRESULT reader_parse_nextnode(xmlreader *reader)
+{
+    HRESULT hr;
+
+    while (1)
+    {
+        switch (reader->instate)
+        {
+        /* if it's a first call for a new input we need to detect stream encoding */
+        case XmlReadInState_Initial:
+            {
+                xml_encoding enc;
+
+                hr = readerinput_growraw(reader->input);
+                if (FAILED(hr)) return hr;
+
+                /* try to detect encoding by BOM or data and set input code page */
+                hr = readerinput_detectencoding(reader->input, &enc);
+                TRACE("detected encoding %s, 0x%08x\n", debugstr_w(xml_encoding_map[enc].name), hr);
+                if (FAILED(hr)) return hr;
+
+                /* always switch first time cause we have to put something in */
+                readerinput_switchencoding(reader->input, enc);
+
+                /* parse xml declaration */
+                hr = reader_parse_xmldecl(reader);
+                if (FAILED(hr)) return hr;
+
+                readerinput_shrinkraw(reader->input, -1);
+                reader->instate = XmlReadInState_Misc_DTD;
+                if (hr == S_OK) return hr;
+            }
+            break;
+        case XmlReadInState_Misc_DTD:
+            hr = reader_parse_misc(reader);
+            if (FAILED(hr)) return hr;
+
+            if (hr == S_FALSE)
+                reader->instate = XmlReadInState_DTD;
+            else
+                return hr;
+            break;
+        case XmlReadInState_DTD:
+            hr = reader_parse_dtd(reader);
+            if (FAILED(hr)) return hr;
+
+            if (hr == S_OK)
+            {
+                reader->instate = XmlReadInState_DTD_Misc;
+                return hr;
+            }
+            else
+                reader->instate = XmlReadInState_Element;
+            break;
+        case XmlReadInState_DTD_Misc:
+            hr = reader_parse_misc(reader);
+            if (FAILED(hr)) return hr;
+
+            if (hr == S_FALSE)
+                reader->instate = XmlReadInState_Element;
+            else
+                return hr;
+            break;
+        case XmlReadInState_Element:
+            return reader_parse_element(reader);
+        case XmlReadInState_Content:
+            return reader_parse_content(reader);
+        default:
+            FIXME("internal state %d not handled\n", reader->instate);
+            return E_NOTIMPL;
+        }
+    }
+
+    return E_NOTIMPL;
+}
+
+static HRESULT WINAPI xmlreader_QueryInterface(IXmlReader *iface, REFIID riid, void** ppvObject)
+{
+    xmlreader *This = impl_from_IXmlReader(iface);
+
+    TRACE("(%p)->(%s %p)\n", This, debugstr_guid(riid), ppvObject);
+
+    if (IsEqualGUID(riid, &IID_IUnknown) ||
+        IsEqualGUID(riid, &IID_IXmlReader))
+    {
+        *ppvObject = iface;
+    }
+    else
+    {
+        FIXME("interface %s not implemented\n", debugstr_guid(riid));
+        *ppvObject = NULL;
+        return E_NOINTERFACE;
+    }
+
+    IXmlReader_AddRef(iface);
+
+    return S_OK;
+}
+
+static ULONG WINAPI xmlreader_AddRef(IXmlReader *iface)
+{
+    xmlreader *This = impl_from_IXmlReader(iface);
+    ULONG ref = InterlockedIncrement(&This->ref);
+    TRACE("(%p)->(%d)\n", This, ref);
+    return ref;
+}
+
+static ULONG WINAPI xmlreader_Release(IXmlReader *iface)
+{
+    xmlreader *This = impl_from_IXmlReader(iface);
+    LONG ref = InterlockedDecrement(&This->ref);
+
+    TRACE("(%p)->(%d)\n", This, ref);
+
+    if (ref == 0)
+    {
+        IMalloc *imalloc = This->imalloc;
+        if (This->input) IUnknown_Release(&This->input->IXmlReaderInput_iface);
+        reader_clear_attrs(This);
+        reader_clear_elements(This);
+        reader_free_strvalues(This);
+        reader_free(This, This);
+        if (imalloc) IMalloc_Release(imalloc);
+    }
+
+    return ref;
+}
+
+static HRESULT WINAPI xmlreader_SetInput(IXmlReader* iface, IUnknown *input)
+{
+    xmlreader *This = impl_from_IXmlReader(iface);
+    IXmlReaderInput *readerinput;
+    HRESULT hr;
+
+    TRACE("(%p)->(%p)\n", This, input);
+
+    if (This->input)
+    {
+        readerinput_release_stream(This->input);
+        IUnknown_Release(&This->input->IXmlReaderInput_iface);
+        This->input = NULL;
+    }
+
+    This->line = This->pos = 0;
+    reader_clear_elements(This);
+    This->depth = 0;
+    This->resumestate = XmlReadResumeState_Initial;
+    memset(This->resume, 0, sizeof(This->resume));
+
+    /* just reset current input */
+    if (!input)
+    {
+        This->state = XmlReadState_Initial;
+        return S_OK;
+    }
+
+    /* now try IXmlReaderInput, ISequentialStream, IStream */
+    hr = IUnknown_QueryInterface(input, &IID_IXmlReaderInput, (void**)&readerinput);
+    if (hr == S_OK)
+    {
+        if (readerinput->lpVtbl == &xmlreaderinputvtbl)
+            This->input = impl_from_IXmlReaderInput(readerinput);
+        else
+        {
+            ERR("got external IXmlReaderInput implementation: %p, vtbl=%p\n",
+                readerinput, readerinput->lpVtbl);
+            IUnknown_Release(readerinput);
+            return E_FAIL;
+
+        }
+    }
+
+    if (hr != S_OK || !readerinput)
+    {
+        /* create IXmlReaderInput basing on supplied interface */
+        hr = CreateXmlReaderInputWithEncodingName(input,
+                                         NULL, NULL, FALSE, NULL, &readerinput);
+        if (hr != S_OK) return hr;
+        This->input = impl_from_IXmlReaderInput(readerinput);
+    }
+
+    /* set stream for supplied IXmlReaderInput */
+    hr = readerinput_query_for_stream(This->input);
+    if (hr == S_OK)
+    {
+        This->state = XmlReadState_Initial;
+        This->instate = XmlReadInState_Initial;
+    }
+
+    return hr;
+}
+
+static HRESULT WINAPI xmlreader_GetProperty(IXmlReader* iface, UINT property, LONG_PTR *value)
+{
+    xmlreader *This = impl_from_IXmlReader(iface);
+
+    TRACE("(%p %u %p)\n", This, property, value);
+
+    if (!value) return E_INVALIDARG;
+
+    switch (property)
     {
         case XmlReaderProperty_DtdProcessing:
             *value = This->dtdmode;
@@ -897,44 +2147,22 @@ static HRESULT WINAPI xmlreader_SetProperty(IXmlReader* iface, UINT property, LO
     return S_OK;
 }
 
-static HRESULT WINAPI xmlreader_Read(IXmlReader* iface, XmlNodeType *node_type)
+static HRESULT WINAPI xmlreader_Read(IXmlReader* iface, XmlNodeType *nodetype)
 {
     xmlreader *This = impl_from_IXmlReader(iface);
+    XmlNodeType oldtype = This->nodetype;
+    HRESULT hr;
 
-    FIXME("(%p)->(%p): stub\n", This, node_type);
+    TRACE("(%p)->(%p)\n", This, nodetype);
 
     if (This->state == XmlReadState_Closed) return S_FALSE;
 
-    /* if it's a first call for a new input we need to detect stream encoding */
-    if (This->state == XmlReadState_Initial)
-    {
-        xml_encoding enc;
-        HRESULT hr;
+    hr = reader_parse_nextnode(This);
+    if (oldtype == XmlNodeType_None && This->nodetype != oldtype)
+        This->state = XmlReadState_Interactive;
+    if (hr == S_OK) *nodetype = This->nodetype;
 
-        hr = readerinput_growraw(This->input);
-        if (FAILED(hr)) return hr;
-
-        /* try to detect encoding by BOM or data and set input code page */
-        hr = readerinput_detectencoding(This->input, &enc);
-        TRACE("detected encoding %s, 0x%08x\n", debugstr_w(xml_encoding_map[enc].name), hr);
-        if (FAILED(hr)) return hr;
-
-        /* always switch first time cause we have to put something in */
-        readerinput_switchencoding(This->input, enc);
-
-        /* parse xml declaration */
-        hr = reader_parse_xmldecl(This);
-        if (FAILED(hr)) return hr;
-
-        if (hr == S_OK)
-        {
-            This->state = XmlReadState_Interactive;
-            This->nodetype = *node_type = XmlNodeType_XmlDeclaration;
-            return S_OK;
-        }
-    }
-
-    return E_NOTIMPL;
+    return hr;
 }
 
 static HRESULT WINAPI xmlreader_GetNodeType(IXmlReader* iface, XmlNodeType *node_type)
@@ -998,11 +2226,14 @@ static HRESULT WINAPI xmlreader_MoveToElement(IXmlReader* iface)
     return S_OK;
 }
 
-static HRESULT WINAPI xmlreader_GetQualifiedName(IXmlReader* iface, LPCWSTR *qualifiedName,
-                                                 UINT *qualifiedName_length)
+static HRESULT WINAPI xmlreader_GetQualifiedName(IXmlReader* iface, LPCWSTR *name, UINT *len)
 {
-    FIXME("(%p %p %p): stub\n", iface, qualifiedName, qualifiedName_length);
-    return E_NOTIMPL;
+    xmlreader *This = impl_from_IXmlReader(iface);
+
+    TRACE("(%p)->(%p %p)\n", This, name, len);
+    *name = This->strvalues[StringValue_QualifiedName].str;
+    *len  = This->strvalues[StringValue_QualifiedName].len;
+    return S_OK;
 }
 
 static HRESULT WINAPI xmlreader_GetNamespaceUri(IXmlReader* iface,
@@ -1013,12 +2244,14 @@ static HRESULT WINAPI xmlreader_GetNamespaceUri(IXmlReader* iface,
     return E_NOTIMPL;
 }
 
-static HRESULT WINAPI xmlreader_GetLocalName(IXmlReader* iface,
-                                             LPCWSTR *local_name,
-                                             UINT *local_name_length)
+static HRESULT WINAPI xmlreader_GetLocalName(IXmlReader* iface, LPCWSTR *name, UINT *len)
 {
-    FIXME("(%p %p %p): stub\n", iface, local_name, local_name_length);
-    return E_NOTIMPL;
+    xmlreader *This = impl_from_IXmlReader(iface);
+
+    TRACE("(%p)->(%p %p)\n", This, name, len);
+    *name = This->strvalues[StringValue_LocalName].str;
+    *len  = This->strvalues[StringValue_LocalName].len;
+    return S_OK;
 }
 
 static HRESULT WINAPI xmlreader_GetPrefix(IXmlReader* iface,
@@ -1029,21 +2262,61 @@ static HRESULT WINAPI xmlreader_GetPrefix(IXmlReader* iface,
     return E_NOTIMPL;
 }
 
-static HRESULT WINAPI xmlreader_GetValue(IXmlReader* iface,
-                                         LPCWSTR *value,
-                                         UINT *value_length)
+static HRESULT WINAPI xmlreader_GetValue(IXmlReader* iface, const WCHAR **value, UINT *len)
 {
-    FIXME("(%p %p %p): stub\n", iface, value, value_length);
-    return E_NOTIMPL;
+    xmlreader *reader = impl_from_IXmlReader(iface);
+    strval *val = &reader->strvalues[StringValue_Value];
+
+    TRACE("(%p)->(%p %p)\n", reader, value, len);
+
+    *value = NULL;
+
+    if ((reader->nodetype == XmlNodeType_Comment && !val->str) || is_reader_pending(reader))
+    {
+        XmlNodeType type;
+        HRESULT hr;
+
+        hr = IXmlReader_Read(iface, &type);
+        if (FAILED(hr)) return hr;
+
+        /* return if still pending, partially read values are not reported */
+        if (is_reader_pending(reader)) return E_PENDING;
+    }
+
+    if (!val->str)
+    {
+        val->str = reader_alloc(reader, (val->len+1)*sizeof(WCHAR));
+        if (!val->str) return E_OUTOFMEMORY;
+        memcpy(val->str, val->start, val->len*sizeof(WCHAR));
+        val->str[val->len] = 0;
+    }
+
+    *value = val->str;
+    if (len) *len = val->len;
+    return S_OK;
 }
 
-static HRESULT WINAPI xmlreader_ReadValueChunk(IXmlReader* iface,
-                                               WCHAR *buffer,
-                                               UINT   chunk_size,
-                                               UINT  *read)
+static HRESULT WINAPI xmlreader_ReadValueChunk(IXmlReader* iface, WCHAR *buffer, UINT chunk_size, UINT *read)
 {
-    FIXME("(%p %p %u %p): stub\n", iface, buffer, chunk_size, read);
-    return E_NOTIMPL;
+    xmlreader *reader = impl_from_IXmlReader(iface);
+    strval *val = &reader->strvalues[StringValue_Value];
+    UINT len;
+
+    TRACE("(%p)->(%p %u %p)\n", reader, buffer, chunk_size, read);
+
+    /* Value is already allocated, chunked reads are not possible. */
+    if (val->str) return S_FALSE;
+
+    if (val->len)
+    {
+        len = min(chunk_size, val->len);
+        memcpy(buffer, val->start, len);
+        val->start += len;
+        val->len -= len;
+        if (read) *read = len;
+    }
+
+    return S_OK;
 }
 
 static HRESULT WINAPI xmlreader_GetBaseUri(IXmlReader* iface,
@@ -1106,8 +2379,10 @@ static HRESULT WINAPI xmlreader_GetAttributeCount(IXmlReader* iface, UINT *count
 
 static HRESULT WINAPI xmlreader_GetDepth(IXmlReader* iface, UINT *depth)
 {
-    FIXME("(%p %p): stub\n", iface, depth);
-    return E_NOTIMPL;
+    xmlreader *This = impl_from_IXmlReader(iface);
+    TRACE("(%p)->(%p)\n", This, depth);
+    *depth = This->depth;
+    return S_OK;
 }
 
 static BOOL WINAPI xmlreader_IsEOF(IXmlReader* iface)
@@ -1151,7 +2426,7 @@ static HRESULT WINAPI xmlreaderinput_QueryInterface(IXmlReaderInput *iface, REFI
 {
     xmlreaderinput *This = impl_from_IXmlReaderInput(iface);
 
-    TRACE("%p %s %p\n", This, debugstr_guid(riid), ppvObject);
+    TRACE("(%p)->(%s %p)\n", This, debugstr_guid(riid), ppvObject);
 
     if (IsEqualGUID(riid, &IID_IXmlReaderInput) ||
         IsEqualGUID(riid, &IID_IUnknown))
@@ -1161,6 +2436,7 @@ static HRESULT WINAPI xmlreaderinput_QueryInterface(IXmlReaderInput *iface, REFI
     else
     {
         WARN("interface %s not implemented\n", debugstr_guid(riid));
+        *ppvObject = NULL;
         return E_NOINTERFACE;
     }
 
@@ -1198,7 +2474,7 @@ static ULONG WINAPI xmlreaderinput_Release(IXmlReaderInput *iface)
     return ref;
 }
 
-static const struct IUnknownVtbl xmlreaderinput_vtbl =
+static const struct IUnknownVtbl xmlreaderinputvtbl =
 {
     xmlreaderinput_QueryInterface,
     xmlreaderinput_AddRef,
@@ -1208,6 +2484,7 @@ static const struct IUnknownVtbl xmlreaderinput_vtbl =
 HRESULT WINAPI CreateXmlReader(REFIID riid, void **obj, IMalloc *imalloc)
 {
     xmlreader *reader;
+    int i;
 
     TRACE("(%s, %p, %p)\n", wine_dbgstr_guid(riid), obj, imalloc);
 
@@ -1227,6 +2504,8 @@ HRESULT WINAPI CreateXmlReader(REFIID riid, void **obj, IMalloc *imalloc)
     reader->ref = 1;
     reader->input = NULL;
     reader->state = XmlReadState_Closed;
+    reader->instate = XmlReadInState_Initial;
+    reader->resumestate = XmlReadResumeState_Initial;
     reader->dtdmode = DtdProcessing_Prohibit;
     reader->line  = reader->pos = 0;
     reader->imalloc = imalloc;
@@ -1235,6 +2514,12 @@ HRESULT WINAPI CreateXmlReader(REFIID riid, void **obj, IMalloc *imalloc)
     list_init(&reader->attrs);
     reader->attr_count = 0;
     reader->attr = NULL;
+    list_init(&reader->elements);
+    reader->depth = 0;
+    memset(reader->resume, 0, sizeof(reader->resume));
+
+    for (i = 0; i < StringValue_Last; i++)
+        reader->strvalues[i] = strval_empty;
 
     *obj = &reader->IXmlReader_iface;
 
@@ -1264,7 +2549,7 @@ HRESULT WINAPI CreateXmlReaderInputWithEncodingName(IUnknown *stream,
         readerinput = heap_alloc(sizeof(*readerinput));
     if(!readerinput) return E_OUTOFMEMORY;
 
-    readerinput->IXmlReaderInput_iface.lpVtbl = &xmlreaderinput_vtbl;
+    readerinput->IXmlReaderInput_iface.lpVtbl = &xmlreaderinputvtbl;
     readerinput->ref = 1;
     readerinput->imalloc = imalloc;
     readerinput->stream = NULL;
@@ -1272,11 +2557,14 @@ HRESULT WINAPI CreateXmlReaderInputWithEncodingName(IUnknown *stream,
     readerinput->encoding = parse_encoding_name(encoding, -1);
     readerinput->hint = hint;
     readerinput->baseuri = readerinput_strdupW(readerinput, base_uri);
+    readerinput->pending = 0;
 
     hr = alloc_input_buffer(readerinput);
     if (hr != S_OK)
     {
+        readerinput_free(readerinput, readerinput->baseuri);
         readerinput_free(readerinput, readerinput);
+        if (imalloc) IMalloc_Release(imalloc);
         return hr;
     }
     IUnknown_QueryInterface(stream, &IID_IUnknown, (void**)&readerinput->input);
index 989d361..39416e9 100644 (file)
@@ -30,7 +30,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(xmllite);
 
 BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved)
 {
-    TRACE("(0x%p, %d, %p)\n", hinstDLL, fdwReason, lpvReserved);
+    TRACE("(%p, %u, %p)\n", hinstDLL, fdwReason, lpvReserved);
 
     switch (fdwReason)
     {
index 9dbf101..30d4e1f 100644 (file)
@@ -201,7 +201,7 @@ reactos/dll/win32/xinput1_1       # Synced to Wine-1.5.19
 reactos/dll/win32/xinput1_2       # Synced to Wine-1.5.19
 reactos/dll/win32/xinput1_3       # Synced to Wine-1.5.19
 reactos/dll/win32/xinput9_1_0     # Synced to Wine-1.5.19
-reactos/dll/win32/xmllite         # Synced to Wine-1.5.19
+reactos/dll/win32/xmllite         # Synced to Wine-1.5.26
 
 reactos/dll/cpl/inetcpl           # Synced to Wine-1.3.21