Fri, 06 Feb 2026 14:16:09 +0100
fix error detection in parse_request_line/parse_response_line
|
662
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
1 | /* |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
2 | * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER. |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
3 | * |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
4 | * Copyright 2026 Olaf Wintermann. All rights reserved. |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
5 | * |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
6 | * Redistribution and use in source and binary forms, with or without |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
7 | * modification, are permitted provided that the following conditions are met: |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
8 | * |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
9 | * 1. Redistributions of source code must retain the above copyright |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
10 | * notice, this list of conditions and the following disclaimer. |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
11 | * |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
12 | * 2. Redistributions in binary form must reproduce the above copyright |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
13 | * notice, this list of conditions and the following disclaimer in the |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
14 | * documentation and/or other materials provided with the distribution. |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
15 | * |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
16 | * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
17 | * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
18 | * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
19 | * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
20 | * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
21 | * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
22 | * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
23 | * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
24 | * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
25 | * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
26 | * POSSIBILITY OF SUCH DAMAGE. |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
27 | */ |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
28 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
29 | #include "httpparser.h" |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
30 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
31 | #include "../daemon/httpparser.h" |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
32 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
33 | CX_TEST(test_parse_request_line) { |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
34 | CX_TEST_DO { |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
35 | HttpParser parser; |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
36 | memset(&parser, 0, sizeof(HttpParser)); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
37 | parser.start_line = cx_mutstr("GET / HTTP/1.1\r\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
38 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
39 | int ret1 = parse_request_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
40 | CX_TEST_ASSERT(!ret1); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
41 | CX_TEST_ASSERT(!cx_strcmp(parser.method, "GET")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
42 | CX_TEST_ASSERT(!cx_strcmp(parser.uri, "/")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
43 | CX_TEST_ASSERT(!cx_strcmp(parser.httpv, "HTTP/1.1")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
44 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
45 | memset(&parser, 0, sizeof(HttpParser)); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
46 | parser.start_line = cx_mutstr("POST /longer/url/test/path/ HTTP/1.0\r\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
47 | int ret2 = parse_request_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
48 | CX_TEST_ASSERT(!ret2); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
49 | CX_TEST_ASSERT(!cx_strcmp(parser.method, "POST")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
50 | CX_TEST_ASSERT(!cx_strcmp(parser.uri, "/longer/url/test/path/")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
51 | CX_TEST_ASSERT(!cx_strcmp(parser.httpv, "HTTP/1.0")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
52 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
53 | memset(&parser, 0, sizeof(HttpParser)); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
54 | parser.start_line = cx_mutstr("G / v\r\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
55 | int ret3 = parse_request_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
56 | CX_TEST_ASSERT(!ret3); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
57 | CX_TEST_ASSERT(!cx_strcmp(parser.method, "G")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
58 | CX_TEST_ASSERT(!cx_strcmp(parser.uri, "/")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
59 | CX_TEST_ASSERT(!cx_strcmp(parser.httpv, "v")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
60 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
61 | memset(&parser, 0, sizeof(HttpParser)); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
62 | parser.start_line = cx_mutstr("HEAD /space HTTP/0.9 \r\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
63 | int ret4 = parse_request_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
64 | CX_TEST_ASSERT(!ret4); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
65 | CX_TEST_ASSERT(!cx_strcmp(parser.method, "HEAD")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
66 | CX_TEST_ASSERT(!cx_strcmp(parser.uri, "/space")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
67 | CX_TEST_ASSERT(!cx_strcmp(parser.httpv, "HTTP/0.9")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
68 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
69 | // unix-style line break tests |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
70 | memset(&parser, 0, sizeof(HttpParser)); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
71 | parser.start_line = cx_mutstr("PROPFIND /collection/ HTTP/1.1\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
72 | int uret1 = parse_request_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
73 | CX_TEST_ASSERT(!uret1); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
74 | CX_TEST_ASSERT(!cx_strcmp(parser.method, "PROPFIND")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
75 | CX_TEST_ASSERT(!cx_strcmp(parser.uri, "/collection/")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
76 | CX_TEST_ASSERT(!cx_strcmp(parser.httpv, "HTTP/1.1")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
77 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
78 | // negative tests |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
79 | parser.start_line = cx_mutstr("\r\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
80 | int nret1 = parse_request_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
81 | CX_TEST_ASSERT(nret1); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
82 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
83 | parser.start_line = cx_mutstr("GET\r\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
84 | int nret2 = parse_request_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
85 | CX_TEST_ASSERT(nret2); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
86 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
87 | parser.start_line = cx_mutstr("GET /test\r\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
88 | int nret3 = parse_request_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
89 | CX_TEST_ASSERT(nret3); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
90 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
91 | parser.start_line = cx_mutstr("GET /test \r\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
92 | int nret4 = parse_request_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
93 | CX_TEST_ASSERT(nret4); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
94 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
95 | parser.start_line = cx_mutstr("/test HTTP/1.1\r\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
96 | int nret5 = parse_request_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
97 | CX_TEST_ASSERT(nret5); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
98 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
99 | parser.start_line = cx_mutstr("GET /uri HTTP/1.1 test\r\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
100 | int nret6 = parse_request_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
101 | CX_TEST_ASSERT(nret6); |
|
663
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
102 | |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
103 | parser.start_line = cx_mutstr(" /uri2 HTTP/1.1\r\n"); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
104 | int nret7 = parse_request_line(&parser); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
105 | CX_TEST_ASSERT(nret7); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
106 | |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
107 | parser.start_line = cx_mutstr("GET HTTP/1.1\r\n"); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
108 | int nret8 = parse_request_line(&parser); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
109 | CX_TEST_ASSERT(nret8); |
|
662
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
110 | } |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
111 | } |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
112 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
113 | CX_TEST(test_parse_response_line) { |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
114 | CX_TEST_DO { |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
115 | HttpParser parser; |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
116 | memset(&parser, 0, sizeof(HttpParser)); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
117 | parser.start_line = cx_mutstr("HTTP/1.1 200 OK\r\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
118 | parser.type = 1; |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
119 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
120 | int ret1 = parse_response_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
121 | CX_TEST_ASSERT(!ret1); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
122 | CX_TEST_ASSERT(!cx_strcmp(parser.httpv, "HTTP/1.1")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
123 | CX_TEST_ASSERT(!cx_strcmp(parser.msg, "OK")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
124 | CX_TEST_ASSERT(parser.status == 200); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
125 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
126 | memset(&parser, 0, sizeof(HttpParser)); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
127 | parser.start_line = cx_mutstr("HTTP/1.0 201 Created\r\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
128 | parser.type = 1; |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
129 | int ret2 = parse_response_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
130 | CX_TEST_ASSERT(!ret2); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
131 | CX_TEST_ASSERT(!cx_strcmp(parser.httpv, "HTTP/1.0")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
132 | CX_TEST_ASSERT(!cx_strcmp(parser.msg, "Created")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
133 | CX_TEST_ASSERT(parser.status == 201); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
134 | |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
135 | memset(&parser, 0, sizeof(HttpParser)); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
136 | parser.start_line = cx_mutstr("HTTP/0.9 204 No Content\r\n"); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
137 | parser.type = 1; |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
138 | int ret3 = parse_response_line(&parser); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
139 | CX_TEST_ASSERT(!ret3); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
140 | CX_TEST_ASSERT(!cx_strcmp(parser.httpv, "HTTP/0.9")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
141 | CX_TEST_ASSERT(!cx_strcmp(parser.msg, "No Content")); |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
142 | CX_TEST_ASSERT(parser.status == 204); |
|
663
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
143 | |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
144 | // negative tests |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
145 | memset(&parser, 0, sizeof(HttpParser)); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
146 | parser.start_line = cx_mutstr("HTTP/1.1 200\r\n"); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
147 | parser.type = 1; |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
148 | int nret1 = parse_response_line(&parser); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
149 | CX_TEST_ASSERT(nret1); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
150 | |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
151 | memset(&parser, 0, sizeof(HttpParser)); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
152 | parser.start_line = cx_mutstr("\r\n"); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
153 | parser.type = 1; |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
154 | int nret2 = parse_response_line(&parser); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
155 | CX_TEST_ASSERT(nret2); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
156 | |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
157 | memset(&parser, 0, sizeof(HttpParser)); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
158 | parser.start_line = cx_mutstr("200\r\n"); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
159 | parser.type = 1; |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
160 | int nret3 = parse_response_line(&parser); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
161 | CX_TEST_ASSERT(nret3); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
162 | |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
163 | memset(&parser, 0, sizeof(HttpParser)); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
164 | parser.start_line = cx_mutstr("HTTP/1.1 2345 Test Message\r\n"); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
165 | parser.type = 1; |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
166 | int nret4 = parse_response_line(&parser); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
167 | CX_TEST_ASSERT(nret4); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
168 | |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
169 | memset(&parser, 0, sizeof(HttpParser)); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
170 | parser.start_line = cx_mutstr("HTTP/1.1 xy1 OK\r\n"); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
171 | parser.type = 1; |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
172 | int nret5 = parse_response_line(&parser); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
173 | CX_TEST_ASSERT(nret5); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
174 | |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
175 | memset(&parser, 0, sizeof(HttpParser)); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
176 | parser.start_line = cx_mutstr(" 200 OK\r\n"); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
177 | parser.type = 1; |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
178 | int nret6 = parse_response_line(&parser); |
|
bd116bd44926
fix error detection in parse_request_line/parse_response_line
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
662
diff
changeset
|
179 | CX_TEST_ASSERT(nret6); |
|
662
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
180 | } |
|
70fdf948b642
refactor HttpParser to support parsing of Http responses
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff
changeset
|
181 | } |