[CRIU] [PATCH 3/3] lib: Use x[re|m]alloc, xstrdup helpers
Pavel Emelyanov
xemul at parallels.com
Tue Sep 23 10:48:01 PDT 2014
On 09/23/2014 09:17 PM, Cyrill Gorcunov wrote:
> On Tue, Sep 23, 2014 at 09:08:38PM +0400, Pavel Emelyanov wrote:
>> On 09/23/2014 09:00 PM, Cyrill Gorcunov wrote:
>>
>> I doubt about this one. Do we want library to print memory
>> allocation errors to unknown log?
>>
>> And, btw, where from will libcriu.so get the print_on_level
>> function?
>
> We already use log helpers there
>
> perror("Can't allocate memory for criu opts");
> perror("Can't read response");
> ...
perror is not log helper. pr_perror is :)
> but this is a good question. As far as I see pr_ helpers
> remains unresolved indeed in library itself so it's up
> to library user to provide print_on_level otherwise we
> either
>
> 1) link libcriu with our log engine
please, no
> 2) eliminate all pr_ helpers from it
> .
>
More information about the CRIU
mailing list