@joeygo commented on this pull request.


> +     }       
+
+       if(rank == PROC_MAIN) {
+               if(rtpengine_dtmf_event_sock.len > 0) {
+                       LM_DBG("Register RTPENGINE DTMF WORKER %d\n", mypid);
+                       /* fork worker process */
+                       mypid = fork_process(PROC_RPC, "RTPENGINE DTMF WORKER", 
1);
+                       if(mypid < 0) {
+                               LM_ERR("failed to fork RTPENGINE DTMF WORKER 
process %d\n", mypid);
+                               return -1;
+                       } else if(mypid == 0) {
+                               if(cfg_child_init())
+                                       return -1;
+                               /* this will loop forever */
+                               rtpengine_dtmf_events_loop();
+                       } 

I don't think it is right to kill the entire kamailio process due to an exit in 
such function.
I print error logs in case of failure in the function. isn't it enough?
If you prefer that I'll kill the process just let me know and I'll do that.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3473#discussion_r1229071053
You are receiving this because you are subscribed to this thread.

Message ID: <kamailio/kamailio/pull/3473/review/1478598...@github.com>
_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org

Reply via email to