[asterisk-dev] [Code Review] 2763: Allow res_parking to be unloadable

jrose reviewboard at asterisk.org
Fri Aug 16 17:23:16 CDT 2013



> On Aug. 16, 2013, 9:37 p.m., rmudgett wrote:
> > /trunk/main/parking.c, line 229
> > <https://reviewboard.asterisk.org/r/2763/diff/6/?file=44123#file44123line229>
> >
> >     While you are here, this message should be removed.  If no module is registered then it has "successfully" unregistered the parking features.  It also causes an unnecessary load failure message.

It seems to me like we should have an assertion in here instead. Attempting to unregister parking features when there aren't any loaded seems like an error of some kind.


- jrose


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2763/#review9423
-----------------------------------------------------------


On Aug. 16, 2013, 8:14 p.m., opticron wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2763/
> -----------------------------------------------------------
> 
> (Updated Aug. 16, 2013, 8:14 p.m.)
> 
> 
> Review request for Asterisk Developers and jrose.
> 
> 
> Bugs: ASTERISK-22142
>     https://issues.asterisk.org/jira/browse/ASTERISK-22142
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> This change protects accesses of res_parking such that it can unload safely once transient uses of its registered functions are complete. The parking API has been restructured such that its consumers do not have access to the vtable exposed by the parking provider, but instead route through stubs to prevent consumers from holding on to function pointers.
> 
> This adds calls to all the parking unload functions and moves application loading and unloading into functions in parking_applications.c similar to the rest of the parts of res_parking.
> 
> 
> Diffs
> -----
> 
>   /trunk/channels/chan_dahdi.c 396876 
>   /trunk/channels/chan_skinny.c 396876 
>   /trunk/channels/sig_analog.c 396876 
>   /trunk/include/asterisk/parking.h 396876 
>   /trunk/main/bridge.c 396876 
>   /trunk/main/bridge_channel.c 396876 
>   /trunk/main/parking.c 396876 
>   /trunk/res/parking/parking_applications.c 396876 
>   /trunk/res/parking/parking_bridge_features.c 396876 
>   /trunk/res/parking/res_parking.h 396876 
>   /trunk/res/res_parking.c 396876 
> 
> Diff: https://reviewboard.asterisk.org/r/2763/diff/
> 
> 
> Testing
> -------
> 
> Unloaded and reloaded res_parking ensuring that created dialplan was removed and that tests passed once reloaded.
> 
> 
> Thanks,
> 
> opticron
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130816/b492bccc/attachment.htm>


More information about the asterisk-dev mailing list