diff options
| author | Eric Sesterhenn <snakebyte@gmx.de> | 2006-09-29 05:01:32 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-09-29 12:18:23 -0400 |
| commit | 8c6ba51bdad0de0dc2a2e2415d4dac2d0db6c1a1 (patch) | |
| tree | 83727f2519978147ff509787ad6cb25379dd2cfd /drivers/net/wan/syncppp.c | |
| parent | 04b1db9fd7eea63c9663072feece616ea41b0a79 (diff) | |
[PATCH] Uninitialized variable in drivers/net/wan/syncppp.c
For len equal to 4, we never call sppp_lcp_conf_parse_options(),
therefore rmagic does not get initialized.
Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Acked-by: Paul Fulghum <paulkf@microgate.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/net/wan/syncppp.c')
| -rw-r--r-- | drivers/net/wan/syncppp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wan/syncppp.c b/drivers/net/wan/syncppp.c index d1173089f33..218f7b574ab 100644 --- a/drivers/net/wan/syncppp.c +++ b/drivers/net/wan/syncppp.c | |||
| @@ -469,7 +469,7 @@ static void sppp_lcp_input (struct sppp *sp, struct sk_buff *skb) | |||
| 469 | struct net_device *dev = sp->pp_if; | 469 | struct net_device *dev = sp->pp_if; |
| 470 | int len = skb->len; | 470 | int len = skb->len; |
| 471 | u8 *p, opt[6]; | 471 | u8 *p, opt[6]; |
| 472 | u32 rmagic; | 472 | u32 rmagic = 0; |
| 473 | 473 | ||
| 474 | if (!pskb_may_pull(skb, sizeof(struct lcp_header))) { | 474 | if (!pskb_may_pull(skb, sizeof(struct lcp_header))) { |
| 475 | if (sp->pp_flags & PP_DEBUG) | 475 | if (sp->pp_flags & PP_DEBUG) |