I'm not sure this is 100% squeaky clean... Shouldn't the delete be in the same branch where bSuccess is set to true? otherwise there might be a double-delete scenario... (e.g. addLine succeeds but addCredentialForLine fails); and to extra prevent this, it should set line=NULL when it deletes line.
I should probably do this in the SAA too. Carolyn ________________________________ From: [email protected] [mailto:[email protected]] On Behalf Of [email protected] Sent: Wednesday, March 17, 2010 3:04 PM To: [email protected] Subject: sipXecs 18336 dworley: Delete SipLine object after we have used it. Project sipXecs New Revision 18336<http://code.sipfoundry.org//changelog/sipXecs/?cs=18336> Committer dworley Date 2010-03-17 14:03:58 -0500 (Wed, 17 Mar 2010) Log Delete SipLine object after we have used it. Modified: * main/sipXrls/src/main.cpp <http://code.sipfoundry.org//browse/sipXecs/main/sipXrls/src/main.cpp?r1=18335&r2=18336>
_______________________________________________ sipx-dev mailing list [email protected] List Archive: http://list.sipfoundry.org/archive/sipx-dev Unsubscribe: http://list.sipfoundry.org/mailman/listinfo/sipx-dev sipXecs IP PBX -- http://www.sipfoundry.org/
