[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/
-----------------------------------------------------------

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


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
-----

  /branches/1.8/include/asterisk/res_odbc.h 354164 
  /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/20120207/cf5e48d1/attachment-0001.htm>