On Mon, 25.11.13 09:13, Patrik Flykt ([email protected]) wrote:

> -static int client_receive_offer(DHCPClient *client,
> -                                DHCPPacket *offer, int len)
> +static int client_verify_headers(DHCPClient *client,
> +                                 DHCPPacket *message, int len)

Thhis all looks like "size_t len" is the right param here, not "int".

> +        lease = new0(DHCPLease, 1);
> +        if (!lease)
> +                return -ENOBUFS;

OOM should result in ENOMEM, nothing else please. ENOBUFS is something
else: it indicates some local limit to be hit, not a global memory
shortage.

Lennart

-- 
Lennart Poettering, Red Hat
_______________________________________________
systemd-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to