This change looks good to me.

=============================================
Eric Hyche (ehyche at real.com)
Technical Lead
RealNetworks, Inc.  

> -----Original Message-----
> From: helix-client-dev-bounces at helixcommunity.org 
> [mailto:helix-client-dev-bounces at helixcommunity.org] On 
> Behalf Of David Audrain
> Sent: Monday, December 04, 2006 5:34 AM
> To: protocol-dev at helixcommunity.org; 
> helix-client-dev at helixcommunity.org
> Subject: [Helix-client-dev] CN-Client : Fix tcp transport 
> request handling
> 
> Synopsis:  Fix tcp transport request handling done in 
> RTSPClientProtocol::sendSetupRequestMessageExt
> 
> Overview: When HELIX_FEATURE_RTP is defined, and TCP 
> transport requests 
> are processed in
>          RTSPClientProtocol::sendSetupRequestMessageExt, a 
> crash occurs 
> in the second switch because
>          pRtpAddr and pRtcpAddr are still null.
> 
> Files Added: None
> 
> Files Modified:
> protocol/rtsp/rtspclnt.cpp
> 
> 
> Image Size and Heap Use impact (Client -Only): None
> 
> Platforms and Profiles Affected:
> 
> Distribution Libraries Affected: None
> 
> Distribution library impact and planned action:
> None
> 
> Platforms and Profiles Build Verified:
> win32-i386-vc6 helix-client-all-defines
> 
> Platforms and Profiles Functionality verified:
> win32-i386-vc6 helix-client-all-defines
> 
> Branch: hxclient_1_5_0_cayenne, head
> 
> Copyright assignment:
> My company Roundbox Inc submits this code under the terms
> of a commercial contribution agreement with RealNetworks,
> and I am authorized to contribute this code under said agreement.
> 
> QA Instructions:
> 
> -- 
> David Audrain
> Software Engineer
> 
> Roundbox
> 9 rue de conde
> 4 eme Etage
> 33270 Bordeaux
> FRANCE
> www.roundbox.com
> 
> T: +33 556.001.250
> F: +33 556.442.351
> 
> E: daudrain at roundbox.com
> Yahoo: daudrain33
> Skype: daudrain33
> 
>