On Mon, 25.11.13 09:13, Patrik Flykt (patrik.fl...@linux.intel.com) 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
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to