Feed aggregator

asterisk (rmudgett-bridge_tasks): main/channel: Unlock channel in off-nominal path...

Asterisk Source - Sat, 2014-09-20 20:22
Commit Bot committed 423656 on branch rmudgett-bridge_tasks in asterisk main/channel: Unlock channel in off-nominal path

In r423414 (13) / r423415 (trunk), an API call that determines if a format

capability structure is empty was added. This returns true if the format

capability structure is completely empty or "none". A check for this was added

in channel.c's set_format call. Unfortunately, when this check was true, it

returned from the function while still holding the channel lock. This caused

the CDR unit tests - which have a tendency to create channels with no formats -

to deadlock. Whoops.

This patch unlocks the channel on the off-nominal path.

........

Merged revisions 423641 from http://svn.asterisk.org/svn/asterisk/branches/13

........

Merged revisions 423642 from file:///srv/subversion/repos/asterisk/trunk

· /team/rmudgett/bridge_tasks/main/channel.c (-0, +1) | History | Source | Diff · /team/rmudgett/bridge_tasks (-0, +0) | History | Source | Diff

Categories: Asterisk Project

asterisk (rmudgett-ao2_enhancements): main/channel: Unlock channel in off-nominal path...

Asterisk Source - Sat, 2014-09-20 20:22
Commit Bot committed 423655 on branch rmudgett-ao2_enhancements in asterisk main/channel: Unlock channel in off-nominal path

In r423414 (13) / r423415 (trunk), an API call that determines if a format

capability structure is empty was added. This returns true if the format

capability structure is completely empty or "none". A check for this was added

in channel.c's set_format call. Unfortunately, when this check was true, it

returned from the function while still holding the channel lock. This caused

the CDR unit tests - which have a tendency to create channels with no formats -

to deadlock. Whoops.

This patch unlocks the channel on the off-nominal path.

........

Merged revisions 423641 from http://svn.asterisk.org/svn/asterisk/branches/13

........

Merged revisions 423642 from file:///srv/subversion/repos/asterisk/trunk

· /team/rmudgett/ao2_enhancements/main/channel.c (-0, +1) | History | Source | Diff · /team/rmudgett/ao2_enhancements (-0, +0) | History | Source | Diff

Categories: Asterisk Project

asterisk (mmichelson-queue_bugbug): main/channel: Unlock channel in off-nominal path...

Asterisk Source - Sat, 2014-09-20 20:20
Commit Bot committed 423654 on branch mmichelson-queue_bugbug in asterisk main/channel: Unlock channel in off-nominal path

In r423414 (13) / r423415 (trunk), an API call that determines if a format

capability structure is empty was added. This returns true if the format

capability structure is completely empty or "none". A check for this was added

in channel.c's set_format call. Unfortunately, when this check was true, it

returned from the function while still holding the channel lock. This caused

the CDR unit tests - which have a tendency to create channels with no formats -

to deadlock. Whoops.

This patch unlocks the channel on the off-nominal path.

........

Merged revisions 423641 from http://svn.asterisk.org/svn/asterisk/branches/13

........

Merged revisions 423642 from file:///srv/subversion/repos/asterisk/trunk

· /team/mmichelson/queue_bugbug (-0, +0) | History | Source | Diff · /team/mmichelson/queue_bugbug/main/channel.c (-0, +1) | History | Source | Diff

Categories: Asterisk Project

asterisk (mmichelson-pubsub_bodies): main/channel: Unlock channel in off-nominal path...

Asterisk Source - Sat, 2014-09-20 20:20
Commit Bot committed 423653 on branch mmichelson-pubsub_bodies in asterisk main/channel: Unlock channel in off-nominal path

In r423414 (13) / r423415 (trunk), an API call that determines if a format

capability structure is empty was added. This returns true if the format

capability structure is completely empty or "none". A check for this was added

in channel.c's set_format call. Unfortunately, when this check was true, it

returned from the function while still holding the channel lock. This caused

the CDR unit tests - which have a tendency to create channels with no formats -

to deadlock. Whoops.

This patch unlocks the channel on the off-nominal path.

........

Merged revisions 423641 from http://svn.asterisk.org/svn/asterisk/branches/13

........

Merged revisions 423642 from file:///srv/subversion/repos/asterisk/trunk

· /team/mmichelson/pubsub_bodies/main/channel.c (-0, +1) | History | Source | Diff · /team/mmichelson/pubsub_bodies (-0, +0) | History | Source | Diff

Categories: Asterisk Project

asterisk (mmichelson-conversion_script): main/channel: Unlock channel in off-nominal path...

Asterisk Source - Sat, 2014-09-20 20:20
Commit Bot committed 423652 on branch mmichelson-conversion_script in asterisk main/channel: Unlock channel in off-nominal path

In r423414 (13) / r423415 (trunk), an API call that determines if a format

capability structure is empty was added. This returns true if the format

capability structure is completely empty or "none". A check for this was added

in channel.c's set_format call. Unfortunately, when this check was true, it

returned from the function while still holding the channel lock. This caused

the CDR unit tests - which have a tendency to create channels with no formats -

to deadlock. Whoops.

This patch unlocks the channel on the off-nominal path.

........

Merged revisions 423641 from http://svn.asterisk.org/svn/asterisk/branches/13

........

Merged revisions 423642 from file:///srv/subversion/repos/asterisk/trunk

· /team/mmichelson/conversion_script/main/channel.c (-0, +1) | History | Source | Diff · /team/mmichelson/conversion_script (-0, +0) | History | Source | Diff

Categories: Asterisk Project

Pages

Subscribe to Asterisk.org aggregator