On Tue, 29.04.14 03:27, Tom Gundersen ([email protected]) wrote:

> +        case DHCP_OPTION_NTP_SERVER:
> +                if (len && !(len % 4)) {
> +                        unsigned i;
> +
> +                        lease->ntp_size = len / 4;
> +
> +                        free(lease->ntp);
> +                        lease->ntp = new0(struct in_addr, lease->ntp_size);
> +                        if (!lease->ntp)
> +                                return -ENOMEM;
> +
> +                        for (i = 0; i < lease->ntp_size; i++) {
> +                                memcpy(&lease->ntp[i].s_addr, option + 4 * 
> i, 4);
> +                        }

This should be shorter, no?

    lease->ntp = newdup(struct in_addr, option, lease->ntp_size);
    if (!lease->ntp) ...

Lennart

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

Reply via email to