[prev in list] [next in list] [prev in thread] [next in thread] 

List:       full-disclosure
Subject:    [FD] Asterisk Skinny memory exhaustion vulnerability leads to DoS
From:       Sandro Gauci <sandro () enablesecurity ! com>
Date:       2017-05-22 20:31:20
Message-ID: 1495485080.2897187.985065344.733BA3C3 () webmail ! messagingengine ! com
[Download RAW message or body]

# Asterisk Skinny memory exhaustion vulnerability leads to DoS

- Authors: 
    - Alfred Farrugia <alfred@enablesecurity.com>
    - Sandro Gauci <sandro@enablesecurity.com>
- Vulnerable version: Asterisk 14.4.0 with `chan_skinny` enabled
- References: AST-2017-004
- Enable Security Advisory:
<https://github.com/EnableSecurity/advisories/tree/master/ES2017-03-asterisk-chan-skinny-crash>
- Vendor Advisory:
<http://downloads.asterisk.org/pub/security/AST-2017-004.html>
- Timeline:
    - Report date: 2017-04-13
    - Digium confirmed issue: 2017-04-13
    - Digium patch and advisory: 2017-05-19
    - Enable Security advisory: 2017-05-23

## Description

Sending one malformed Skinny message to port 2000 will exhaust
Asterisk's memory resulting in a crash.

## Impact

Abuse of this issue allows attackers to crash Asterisk when Skinny is
exposed to attackers.

## How to reproduce the issue

Start Asterisk and make sure the `chan_skinny` module is loaded. Then
execute:

	printf
	"\x38\x00\x00\x00\x00\x00\x00\x00\x01\x00\x00\x00\x53\x45\x50\x30\x30\x30\x39"
	| nc localhost 2000
	
After a few seconds Asterisk will crash since it will be using all of
the available memory. Different malformed strings will crash the server
faster or slower depending on the amount by which `req->data` is
extended.


The malformed message will throw the following errors in Asterisk:

	[Apr  6 09:35:26] WARNING[6893]: chan_skinny.c:7587
	skinny_session: Partial data received, waiting (35 bytes read of
	52)

while it will loop forever. This is due to the following code:

```c
while (1) {     
	if ((res = read(s->fd, ((char*)&req->data)+bytesread,
	dlen-bytesread)) < 0) {
		ast_log(LOG_WARNING, "Data read() returned error: %s\n",
		strerror(errno));
		break;
	}
	bytesread += res;
	if (bytesread >= dlen) {
		if (res < bytesread) {
			ast_log(LOG_WARNING, "Rest of partial data
			received.\n");
		}
		if (bytesread > dlen) {
			ast_log(LOG_WARNING, "Client sent wrong amount
			of data (%d), expected (%d).\n", bytesread,
			dlen);
			res = -1;
		}
		break;
	}

	ast_log(LOG_WARNING, "Partial data received, waiting (%d bytes
	read of %d)\n", bytesread, dlen);
	if (sched_yield() < 0) {
		ast_log(LOG_WARNING, "Data yield() returned error:
		%s\n", strerror(errno));
		res = -1;
		break;
	}
}
```

The reason appears to be that `res` always returns 0 so `bytesread` will
not grow and the loop never breaks. However `req->data` will continue to
expand until all the memory is exhausted. 

This issue was found through basic manual testing, before attempting to
start fuzzing `chan_skinny`.

![Memory usage while Asterisk receives a malformed skinny
message](mem-graph.png)

## Solutions and recommendations

Apply fix issued by Asterisk, upgrade to Asterisk 13.15.1, 14.4.1 or
13.13-cert4. Enable Security highly recommends disabling this module.

## About Enable Security

