<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" "http://www.w3.org/TR/REC-html40/strict.dtd"><html><head><meta name="qrichtext" content="1" /><style type="text/css">p, li { white-space: pre-wrap; }</style></head><body style=" font-family:'Lucida Console'; font-size:9pt; font-weight:400; font-style:normal;">On Dienstag, 6. Oktober 2009, Andrei Pelinescu-Onciul wrote:<br>
&gt; &gt; &gt; I remember that Miklos changes in the last week something in the cfg<br>
&gt; &gt; &gt; framework, perhaps its related to this?<br>
&gt;<br>
&gt; Should be fixed now (at least the config changes are still seen on exit<br>
&gt; by all the processes and mi_fifo does not crash).<br>
&gt;<br>
&gt; &gt; It's my patch so I'm responsible :-)<br>
&gt; &gt; It is related to it. The 0 pointer crash is  intentional and it<br>
&gt; &gt; should signal that someone uses cfg_update() on a process not registered<br>
&gt; &gt; for config framework use.<br>
&gt; &gt; Unfortunately my patch added a cfg_update() on process exit (to make<br>
&gt; &gt; sure they have the latest update values of the debug, memlog and<br>
&gt; &gt; memdebug) and I failed to account for kamailio modules that fork<br>
&gt; &gt; processes.<br>
&gt; &gt;<br>
&gt; &gt; So I'll either disable the 0-crash (which has the disadvantage that in<br>
&gt; &gt; the future one might try to use the cfg. framework without proper<br>
&gt; &gt; intialization and it would seem to work at least for some of the vars =&gt;<br>
&gt; &gt;  difficult debugging), remove the cfg_update() on exit (no memlog, debug<br>
&gt; &gt;  update in some cases) or come up with something else.<br>
&gt; &gt; Another alternative would be to update all the k modules, but I want to<br>
&gt; &gt; avoid it since they don't use the cfg framework yet and it would<br>
&gt; &gt; introduce a lot of extra testing before a release.<br>
&gt;<br>
&gt; In the end, I just added a special update call, that is used only on<br>
&gt; process exit, before dumping memory debugging info.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>Thanks Andrei,<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>it not crashes anymore now.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>Henning</p></body></html>