Hi!

As the Debian's maintainer of bridge-utils on my last upload to Debian I
cared to comment the source stating the bugs we had fixed on bridge-utils on
Debian that are still opened on your upstream version, you can get the patch
directly from our debian sources and forget about the debian dir, but I have
cut that out for you and what remains is this:

--- bridge-utils-1.5.orig/doc/brctl.8
+++ bridge-utils-1.5/doc/brctl.8
@@ -89,7 +89,7 @@
 .B brctl showmacs <brname>
 shows a list of learned MAC addresses for this bridge.
 
-.B brctl setageingtime <brname> <time>
+.B brctl setageing <brname> <time>
 sets the ethernet (MAC) address ageing time, in seconds. After <time>
 seconds of not having seen a frame coming from a certain address, the
 bridge will time out (delete) that address from the Forwarding
--- bridge-utils-1.5.orig/brctl/brctl.c
+++ bridge-utils-1.5/brctl/brctl.c
@@ -69,7 +69,8 @@
        argc -= optind;
        argv += optind;
        if ((cmd = command_lookup(*argv)) == NULL) {
-               fprintf(stderr, "never heard of command [%s]\n", argv[1]);
+/* Debian bug #406907 */
+               fprintf(stderr, "never heard of command [%s]\n", argv[0]);
                goto help;
        }
        
--- bridge-utils-1.5.orig/libbridge/libbridge_devif.c
+++ bridge-utils-1.5/libbridge/libbridge_devif.c
@@ -288,12 +288,16 @@
        char path[SYSFS_PATH_MAX];
        FILE *f;
 
-       snprintf(path, SYSFS_PATH_MAX, SYSFS_CLASS_NET "%s/%s", bridge, name);
+/* Debian bug #496491 */
+       snprintf(path, SYSFS_PATH_MAX, SYSFS_CLASS_NET "%s/bridge/%s", bridge, 
name);
 
        f = fopen(path, "w");
        if (f) {
                ret = fprintf(f, "%ld\n", value);
                fclose(f);
+/* Debian bug #574363 */
+               if (errno)
+                       ret=-1;
        } else {
                /* fallback to old ioctl */
                struct ifreq ifr;
@@ -355,6 +359,9 @@
        if (f) {
                ret = fprintf(f, "%ld\n", value);
                fclose(f);
+/* Debian bug #574363 */
+               if (errno)
+                       ret=-1;
        } else {
                int index = get_portno(bridge, ifname);
 

If you need comments on the bugs or the solutions or want us to test
different solutions for these bugs just let me know.

Thanks for your help.

Regards...
-- 
Manty/BestiaTester -> http://manty.net
_______________________________________________
Bridge mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/bridge

Reply via email to