[Asterisk-code-review] loader: Allow declined modules to be unloaded. (asterisk[master])

N A asteriskteam at digium.com
Thu Dec 8 15:49:52 CST 2022


N A has uploaded a new patch set (#2). ( https://gerrit.asterisk.org/c/asterisk/+/19657 )

Change subject: loader: Allow declined modules to be unloaded.
......................................................................

loader: Allow declined modules to be unloaded.

Currently, if a module declines to load, dlopen is called
to register the module but dlclose never gets called.
Furthermore, loader.c currently doesn't allow dlclose
to ever get called on the module, since it declined to
load and the unload function bails early in this case.

This can be problematic if a module is updated, since the
new module cannot be loaded into memory since we haven't
closed all references to it. To fix this, we now allow
modules to be unloaded, even if they never "loaded" in
Asterisk itself, so that dlclose is called and the module
can be properly cleaned up, allowing the updated module
to be loaded from scratch next time.

ASTERISK-30345 #close

Change-Id: Ifc743aadfa85ebe3284e02a63e124dafa64988d5
---
M main/loader.c
1 file changed, 37 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/57/19657/2
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/19657
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: Ifc743aadfa85ebe3284e02a63e124dafa64988d5
Gerrit-Change-Number: 19657
Gerrit-PatchSet: 2
Gerrit-Owner: N A <asterisk at phreaknet.org>
Gerrit-CC: Friendly Automation
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20221208/04ec119c/attachment.html>


More information about the asterisk-code-review mailing list