|
On Sat, 5 Aug 2017 14:38:28 +0300
Julian Anastasov <ja@xxxxxx> wrote:
> nl_recvmsgs_default() returns NLE_* error codes and not
> errno values. As result, attempt to delete virtual service
> returns NLE_OBJ_NOTFOUND (12) which matches the ENOMEM value.
>
> Problem as reported by Emanuele Rocca:
>
> ipvsadm -D -t example.org:80
> Memory allocation problem
>
> Fix it by providing generic error handler to catch the errno
> value as returned in netlink answer. By this way all netlink
> commands will get proper error string. The problem is present
> only when ipvsadm is compiled with libnl.
>
> ipvsadm -D -t example.org:80
> No such service
>
> Reported-by: Emanuele Rocca <ema@xxxxxxxxxxxxx>
> Signed-off-by: Julian Anastasov <ja@xxxxxx>
I've gone through the full call path from the kernel via netlink, and I
was going to claim that we also needed to handle errno "-EEXIST" in the
ipvsadm translation function ipvs_strerror(). Note, this fix uses the
errno "-ESRCH".
As kernel function ip_vs_del_service() return -EEXIST (if svc==NULL)
http://elixir.free-electrons.com/linux/v4.12.5/source/net/netfilter/ipvs/ip_vs_ctl.c#L1480
BUT a closer look shows that -EEXIST will never get returned by
ip_vs_del_service() as all callers of this function does a svc==NULL
check and return "-ESRCH" instead.
Thus, this patch is correct, but the kernel code is confusing ;-)
Applied:
https://git.kernel.org/pub/scm/utils/kernel/ipvsadm/ipvsadm.git/commit/?id=f8cff0808a24b
--
Best regards,
Jesper Dangaard Brouer
MSc.CS, Principal Kernel Engineer at Red Hat
LinkedIn: http://www.linkedin.com/in/brouer
--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
|
