commit cf06742ec8fd4e7ac7d393ebe093839379aa3f46 Author: Santiago Garcia Mantinan <bridge@manty.net> Date: Sat Aug 26 19:41:28 2006 +0200 patch for a message bug Hi! Jens Seidel reported a bug to Debian about a wrong message on brctl, you can see it at http://bugs.debian.org/383938, it is kind of the continuation of the ENODEV bug #348617 that was already patched by Stephen on the git. I have applied the same solution that Stephen applied for the first one, follows the patch against git plus a minor exclamation mark modification I did so that all ENODEV messages were exclamative (Stephen patch had missed that mark). Regards... -- Manty/BestiaTester -> http://manty.net Signed-off-by: Stephen Hemminger <shemminger@osdl.org> diff --git a/brctl/brctl_cmd.c b/brctl/brctl_cmd.c index a05d5b1..c93dd55 100644 --- a/brctl/brctl_cmd.c +++ b/brctl/brctl_cmd.c @@ -99,7 +99,7 @@ static int br_cmd_addif(int argc, char * if (if_nametoindex(ifname) == 0) fprintf(stderr, "interface %s does not exist!\n", ifname); else - fprintf(stderr, "bridge %s does not exist\n", brname); + fprintf(stderr, "bridge %s does not exist!\n", brname); break; case EBUSY: @@ -139,8 +139,10 @@ static int br_cmd_delif(int argc, char * continue; case ENODEV: - fprintf(stderr, "interface %s does not exist!\n", - ifname); + if (if_nametoindex(ifname) == 0) + fprintf(stderr, "interface %s does not exist!\n", ifname); + else + fprintf(stderr, "bridge %s does not exist!\n", brname); break; case EINVAL: