|Summary:||ASTERISK-07153: [patch] res_osp: remove unneeded error messages|
|Reporter:||Andrey S Pankov (casper)||Labels:|
|Date Opened:||2006-06-12 17:29:51||Date Closed:||2006-06-15 07:27:28|
|Environment:||Attachments:||( 0) res_osp.c.r33725.diff|
|Description:||There is no need to output errors on ast_calloc failures.|
|Comments:||By: Tilghman Lesher (tilghman) 2006-06-13 22:03:56|
Nonsense. There is always a need to report out of memory conditions, when possible.
By: Andrey S Pankov (casper) 2006-06-15 05:51:04
Oh yeah... now I reopen the issue to show that you are 100% wrong here.
Please open include/asterisk/utils.h file at line 310 and read _carefully_
description above and the code below. "Memory conditions" are already
handled there in a pretty standard way. Even if built with MALLOC_DEBUG
memory allocation failures are reported as well. Another thing here is that
_current_ coding practice is not to report "out of memory" when using
And I hope you'll review your opinion as to ASTERISK-7287342 "won't fix"...
By: Tilghman Lesher (tilghman) 2006-06-15 07:27:28
Fixed in revision 34240.