diff options
author | Simon Horman <horms@verge.net.au> | 2006-09-13 22:58:44 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2006-09-18 02:21:05 -0400 |
commit | 3f5af5b353ca36aca4f8a46e3da2172f669dbbbc (patch) | |
tree | 25e92e2d951e8d6d709208a71fca6cc738e7cbcd /net/ipv4/ipvs | |
parent | 70e76b768bac81149c8c189e781c3f130ea2eba5 (diff) |
[IPVS]: Make sure ip_vs_ftp ports are valid
I'm not entirely sure what happens in the case of a valid port,
at best it'll be silently ignored. This patch ignores them a little
more verbosely.
Signed-Off-By: Simon Horman <horms@verge.net.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/ipvs')
-rw-r--r-- | net/ipv4/ipvs/ip_vs_ftp.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/net/ipv4/ipvs/ip_vs_ftp.c b/net/ipv4/ipvs/ip_vs_ftp.c index 347a66c05569..f0c553c038de 100644 --- a/net/ipv4/ipvs/ip_vs_ftp.c +++ b/net/ipv4/ipvs/ip_vs_ftp.c | |||
@@ -373,6 +373,12 @@ static int __init ip_vs_ftp_init(void) | |||
373 | for (i=0; i<IP_VS_APP_MAX_PORTS; i++) { | 373 | for (i=0; i<IP_VS_APP_MAX_PORTS; i++) { |
374 | if (!ports[i]) | 374 | if (!ports[i]) |
375 | continue; | 375 | continue; |
376 | if (ports[i] < 0 || ports[i] > 0xffff) { | ||
377 | IP_VS_WARNING("ip_vs_ftp: Ignoring invalid " | ||
378 | "configuration port[%d] = %d\n", | ||
379 | i, ports[i]); | ||
380 | continue; | ||
381 | } | ||
376 | ret = register_ip_vs_app_inc(app, app->protocol, ports[i]); | 382 | ret = register_ip_vs_app_inc(app, app->protocol, ports[i]); |
377 | if (ret) | 383 | if (ret) |
378 | break; | 384 | break; |