I see that an dlg_unref is removed and not re-added -- from the patch:

            if(dlg_bye_all(dlg, NULL)<0)
-               dlg_unref(dlg, 1);

Any reason for that? If BYE failed to be be sent, then the dialog will hang around forever.

—
Reply to this email directly or view it on GitHub.