diff options
| author | Gabor Juhos <juhosg@openwrt.org> | 2009-12-25 10:06:22 +0000 |
|---|---|---|
| committer | Gabor Juhos <juhosg@openwrt.org> | 2009-12-25 10:06:22 +0000 |
| commit | 695d0b4cedd9f45019cf9557ef2c925706437a2b (patch) | |
| tree | 3b799bbecf6e8d28f23e44357c0d9be099b18a48 /target | |
| parent | 6887b9b476aada30d551d8b96fa88bb7ba676620 (diff) | |
| download | upstream-695d0b4cedd9f45019cf9557ef2c925706437a2b.tar.gz upstream-695d0b4cedd9f45019cf9557ef2c925706437a2b.tar.bz2 upstream-695d0b4cedd9f45019cf9557ef2c925706437a2b.zip | |
ledtrig-netdev: use dev_get_stats to get statistics on kernels >=2.6.30
Fixes a possible NULL pointer dereference.
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@18925 3c298f89-4303-0410-b956-a3cf2f4a3e73
Diffstat (limited to 'target')
3 files changed, 12 insertions, 21 deletions
diff --git a/target/linux/generic-2.6/files-2.6.30/drivers/leds/ledtrig-netdev.c b/target/linux/generic-2.6/files-2.6.30/drivers/leds/ledtrig-netdev.c index 4193e4eda3..8dba8e654d 100644 --- a/target/linux/generic-2.6/files-2.6.30/drivers/leds/ledtrig-netdev.c +++ b/target/linux/generic-2.6/files-2.6.30/drivers/leds/ledtrig-netdev.c @@ -46,7 +46,7 @@ * link: LED's normal state reflects whether the link is up (has carrier) or not * tx: LED blinks on transmitted data * rx: LED blinks on receive data - * + * * Some suggestions: * * Simple link status LED: @@ -307,7 +307,7 @@ done: static void netdev_trig_timer(unsigned long arg) { struct led_netdev_data *trigger_data = (struct led_netdev_data *)arg; - struct net_device_stats *dev_stats; + const struct net_device_stats *dev_stats; unsigned new_activity; write_lock(&trigger_data->lock); @@ -317,11 +317,8 @@ static void netdev_trig_timer(unsigned long arg) led_set_brightness(trigger_data->led_cdev, ((trigger_data->mode & MODE_LINK) != 0 && trigger_data->link_up) ? LED_FULL : LED_OFF); goto no_restart; } -#ifdef CONFIG_COMPAT_NET_DEV_OPS - dev_stats = trigger_data->net_dev->get_stats(trigger_data->net_dev); -#else - dev_stats = trigger_data->net_dev->netdev_ops->ndo_get_stats(trigger_data->net_dev); -#endif + + dev_stats = dev_get_stats(trigger_data->net_dev); new_activity = ((trigger_data->mode & MODE_TX) ? dev_stats->tx_packets : 0) + ((trigger_data->mode & MODE_RX) ? dev_stats->rx_packets : 0); diff --git a/target/linux/generic-2.6/files-2.6.31/drivers/leds/ledtrig-netdev.c b/target/linux/generic-2.6/files-2.6.31/drivers/leds/ledtrig-netdev.c index 4193e4eda3..8dba8e654d 100644 --- a/target/linux/generic-2.6/files-2.6.31/drivers/leds/ledtrig-netdev.c +++ b/target/linux/generic-2.6/files-2.6.31/drivers/leds/ledtrig-netdev.c @@ -46,7 +46,7 @@ * link: LED's normal state reflects whether the link is up (has carrier) or not * tx: LED blinks on transmitted data * rx: LED blinks on receive data - * + * * Some suggestions: * * Simple link status LED: @@ -307,7 +307,7 @@ done: static void netdev_trig_timer(unsigned long arg) { struct led_netdev_data *trigger_data = (struct led_netdev_data *)arg; - struct net_device_stats *dev_stats; + const struct net_device_stats *dev_stats; unsigned new_activity; write_lock(&trigger_data->lock); @@ -317,11 +317,8 @@ static void netdev_trig_timer(unsigned long arg) led_set_brightness(trigger_data->led_cdev, ((trigger_data->mode & MODE_LINK) != 0 && trigger_data->link_up) ? LED_FULL : LED_OFF); goto no_restart; } -#ifdef CONFIG_COMPAT_NET_DEV_OPS - dev_stats = trigger_data->net_dev->get_stats(trigger_data->net_dev); -#else - dev_stats = trigger_data->net_dev->netdev_ops->ndo_get_stats(trigger_data->net_dev); -#endif + + dev_stats = dev_get_stats(trigger_data->net_dev); new_activity = ((trigger_data->mode & MODE_TX) ? dev_stats->tx_packets : 0) + ((trigger_data->mode & MODE_RX) ? dev_stats->rx_packets : 0); diff --git a/target/linux/generic-2.6/files-2.6.32/drivers/leds/ledtrig-netdev.c b/target/linux/generic-2.6/files-2.6.32/drivers/leds/ledtrig-netdev.c index 4193e4eda3..8dba8e654d 100644 --- a/target/linux/generic-2.6/files-2.6.32/drivers/leds/ledtrig-netdev.c +++ b/target/linux/generic-2.6/files-2.6.32/drivers/leds/ledtrig-netdev.c @@ -46,7 +46,7 @@ * link: LED's normal state reflects whether the link is up (has carrier) or not * tx: LED blinks on transmitted data * rx: LED blinks on receive data - * + * * Some suggestions: * * Simple link status LED: @@ -307,7 +307,7 @@ done: static void netdev_trig_timer(unsigned long arg) { struct led_netdev_data *trigger_data = (struct led_netdev_data *)arg; - struct net_device_stats *dev_stats; + const struct net_device_stats *dev_stats; unsigned new_activity; write_lock(&trigger_data->lock); @@ -317,11 +317,8 @@ static void netdev_trig_timer(unsigned long arg) led_set_brightness(trigger_data->led_cdev, ((trigger_data->mode & MODE_LINK) != 0 && trigger_data->link_up) ? LED_FULL : LED_OFF); goto no_restart; } -#ifdef CONFIG_COMPAT_NET_DEV_OPS - dev_stats = trigger_data->net_dev->get_stats(trigger_data->net_dev); -#else - dev_stats = trigger_data->net_dev->netdev_ops->ndo_get_stats(trigger_data->net_dev); -#endif + + dev_stats = dev_get_stats(trigger_data->net_dev); new_activity = ((trigger_data->mode & MODE_TX) ? dev_stats->tx_packets : 0) + ((trigger_data->mode & MODE_RX) ? dev_stats->rx_packets : 0); |