Hello,

On 12/4/06, yuhang <[EMAIL PROTECTED]> wrote:
> It is an old bug: assign >=2 ip addresses to one adapter under Win32.
> media-update branch tries to fix it, but the fix is not complete. The new 
> problem
> is a missing (numAddresses<maxAddresses) check in the inner loop. That's
> why VS2005 complains about stack corruption round local variable "const char*
> szAdapters[1] ;" when sipxInitialize() returns.
Looks good.
But is there any reason to change while() {} to do {} while()? I think we should
check pNextAddr && (numAddresses<maxAddresses) right after we have got
first pNextAddr:
PIP_ADDR_STRING pNextAddr = &pNextInfoRecord->IpAddressList;

-- 
Regards,
Alexander Chemeris.
_______________________________________________
sipxtapi-dev mailing list
[email protected]
List Archive: http://list.sipfoundry.org/archive/sipxtapi-dev/

Reply via email to