fix gcc4-related bug in switch-robo
authorFelix Fietkau <nbd@openwrt.org>
Sat, 25 Mar 2006 14:31:42 +0000 (14:31 +0000)
committerFelix Fietkau <nbd@openwrt.org>
Sat, 25 Mar 2006 14:31:42 +0000 (14:31 +0000)
SVN-Revision: 3483

openwrt/target/linux/package/switch/src/switch-core.c
openwrt/target/linux/package/switch/src/switch-core.h
openwrt/target/linux/package/switch/src/switch-robo.c

index 8216646..6b59b9b 100644 (file)
@@ -68,13 +68,6 @@ static struct file_operations switch_proc_fops = {
        write: switch_proc_write
 };
 
        write: switch_proc_write
 };
 
-static inline char *strdup(char *str)
-{
-       char *new = kmalloc(strlen(str) + 1, GFP_KERNEL);
-       strcpy(new, str);
-       return new;
-}
-
 static ssize_t switch_proc_read(struct file *file, char *buf, size_t count, loff_t *ppos)
 {
 #ifdef LINUX_2_4
 static ssize_t switch_proc_read(struct file *file, char *buf, size_t count, loff_t *ppos)
 {
 #ifdef LINUX_2_4
index a6621ee..5292469 100644 (file)
@@ -48,4 +48,12 @@ extern switch_vlan_config *switch_parse_vlan(switch_driver *driver, char *buf);
 extern int switch_parse_media(char *buf);
 extern int switch_print_media(char *buf, int media);
 
 extern int switch_parse_media(char *buf);
 extern int switch_print_media(char *buf, int media);
 
+static inline char *strdup(char *str)
+{
+       char *new = kmalloc(strlen(str) + 1, GFP_KERNEL);
+       strcpy(new, str);
+       return new;
+}
+
+
 #endif
 #endif
index 7e4c4de..b23fca6 100644 (file)
@@ -51,6 +51,7 @@
 #define SIOCGETCPHYRD           (SIOCDEVPRIVATE + 9)
 #define SIOCSETCPHYWR           (SIOCDEVPRIVATE + 10)
 
 #define SIOCGETCPHYRD           (SIOCDEVPRIVATE + 9)
 #define SIOCSETCPHYWR           (SIOCDEVPRIVATE + 10)
 
+static char *device;
 static int use_et = 0;
 static int is_5350 = 0;
 static struct ifreq ifr;
 static int use_et = 0;
 static int is_5350 = 0;
 static struct ifreq ifr;
@@ -425,17 +426,18 @@ static int handle_reset(void *driver, char *buf, int nr)
 
 static int __init robo_init()
 {
 
 static int __init robo_init()
 {
-       char *device = "ethX";
        int notfound = 1;
 
        int notfound = 1;
 
+       device = strdup("ethX");
        for (device[3] = '0'; (device[3] <= '3') && notfound; device[3]++) {
                notfound = robo_probe(device);
        }
        device[3]--;
        
        for (device[3] = '0'; (device[3] <= '3') && notfound; device[3]++) {
                notfound = robo_probe(device);
        }
        device[3]--;
        
-       if (notfound)
+       if (notfound) {
+               kfree(device);
                return -ENODEV;
                return -ENODEV;
-       else {
+       else {
                switch_config cfg[] = {
                        {"enable", handle_enable_read, handle_enable_write},
                        {"enable_vlan", handle_enable_vlan_read, handle_enable_vlan_write},
                switch_config cfg[] = {
                        {"enable", handle_enable_read, handle_enable_write},
                        {"enable_vlan", handle_enable_vlan_read, handle_enable_vlan_write},
@@ -465,6 +467,7 @@ static int __init robo_init()
 static void __exit robo_exit()
 {
        switch_unregister_driver(DRIVER_NAME);
 static void __exit robo_exit()
 {
        switch_unregister_driver(DRIVER_NAME);
+       kfree(device);
 }
 
 
 }