LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

Re: [PATCH] ipvsadm: check if a virtual service exists before attempting

To: Emanuele Rocca <ema@xxxxxxxxxxxxx>
Subject: Re: [PATCH] ipvsadm: check if a virtual service exists before attempting removal
Cc: Jesper Dangaard Brouer <brouer@xxxxxxxxxx>, lvs-devel@xxxxxxxxxxxxxxx, lvs-users@xxxxxxxxxxxxxxxxxxxxxx
From: Julian Anastasov <ja@xxxxxx>
Date: Sat, 5 Aug 2017 14:33:40 +0300 (EEST)
        Hello,

On Mon, 31 Jul 2017, Emanuele Rocca wrote:

> Removing a non-existing service results in a confusing "Memory
> allocation problem" error message:
> 
>         # ipvsadm -D -t example.org:80
>         Memory allocation problem
> 
> Check if the given service exists before attempting removal. Inform the
> user if the service is not defined.
> 
> Signed-off-by: Emanuele Rocca <ema@xxxxxxxxxxxxx>
> ---
>  ipvsadm.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/ipvsadm.c b/ipvsadm.c
> index 1a28d72..f958b11 100644
> --- a/ipvsadm.c
> +++ b/ipvsadm.c
> @@ -942,6 +942,11 @@ static int process_options(int argc, char **argv, int 
> reading_stdin)
>               break;
>  
>       case CMD_DEL:
> +             if (!(ipvs_get_service(ce.svc.fwmark, ce.svc.af, 
> ce.svc.protocol,
> +                                                ce.svc.addr, ce.svc.port))) {
> +                     fprintf(stderr, "No such service\n");
> +                     exit(1);
> +             }
>               result = ipvs_del_service(&ce.svc);

        After some debugging, I think I found the cause for
this problem. It affects all command failures when ipvsadm
is compiled to use libnl, so I'm providing more general patch
to fix this problem. I'll post it soon...

Regards

--
Julian Anastasov <ja@xxxxxx>
--
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

<Prev in Thread] Current Thread [Next in Thread>
  • Re: [PATCH] ipvsadm: check if a virtual service exists before attempting removal, Julian Anastasov <=