Hi, I could not get the sessionStart event in an inbound DSM script with SDP offer in the 200 OK on current master. So I dug into the code and found a possible bug in AmSipDialog.cpp: The onSessionStart handler doesn't get called when the SDP state changes to completed and the session is already in state connected. Here's a diff that fixes the problem:
Regards,
Emil
diff --git a/core/AmSipDialog.cpp b/core/AmSipDialog.cpp
index a2d934e..576636f 100644
--- a/core/AmSipDialog.cpp
+++ b/core/AmSipDialog.cpp
@@ -381,6 +381,10 @@ int AmSipDialog::checkStateChange()
(oa_trans.state == OA_Completed)) {
ret = onSdpCompleted();
+
+ if (getStatus() == Connected) {
+ hdl->onSessionStart();
+ }
}
if(saved_status != getStatus()){
--
Emil Kroymann
VoIP Services Engineer
Email: [email protected]
Tel: +49-30-203899885
Mobile: +49-176-38389303
ISACO GmbH
Kurfürstenstraße 79
10787 Berlin
Germany
Amtsgericht Charlottenburg, HRB 112464B
Geschäftsführer: Daniel Frommherz
signature.asc
Description: PGP signature
_______________________________________________ Sems mailing list [email protected] http://lists.iptel.org/mailman/listinfo/sems
