Sat, 13 Dec 2014 04:50:33 +0200
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
9
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
1 | /* |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
2 | Copyright 2014 Teemu Piippo |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
3 | All rights reserved. |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
4 | |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
5 | Redistribution and use in source and binary forms, with or without |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
6 | modification, are permitted provided that the following conditions |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
7 | are met: |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
8 | |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
9 | 1. Redistributions of source code must retain the above copyright |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
10 | notice, this list of conditions and the following disclaimer. |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
11 | 2. Redistributions in binary form must reproduce the above copyright |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
12 | notice, this list of conditions and the following disclaimer in the |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
13 | documentation and/or other materials provided with the distribution. |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
14 | 3. Neither the name of the copyright holder nor the names of its |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
15 | contributors may be used to endorse or promote products derived from |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
16 | this software without specific prior written permission. |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
17 | |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
18 | THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
19 | "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
20 | TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
21 | PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
22 | OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
23 | EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
24 | PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
25 | PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
26 | LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
27 | NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
28 | SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
29 | */ |
e7a09ceb4505
- now with more license headers
Teemu Piippo <crimsondusk64@gmail.com>
parents:
5
diff
changeset
|
30 | |
5 | 31 | #include "bytestream.h" |
32 | #include <string.h> | |
33 | ||
34 | // ------------------------------------------------------------------------------------------------- | |
35 | // | |
36 | Bytestream::Bytestream (unsigned long length) : | |
37 | m_data (nullptr) | |
38 | { | |
39 | resize (length); | |
40 | clear(); | |
41 | } | |
42 | ||
43 | // ------------------------------------------------------------------------------------------------- | |
44 | // | |
45 | Bytestream::Bytestream (const unsigned char* data, unsigned long length) : | |
46 | m_data (nullptr) | |
47 | { | |
48 | init (data, length); | |
49 | } | |
50 | ||
51 | // ------------------------------------------------------------------------------------------------- | |
52 | // | |
53 | Bytestream::Bytestream (const Vector<unsigned char>& bytes) : | |
54 | m_data (nullptr) | |
55 | { | |
56 | init (bytes.data(), bytes.size()); | |
57 | } | |
58 | ||
59 | // ------------------------------------------------------------------------------------------------- | |
60 | // | |
11 | 61 | Bytestream::Bytestream (const Bytestream& other) : |
62 | m_data (nullptr) | |
63 | { | |
64 | init (other.data(), other.written_length()); | |
65 | } | |
66 | ||
67 | // ------------------------------------------------------------------------------------------------- | |
68 | METHOD Bytestream::operator= (const Bytestream& other) -> Bytestream& | |
69 | { | |
70 | init (other.data(), other.written_length()); | |
71 | return *this; | |
72 | } | |
73 | ||
74 | // ------------------------------------------------------------------------------------------------- | |
75 | // | |
5 | 76 | Bytestream::~Bytestream() |
77 | { | |
78 | delete m_data; | |
79 | } | |
80 | ||
81 | // ------------------------------------------------------------------------------------------------- | |
82 | // | |
83 | void Bytestream::resize (unsigned long newsize) | |
84 | { | |
85 | Vector<unsigned char> olddata; | |
86 | unsigned long oldsize = 0L; | |
87 | ||
88 | if (m_data != nullptr) | |
89 | { | |
90 | oldsize = allocated_size(); | |
91 | olddata.resize (oldsize); | |
92 | memcpy (olddata.data(), m_data, oldsize); | |
93 | } | |
94 | ||
95 | delete[] m_data; | |
96 | m_allocatedSize = newsize; | |
97 | m_data = new unsigned char[newsize]; | |
98 | ||
99 | if (olddata > 0L) | |
100 | memcpy (m_data, olddata, min (oldsize, newsize)); | |
101 | } | |
102 | ||
103 | // ------------------------------------------------------------------------------------------------- | |
104 | // | |
105 | void Bytestream::init (const unsigned char* data, unsigned long length) | |
106 | { | |
107 | resize (length); | |
108 | memcpy (m_data, data, length); | |
109 | m_cursor = &m_data[0]; | |
110 | m_writtenLength = length; | |
111 | } | |
112 | ||
113 | // ------------------------------------------------------------------------------------------------- | |
114 | // | |
115 | void Bytestream::clear() | |
116 | { | |
117 | m_cursor = &m_data[0]; | |
118 | m_writtenLength = 0; | |
119 | } | |
120 | ||
121 | // ------------------------------------------------------------------------------------------------- | |
122 | // | |
13
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
123 | METHOD |
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
124 | Bytestream::ensure_read_space (unsigned int bytes) -> void |
5 | 125 | { |
13
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
126 | if (bytes_left() < bytes) |
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
127 | { |
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
128 | throw IOError (format ("attempted to read %1 byte(s) past the end of bytestream", |
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
129 | bytes - bytes_left())); |
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
130 | } |
5 | 131 | } |
132 | ||
133 | // ------------------------------------------------------------------------------------------------- | |
134 | // | |
13
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
135 | char Bytestream::read_byte() |
5 | 136 | { |
13
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
137 | ensure_read_space (1); |
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
138 | return *m_cursor++; |
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
139 | } |
5 | 140 | |
13
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
141 | // ------------------------------------------------------------------------------------------------- |
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
142 | // |
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
143 | short int Bytestream::read_short() |
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
144 | { |
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
145 | ensure_read_space (2); |
10
3874575d924d
- begin work on rcon sessions
Teemu Piippo <crimsondusk64@gmail.com>
parents:
9
diff
changeset
|
146 | short int result = 0; |
5 | 147 | |
148 | for (int i = 0; i < 2; ++i) | |
10
3874575d924d
- begin work on rcon sessions
Teemu Piippo <crimsondusk64@gmail.com>
parents:
9
diff
changeset
|
149 | result |= m_cursor[i] << (i * 8); |
5 | 150 | |
10
3874575d924d
- begin work on rcon sessions
Teemu Piippo <crimsondusk64@gmail.com>
parents:
9
diff
changeset
|
151 | m_cursor += 2; |
3874575d924d
- begin work on rcon sessions
Teemu Piippo <crimsondusk64@gmail.com>
parents:
9
diff
changeset
|
152 | return result; |
5 | 153 | } |
154 | ||
155 | // ------------------------------------------------------------------------------------------------- | |
156 | // | |
13
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
157 | long int Bytestream::read_long() |
5 | 158 | { |
13
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
159 | ensure_read_space (4); |
10
3874575d924d
- begin work on rcon sessions
Teemu Piippo <crimsondusk64@gmail.com>
parents:
9
diff
changeset
|
160 | long int result = 0; |
5 | 161 | |
162 | for (int i = 0; i < 4; ++i) | |
10
3874575d924d
- begin work on rcon sessions
Teemu Piippo <crimsondusk64@gmail.com>
parents:
9
diff
changeset
|
163 | result |= m_cursor[i] << (i * 8); |
5 | 164 | |
10
3874575d924d
- begin work on rcon sessions
Teemu Piippo <crimsondusk64@gmail.com>
parents:
9
diff
changeset
|
165 | m_cursor += 4; |
3874575d924d
- begin work on rcon sessions
Teemu Piippo <crimsondusk64@gmail.com>
parents:
9
diff
changeset
|
166 | return result; |
5 | 167 | } |
168 | ||
169 | // ------------------------------------------------------------------------------------------------- | |
170 | // | |
13
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
171 | float Bytestream::read_float() |
5 | 172 | { |
13
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
173 | int value = read_long(); |
5 | 174 | return reinterpret_cast<float&> (value); |
175 | } | |
176 | ||
177 | // ------------------------------------------------------------------------------------------------- | |
178 | // | |
13
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
179 | String Bytestream::read_string() |
5 | 180 | { |
181 | // Zandronum sends strings of maximum 2048 characters, though it only | |
182 | // reads 2047-character long ones so I guess we can follow up and do | |
183 | // the same :-) | |
184 | static char buffer[MAX_NETWORK_STRING]; | |
185 | unsigned char* stringEnd; | |
186 | unsigned char* stringBegin = m_cursor; | |
187 | unsigned char* end = m_data + allocated_size(); | |
188 | ||
189 | // where's the end of the string? | |
190 | for (stringEnd = m_cursor; *stringEnd != '\0'; ++stringEnd) | |
191 | { | |
192 | if (stringEnd == end) | |
193 | // past the end of the buffer! Argh! | |
13
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
194 | throw IOError ("unterminated string in packet"); |
5 | 195 | } |
196 | ||
197 | m_cursor = stringEnd + 1; | |
198 | unsigned int length = stringEnd - m_cursor; | |
199 | ||
200 | // ensure we won't write past the buffer (note: we still moved | |
201 | // past the excess bytes in the above statement, those are ignored) | |
202 | if (length >= MAX_NETWORK_STRING) | |
203 | length = MAX_NETWORK_STRING - 1; | |
204 | ||
205 | memcpy (buffer, stringBegin, length); | |
206 | buffer[length] = '\0'; | |
207 | return String (buffer); | |
208 | } | |
209 | ||
210 | // ------------------------------------------------------------------------------------------------- | |
211 | // | |
212 | METHOD | |
13
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
213 | Bytestream::read (unsigned char* buffer, unsigned long length) -> void |
5 | 214 | { |
13
09dcaeaa216b
- to hell with that 'ok' field. now throws an exception if attempts to read past the end
Teemu Piippo <crimsondusk64@gmail.com>
parents:
11
diff
changeset
|
215 | ensure_read_space (length); |
5 | 216 | memcpy (buffer, m_cursor, length); |
217 | m_cursor += length; | |
218 | } | |
219 | ||
220 | // ------------------------------------------------------------------------------------------------- | |
221 | // | |
222 | void Bytestream::write (unsigned char val) | |
223 | { | |
224 | *m_cursor++ = val; | |
225 | m_writtenLength++; | |
226 | } | |
227 | ||
228 | // ------------------------------------------------------------------------------------------------- | |
229 | // | |
230 | void Bytestream::write (const unsigned char* val, unsigned int length) | |
231 | { | |
232 | grow_to_fit (length); | |
233 | memcpy (m_cursor, val, length); | |
234 | m_cursor += length; | |
235 | m_writtenLength += length; | |
236 | } | |
237 | ||
238 | // ------------------------------------------------------------------------------------------------- | |
239 | // | |
240 | void Bytestream::grow_to_fit (unsigned long bytes) | |
241 | { | |
242 | if (space_left() < bytes) | |
243 | resize (allocated_size() + bytes + 128); | |
244 | } | |
245 | ||
246 | // ------------------------------------------------------------------------------------------------- | |
247 | // | |
248 | void Bytestream::write_byte (char val) | |
249 | { | |
250 | grow_to_fit (1); | |
251 | write (val); | |
252 | } | |
253 | ||
254 | // ------------------------------------------------------------------------------------------------- | |
255 | // | |
256 | void Bytestream::write_short (short int val) | |
257 | { | |
258 | grow_to_fit (2); | |
259 | ||
260 | for (int i = 0; i < 2; ++i) | |
261 | write ((val >> (i * 8)) & 0xFF); | |
262 | } | |
263 | ||
264 | // ------------------------------------------------------------------------------------------------- | |
265 | // | |
266 | void Bytestream::write_long (long int val) | |
267 | { | |
268 | grow_to_fit (4); | |
269 | ||
270 | for (int i = 0; i < 4; ++i) | |
271 | write ((val >> (i * 8)) & 0xFF); | |
272 | } | |
273 | ||
274 | // ------------------------------------------------------------------------------------------------- | |
275 | // | |
276 | void Bytestream::write_float (float val) | |
277 | { | |
278 | // I know this is probably dangerous but this is what Zandronum does so yeah | |
279 | write_long (reinterpret_cast<int&> (val)); | |
280 | } | |
281 | ||
282 | // ------------------------------------------------------------------------------------------------- | |
283 | // | |
284 | void Bytestream::write_string (const String& val) | |
285 | { | |
286 | grow_to_fit (val.length() + 1); | |
287 | write (reinterpret_cast<const unsigned char*> (val.chars()), val.length()); | |
288 | write (0); | |
289 | } | |
290 | ||
291 | // ------------------------------------------------------------------------------------------------- | |
292 | // | |
293 | void Bytestream::write_buffer (const Bytestream& other) | |
294 | { | |
295 | write (other.data(), other.written_length()); | |
296 | } |