[asterisk-commits] coreyfarrell: trunk r408699 - /trunk/channels/chan_sip.c

[SVN commits to the Asterisk project]

Author: coreyfarrell
Date: Fri Feb 21 10:49:03 2014
New Revision: 408699

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=408699
Log:
chan_sip: prevent add_route from adding empty header.

Fix regression caused by ASTERISK-22582.  Empty Route
headers were added when the route had a single strict
hop.

(closes issue ASTERISK-23306)
Reported by: Matt Jordan
Review: https://reviewboard.asterisk.org/r/3236/

Modified:
    trunk/channels/chan_sip.c

Modified: trunk/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/trunk/channels/chan_sip.c?view=diff&rev=408699&r1=408698&r2=408699
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Fri Feb 21 10:49:03 2014
@@ -11687,7 +11687,9 @@
 	}
 
 	if ((r = sip_route_list(route, 0, skip))) {
-		add_header(req, "Route", ast_str_buffer(r));
+		if (ast_str_strlen(r)) {
+			add_header(req, "Route", ast_str_buffer(r));
+		}
 		ast_free(r);
 	}
 }