[asterisk-bugs] [JIRA] (ASTERISK-20346) Modules need to ensure that any functions, apps, AMI actions, etc. they register are unregistered if the module declines loading

[Friendly Automation (JIRA)]

    [ https://issues.asterisk.org/jira/browse/ASTERISK-20346?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=241356#comment-241356 ] 

Friendly Automation commented on ASTERISK-20346:
------------------------------------------------

Change 7804 merged by Joshua Colp:
translators: Don't use ast_module_running_ref.

[https://gerrit.asterisk.org/7804|https://gerrit.asterisk.org/7804]

> Modules need to ensure that any functions, apps, AMI actions, etc. they register are unregistered if the module declines loading
> --------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: ASTERISK-20346
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-20346
>             Project: Asterisk
>          Issue Type: Bug
>      Security Level: None
>          Components: General
>    Affects Versions: SVN, 1.8.15.0, 10.7.0, 11.0.0-beta1, 13.18.4
>            Reporter: Mark Michelson
>
> ASTERISK-20305 brought to light that app_confbridge would register its functions and applications, then would attempt to load configuration. If configuration loading failed, then the module would not load, but the functions and applications it registered would still be registered. In the case of app_confbridge, this eventually led to a crash.
> Other modules need to be scanned and corrected as well. There are two methods of handling this:
> 1. Place any conditions that may result in AST_MODULE_LOAD_DECLINE before registering functions, applications, etc. This way, they will never be registered in the first place.
> 2. If (1) is not possible for a module, then ensure that cases where AST_MODULE_LOAD_DECLINE is returned unregister things properly.



--
This message was sent by Atlassian JIRA
(v6.2#6252)