[asterisk-dev] [Code Review] Resolve odbc segfaults by adding shared locks around usage of odbc handle in res_odbc

[jrose]

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

(Updated Feb. 24, 2012, 4:44 p.m.)


Review request for Asterisk Developers, Mark Michelson, Tilghman Lesher, rmudgett, and wdoekes.


Changes
-------

1.8 version of patch from before.  This one uses mutex locks, and will be merged up to trunk before doing the change to ao2 locks.


Summary
-------

This is a continuation of wdoekes patch seen in https://reviewboard.asterisk.org/r/1622/
This adds Tilghman Lesher's suggested changes.


This addresses bug ASTERISK-19011.
    https://issues.asterisk.org/jira/browse/ASTERISK-19011


Diffs (updated)
-----

  /branches/1.8/res/res_odbc.c 354164 

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


Testing
-------

I started by confirming that the segfault could be easily reproduced with wdoekes' setup mentioned in the issue report.
After finishing the changes to the patch, I confirmed that following the same steps no longer produced the segfault, undid the patch to double check that I was doing it right, and then decided it was safe to post this review.


Thanks,

jrose

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