diff options
| author | Simon Horman <horms@verge.net.au> | 2006-09-27 22:53:24 -0700 |
|---|---|---|
| committer | David S. Miller <davem@sunset.davemloft.net> | 2006-09-28 18:02:56 -0700 |
| commit | 28b06c380fca7e248c90e45f0aad1753097f6824 (patch) | |
| tree | 7070f82b1f85780927e69abe8d8aa50bed9655cd /net/ipv4/ipvs/ip_vs_ftp.c | |
| parent | e44fd82caf89a8548e7868592618b243e85c69e6 (diff) | |
| download | kernel-crypto-28b06c380fca7e248c90e45f0aad1753097f6824.tar.gz kernel-crypto-28b06c380fca7e248c90e45f0aad1753097f6824.tar.xz kernel-crypto-28b06c380fca7e248c90e45f0aad1753097f6824.zip | |
[IPVS]: Make sure ip_vs_ftp ports are valid: module_param_array approach
I'm not entirely sure what happens in the case of a valid port,
at best it'll be silently ignored. This patch ensures that
the port values are unsigned short values, and thus always valid.
This is a second take at fixing this problem, it is simpler
and arguably more correct than the previous approach
that was committed as 3f5af5b353ca36aca4f8a46e3da2172f669dbbbc.
Prior to this patch a patch that reverses
3f5af5b353ca36aca4f8a46e3da2172f669dbbbc was sent.
Signed-off-by: Simon Horman <horms@verge.net.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/ipvs/ip_vs_ftp.c')
| -rw-r--r-- | net/ipv4/ipvs/ip_vs_ftp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/ipv4/ipvs/ip_vs_ftp.c b/net/ipv4/ipvs/ip_vs_ftp.c index 4df215c51c7..62505d8f9ff 100644 --- a/net/ipv4/ipvs/ip_vs_ftp.c +++ b/net/ipv4/ipvs/ip_vs_ftp.c @@ -44,8 +44,8 @@ * List of ports (up to IP_VS_APP_MAX_PORTS) to be handled by helper * First port is set to the default port. */ -static int ports[IP_VS_APP_MAX_PORTS] = {21, 0}; -module_param_array(ports, int, NULL, 0); +static unsigned short ports[IP_VS_APP_MAX_PORTS] = {21, 0}; +module_param_array(ports, ushort, NULL, 0); MODULE_PARM_DESC(ports, "Ports to monitor for FTP control commands"); |