[asterisk-dev] [Code Review] 2499: Fix Segfault In app_queue When "persistentmembers" Is Enabled And Using Realtime

[svnbot]

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

(Updated May 8, 2013, 10:30 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 388108


Bugs: ASTERISK-21738
    https://issues.asterisk.org/jira/browse/ASTERISK-21738


Repository: Asterisk


Description
-------

When the "ignorebusy" setting was deprecated, we added some code to allow us to be compatible with older setups that are still using the "ignorebusy" setting instead of "ringinuse".  We set a char *variable with the column name to use, which helps the realtime functions to use the correct column in their SQL queries.  When "persistentmembers" is enabled, we are not setting this variable before the realtime functions were called to load members.  This results in the variable being NULL and therefore causing a segfault when loading members during the module's process of loading.

The solution was to move the code that sets that variable to be before these realtime functions are called during the loading of the module.


Diffs
-----

  /branches/11/apps/app_queue.c 387094 

Diff: https://reviewboard.asterisk.org/r/2499/diff/


Testing
-------

Reporter tested this patch and reported back successful results.


Thanks,

Michael Young

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