Fix crash during destruction of second channel when variable set with setvar.
authorJeff Peeler <jpeeler@digium.com>
Tue, 6 Oct 2009 19:24:59 +0000 (19:24 +0000)
committerJeff Peeler <jpeeler@digium.com>
Tue, 6 Oct 2009 19:24:59 +0000 (19:24 +0000)
The setvar line in chan_dahdi.conf is shared among all the channels, so make
sure to only free the resources only when the last channel is destroyed.

(closes issue #15899)
Reported by: tzafrir

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@222298 65c4cc65-6c06-0410-ace0-fbb531ad65f3

channels/chan_dahdi.c

index 8a43bac..0af7d7d 100644 (file)
@@ -4881,8 +4881,9 @@ static void destroy_dahdi_pvt(struct dahdi_pvt *pvt)
                ast_smdi_interface_unref(p->smdi_iface);
        if (p->mwi_event_sub)
                ast_event_unsubscribe(p->mwi_event_sub);
-       if (p->vars)
+       if (p->vars && iflist && !iflist->next) {
                ast_variables_destroy(p->vars);
+       }
        ast_mutex_destroy(&p->lock);
        dahdi_close_sub(p, SUB_REAL);
        if (p->owner)