[OpenSIPS-Devel] [OpenSIPS/opensips] 5451ab: rest_client: add async capabilities to rest_post()

[Liviu Chircu]

That is due to the way libcurl uses our internal allocator.

However, it still looks annoying. Since the libc free() tolerates 
free(NULL), F_MALLOC should do the same, and that should be a DBG 
message. Same cannot be said for DBG_QM_MALLOC, which should throw the 
warning just to speed up debugging.

Best regards,

Liviu Chircu
OpenSIPS Developer
http://www.opensips-solutions.com

On 21.04.2015 19:48, Ovidiu Sas wrote:
> Actually, the return code it's working ok.
> However, there are some issues while handling rest_get().  There are warnings:
> WARNING:core:fm_free: free(0) called
>
> -ovidiu
>
> On Tue, Apr 21, 2015 at 12:37 PM, Ovidiu Sas <osas at voipembedded.com> wrote:
>> The documentation states that $rc can be tested in the resume route:
>> http://www.opensips.org/html/docs/modules/2.1.x/rest_client.html#rest_get
>> It seems that this is incorrect.
>> Please confirm.
>>
>> Regards,
>> Ovidiu Sas
>>
>> On Wed, Mar 4, 2015 at 1:34 PM, Liviu Chircu <liviu at opensips.org> wrote:
>>>    Branch: refs/heads/master
>>>    Home:   https://github.com/OpenSIPS/opensips
>>>    Commit: 5451ab6d720d46ab0ede4a27f117510c59dbffc3
>>>        https://github.com/OpenSIPS/opensips/commit/5451ab6d720d46ab0ede4a27f117510c59dbffc3
>>>    Author: Liviu Chircu <liviu at opensips.org>
>>>    Date:   2015-03-04 (Wed, 04 Mar 2015)
>>>
>>>    Changed paths:
>>>      M modules/rest_client/README
>>>      M modules/rest_client/doc/rest_client_admin.xml
>>>      M modules/rest_client/rest_client.c
>>>      M modules/rest_client/rest_methods.c
>>>      M modules/rest_client/rest_methods.h
>>>
>>>    Log Message:
>>>    -----------
>>>    rest_client: add async capabilities to rest_post()
>>>
>>>
>>>
>>> _______________________________________________
>>> Devel mailing list
>>> Devel at lists.opensips.org
>>> http://lists.opensips.org/cgi-bin/mailman/listinfo/devel
>>>
>>
>>
>> --
>> VoIP Embedded, Inc.
>> http://www.voipembedded.com
>
>