Hello, On 7/17/07, Stephen C. Steel <[EMAIL PROTECTED]> wrote: > I've attached a patch fixing an error in the function res_init_ip(const > char* localIp) found in the file "sipXtackLib/src/resparse/res_init.c". I > haven't seen an error due to the existing code, but it's obviously wrong by > inspection. > > 1. If the argument localIp is NULL, then the test "if (localIp || localIp[0] > == 0)" will dereference it, leading to a crash. > > 2. If the argument localIp is non-NULL, then the test " if (localIp || > localIp[0] == 0)" will always be true and will invoke the code to IGNORE > localIp and use the default value instead.
Fix checked in, thank you. -- Regards, Alexander Chemeris. SIPez LLC. SIP VoIP, IM and Presence Consulting http://www.SIPez.com tel: +1 (617) 273-4000 _______________________________________________ sipxtapi-dev mailing list [email protected] List Archive: http://list.sipfoundry.org/archive/sipxtapi-dev/
