[v2 2/5] webservices: Don't validate reader state in read_endelement.

Hans Leidekker hans at codeweavers.com
Wed Feb 3 08:43:44 CST 2016


Signed-off-by: Hans Leidekker <hans at codeweavers.com>
---
 dlls/webservices/reader.c       | 10 ----------
 dlls/webservices/tests/reader.c |  2 +-
 2 files changed, 1 insertion(+), 11 deletions(-)

diff --git a/dlls/webservices/reader.c b/dlls/webservices/reader.c
index 08f2287..515ed1e 100644
--- a/dlls/webservices/reader.c
+++ b/dlls/webservices/reader.c
@@ -1101,16 +1101,6 @@ static HRESULT read_endelement( struct reader *reader )
     WS_XML_STRING *prefix, *localname;
     HRESULT hr;
 
-    switch (reader->state)
-    {
-    case READER_STATE_TEXT:
-    case READER_STATE_STARTELEMENT:
-    case READER_STATE_STARTENDELEMENT:
-        break;
-    default:
-        return WS_E_INVALID_FORMAT;
-    }
-
     if (read_cmp( reader, "</", 2 )) return WS_E_INVALID_FORMAT;
     read_skip( reader, 2 );
 
diff --git a/dlls/webservices/tests/reader.c b/dlls/webservices/tests/reader.c
index d2f2308..958d08c 100644
--- a/dlls/webservices/tests/reader.c
+++ b/dlls/webservices/tests/reader.c
@@ -885,7 +885,7 @@ static void test_WsReadStartElement(void)
     if (node) ok( node->nodeType == WS_XML_NODE_TYPE_END_ELEMENT, "got %u\n", node->nodeType );
 
     hr = WsReadEndElement( reader, NULL );
-    todo_wine ok( hr == S_OK, "got %08x\n", hr );
+    ok( hr == S_OK, "got %08x\n", hr );
 
     hr = WsGetReaderNode( reader, &node, NULL );
     ok( hr == S_OK, "got %08x\n", hr );
-- 
2.7.0




More information about the wine-patches mailing list