[Enable Security](https://www.enablesecurity.com) provides Information
Security services, including Penetration Testing, Research and
Development, to help protect client networks and applications against
online attackers.

## Disclaimer

The information in the advisory is believed to be accurate at the time
of publishing based on currently available information. Use of the
information constitutes acceptance for use in an AS IS condition. There
are no warranties with regard to this information. Neither the author
nor the publisher accepts any liability for any direct, indirect, or
consequential loss or damage arising from use of, or reliance on, this
information.


["mem-graph.png" (mem-graph.png)]

PNG


IHDR XrB PLTE   @Ai \
@0`@ԥ**@333MMMfff22U݂d"".Wp͇PErz挽k \
  ݐP@Uk/@@``@ @ ` pͷ |@  \
K	pHYs+JIDATxݍz⸺Da5ϐ͟-iz>t&]TDŽ4I$I$I$IRzRzS.Y]T|tⱤraI]9ygHpΞ4/p1{9OՁ. \
k||oGpH>K{@<p]_  \
dynߪGnyOlzX(юWYGEv \
g?VH?V(CpLV(wtg0PŮ}(Ϳt \
("ͿѰ>HEQ֏tzR+AQAj%( ?oݼ}Z	Aj%( \
(rVhT?}Z	_7[JPu/Aj%(i=Y< ( \
#JPp	yo@'cOV]2 E[܋?@,M3y_JP \
>HEX^JPŁH (ɻ}Z	 (^{R+AQ4 ER+AQ4 \
"qVhAj%(aVhDzm>HE6JPŷmV(>"qV( Em ( ΁Hm \
( m>HEQd{Aj%("s kAj%(ڷR+AQ},[RzX=DH~/uXQq{d \
^Nv}C_v@<h @4/E>W8>@QW E|R+AQwj%( \
jZ	h@E<PEQ0j%(bٹy \
Z	Pcp+gqzDere[9-E:cymAi`D%l<}';ߟӺǟ8Qq \
XAQA|Sŋjٴ2?|ɵxmǛyH~An/.1R \
Q4y;T+AQħ}2E1|uLV(2T+AQ }l (>JPEj g@EQ/g \
(} (߳ (ޓ}|1T+AQԻ}2E[ \
uLVs}|?T+AQԷ>JPm6c@E]+P+AQԳ>cBE=)@EI@Ey \
Z	~pe>j%(s:&T+AQԩ_k VKm@.JP}Z	\r \
҃ݺAj%(qG}Z	^̿)@̋JPu`2 Exr \
JP=yn}Z	"f/R+AQDg='c6;(҃?)Xpy:iKc&]u \
zϙH~I5 ga(zY2MD0>>=O},[[/)ĻyJJPJP5CJP5u㹧+JPkJPJP5u=UoJPŋBU \
(雗LR+AQT͗(@4o_o>HEQ ߿m}Z	⎼Xs}Z	 QdGN5Aj%(9 \
E=yLAj%(8Ɂ(NAj%(v:JPꤓ`VDcr  y4 \
(:y_[jR+AQts13^Dr>rB$yH篣>j2?c|>]_ea6  \
<Xy/lu^J)2A2{kH ͣ>A;`
\gf q6XuLZ\b/+FQ%nVhhR+AQOy8? \
(ڡܕU}Z	ʯ>HE6xVh'ZhR+AQVu4 \
(z:&2Wj<.ZJP=U{}Z	'R+AQVq@@Qd}Z	6'R+AQtj}Z	<P]4cVc \
JP=yL}ZyڪQNu~&m@'O@psWMdXy0 \
/J&h~e@AǓx>"<`X݋u}QTg쁴2<h e \
+R+AQZpV6>HEn=PA (3>HE >HE:>HE:>HE
;(JH+R+AQAR+AQaR+AQB<&>HEɁlD`JPX|VD \
(gg]R+AQ: \
S<Aj%(J[@vENaVtutR+AQzccVtutR+AQ::&(Jz>y\R+AQ:_G7 \
>HEA( XyR+AQ|m}Z	 d]JP ('P@kVG_ \
%<: ( @B<lV0[G  <GVzNG()m(9Qk \
F#i&OL1E>PgQTp \
}x::݇w^Aj%(@EɺǢ{Y5Ձ>Ҧ$p6t߸~D@-:GnѹއQaw(},[w(}Z	tR+AQtAj%(N`VT \
(:C}Z	 (:.>HEaAAj%(s EQQAj%( R+AQt)E \
(:$>HE!PiV舡j!>>^mHiD텢 dKm 8E9 \
$@m2?HZ@n[RHH㤁~т;9 \
g"8^,!X[}X,$>z,v ]u k{ FHς̟;#u  \
C"^.>6\{Na;tƃHΉ"(t O" >y h E; \
(.>HEfAj%(R+AQQ}Z	mJPm2>HE#JPm0>HEgcJP}4>HEG:P}2>HEc3^J[cm@z1>L5_/}q
 g?AK#s g},{Fq zc}<ˋt4>W{LןzR3>*m(:f}o<z#Qa \
)R=j :<a}8m4>j ċ
b7WSJP9@Q2>HEҸ EJP-8P͍R+AQ43>HE \
E7JP8P]R+AQ R+AQ}Z	_JPr V \
}LVhr@E@E@E@E@Eex \
JPѹP+AQ>@EeldJP9P+AQF>@EYe`c	JPqP+AQ>@EYeT \
(r@Eyj%(ʘ V}<j%(ʐ3 V}<j%(ʀs V}j%(x \
V}j%(hKV>(jr /6Uu \
֭LיJScx3;dt-/>n9@EJPaO@EJPQ8@EJP1
 @EJP!-@EJPm@EJPF V>%< \
(ѹ@E	}lj%(Jpd;P+AQbs;hx_Ec \
LOm(r;oenFm(N[yT:؇4Q>v+?߳p ;պˋtw}j%(JPc?P+AQr \
Z	 (!oZ	  (Z	 (8/Z	 (ѸZ	 \
(@E	}j%(J$P+AQqǀZ	9JP8A V>% q \
(A@E}JP@E}JP)@E	}JPZ	=qP+AQz>Nhe~FMu \
gqI	EDV._Ӧi}}K^-E }*=΁t}+=Ɓ}l}>y5r@5wv} \
JP^ӁZ	)q>P+AQ> KP+AQz>CP+AQ>
;P+AQz>3P+AQ:@/ \
P+AQ>J'(P+AQ:>0P+AQ>J8P+AQz@<P+AQ>*P+AQ@j
 P+AQ:>}Tj%(
JP<Q (tP+AQG= V9z@EasZ	>j}Tj%(

JP0QWV0h>Ыu>*+)m(ڪ:F+Ӵm Bz}TWi8Q:>p~}@r
THH?i @uw>Z}4j%(
hJP (<P+AQpG+ Vиf@EqZ	@}4j%(
h	JPTzFMu O9x:gG[igGQ>y}@Hk}4^,/)G{O6} Z	 \
@@EAp VP+AQ (JP (͹ P+AQZs \
V@0@Ei}pZ	ϯ)tj%(J]?wh 3z4 VH (Rj%(Z	"e VH \
(Rj%(Z	"e VH (Rj%(Z	"e VH (Rj%(Z	"e VH (Rj%(Z	"e VH \
(Rj%(Z	"e VH (Rj%(Z	"e VH (Rj%(Z	"e VH (Rj%(Z	"e \
V֍RGqUGqUGqU	GqU֟KwM텤Yz4763́Hohk[->*I$I.jG.i9ZۗS]}P^檷\ \
7ݶfO*߸~	$n[C Gx7iroKu6.Uqy _?s \
]ֽmEz{ݾW<y8Yw/*k}|+Ѫmꍃ+I$I$I$#y)IENDB`



_______________________________________________
Sent through the Full Disclosure mailing list
https://nmap.org/mailman/listinfo/fulldisclosure
Web Archives & RSS: http://seclists.org/fulldisclosure/

[prev in list] [next in list] [prev in thread] [next in thread]

Configure | About | News | Add a list | Sponsored by KoreLogic