On Sun, Jun 29, 2014 at 9:21 PM, Lennart Poettering
<[email protected]> wrote:
> On Sun, 29.06.14 07:25, Tom Gundersen ([email protected]) wrote:
>
>> +
>> +        FOREACH_WORD(word, len, string, state) {
>> +                /* WORD FORMAT: dst_ip/dst_prefixlen,gw_ip */
>> +                _cleanup_free_ char* entry;
>
> missing initialization to NULL.

Thanks. Fixed.

>> +                char *tok, *tok_end;
>> +                unsigned n;
>> +                int r;
>> +
>> +                if (!GREEDY_REALLOC(routes, allocated, size + 1))
>> +                        return -ENOMEM;
>> +
>> +                entry = strndup(word, len);
>
> Missing OOM check.

Already got a patch from Susant for this. Pushed both.

-t
_______________________________________________
systemd-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to