Claude Klimos wrote:
218,220d217 < < if (vp) < free (vp);
Your patch is backwards, it shows deleting these two lines, when actually you are adding them.
It's not necessary to check if vp is non-NULL, because the code a couple of lines above has already done that.
I can't say for sure that freeing the val structure here is correct, but it does appear to be the right thing to do.
You should post a bug in bugs.digium.com, with the correctly-formatted patch, to get it fixed in the tree.
_______________________________________________
Asterisk-Dev mailing list
[EMAIL PROTECTED]
http://lists.digium.com/mailman/listinfo/asterisk-dev
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
