diff options
author | Kaihui Luo <kaih.luo@gmail.com> | 2008-09-22 22:02:36 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-09-22 22:02:36 -0400 |
commit | 2cdc55751c33829f00510e0104562d0f8d8a9b85 (patch) | |
tree | 546f473942d6f0dba3d1b4dca2039073b0e20e87 /net | |
parent | 147e70e62fdd5af6263106ad634b03c5154c1e56 (diff) |
netfilter: xt_time gives a wrong monthday in a leap year
The function localtime_3 in xt_time.c gives a wrong monthday in a leap
year after 28th 2. calculating monthday should use the array
days_since_leapyear[] not days_since_year[] in a leap year.
Signed-off-by: Kaihui Luo <kaih.luo@gmail.com>
Acked-by: Jan Engelhardt <jengelh@computergmbh.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r-- | net/netfilter/xt_time.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/net/netfilter/xt_time.c b/net/netfilter/xt_time.c index 9f328593287e..307a2c3c2df4 100644 --- a/net/netfilter/xt_time.c +++ b/net/netfilter/xt_time.c | |||
@@ -136,17 +136,19 @@ static void localtime_3(struct xtm *r, time_t time) | |||
136 | * from w repeatedly while counting.) | 136 | * from w repeatedly while counting.) |
137 | */ | 137 | */ |
138 | if (is_leap(year)) { | 138 | if (is_leap(year)) { |
139 | /* use days_since_leapyear[] in a leap year */ | ||
139 | for (i = ARRAY_SIZE(days_since_leapyear) - 1; | 140 | for (i = ARRAY_SIZE(days_since_leapyear) - 1; |
140 | i > 0 && days_since_year[i] > w; --i) | 141 | i > 0 && days_since_leapyear[i] > w; --i) |
141 | /* just loop */; | 142 | /* just loop */; |
143 | r->monthday = w - days_since_leapyear[i] + 1; | ||
142 | } else { | 144 | } else { |
143 | for (i = ARRAY_SIZE(days_since_year) - 1; | 145 | for (i = ARRAY_SIZE(days_since_year) - 1; |
144 | i > 0 && days_since_year[i] > w; --i) | 146 | i > 0 && days_since_year[i] > w; --i) |
145 | /* just loop */; | 147 | /* just loop */; |
148 | r->monthday = w - days_since_year[i] + 1; | ||
146 | } | 149 | } |
147 | 150 | ||
148 | r->month = i + 1; | 151 | r->month = i + 1; |
149 | r->monthday = w - days_since_year[i] + 1; | ||
150 | return; | 152 | return; |
151 | } | 153 | } |
152 | 154 | ||