[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Improving FCFS daemon

From: Schanzenbach, Martin
Subject: Re: Improving FCFS daemon
Date: Mon, 17 May 2021 11:40:36 +0000

Ah nevermind. It is because your make_json checks for that.
Looks good to me. The crash is fixed.
Can be merged from my point of view.
The only thing left is probably a link from the gns section on the homepage
to the fcfs service.


> On 17. May 2021, at 13:36, Schanzenbach, Martin <> 
> wrote:
> I am not sure of the NULL is a requirement of va_arg.
> The examples do not indicate that. And as I said the issue only arises when
> the argument is coming from gettext.
> So the issue lies somewhere else, but I do not know where.
> Maybe this is a general gettext/va_arg incompatibility.
> BR
>> On 16. May 2021, at 23:13, Alessio Vanni <> wrote:
>> Alessio Vanni <> writes:
>>> Huh, that's strange. Admittedly, I didn't properly test successful
>>> insertions, but I didn't get any crash on errors.
>>> I'll check it out.
>> I pushed a fix for this.
>> It was indeed because I had forgotten to put terminating NULL values in
>> the `make_json' function calls.
>>> I'll add a section in the handbook after fixing the crash.
>> I also pushed a draft of a section dedicated to FCFSD.
>> It was placed under GNS, as it's probably more appropriate.
>> I tried to be as informative as possible; please do provide feedback.
>> Thanks,
>> A.V.

Attachment: signature.asc
Description: Message signed with OpenPGP

reply via email to

[Prev in Thread] Current Thread [Next in Thread]