Ticket #1336: madwifi-0.9.3-debug-usability-enhancement.2.diff

File madwifi-0.9.3-debug-usability-enhancement.2.diff, 2.9 kB (added by mtaylor, 5 years ago)

Correct brain fart in previous version. Test, then post. Test, then post.

  • madwifi/tools/athdebug.c

    old new  
    212212        snprintf(oid, sizeof(oid), "dev.ath.%s.debug", ifname+3); 
    213213#endif 
    214214        debuglen = sizeof(debug); 
    215         if (sysctlbyname(oid, &debug, &debuglen, NULL, 0) < 0) 
    216                 err(1, "sysctl-get(%s)", oid); 
     215        if (sysctlbyname(oid, &debug, &debuglen, NULL, 0) < 0) { 
     216        if (argc <= 1) 
     217            usage(); /* no user input, device not found - show usage instead of error message */ 
     218        else  
     219            err(1, "sysctl-get(%s)", oid); /* user specified arguments indicating a command, show error message */ 
     220    } 
    217221        ndebug = debug; 
    218222        for (; argc > 1; argc--, argv++) { 
    219223                cp = argv[1]; 
    …… 
    262267                        sep = ","; 
    263268                } 
    264269        printf("%s\n", *sep != '<' ? ">" : ""); 
     270    printf("Details:\n"); 
     271    for (i = 0; i < N(flags); i++) 
     272        printf("%12s %s 0x%08x - %s\n", flags[i].name, debug & flags[i].bit ? "+" : " ", flags[i].bit, flags[i].desc); 
     273    exit(-1); 
    265274        return 0; 
    266275} 

  • madwifi/tools/80211debug.c

    old new  
    208208        snprintf(oid, sizeof(oid), "dev.wlan.%s.debug", ifname+4); 
    209209#endif 
    210210        debuglen = sizeof(debug); 
    211         if (sysctlbyname(oid, &debug, &debuglen, NULL, 0) < 0) 
    212                 err(1, "sysctl-get(%s)", oid); 
     211        if (sysctlbyname(oid, &debug, &debuglen, NULL, 0) < 0) { 
     212        if (argc <= 1)  
     213            usage(); /* no user input, device not found - show usage instead of error message */ 
     214        else  
     215            err(1, "sysctl-get(%s)", oid); /* user specified arguments indicating a command, show error message */ 
     216    } 
    213217        ndebug = debug; 
    214218        for (; argc > 1; argc--, argv++) { 
    215219                cp = argv[1]; 
    …… 
    258262                        sep = ","; 
    259263                } 
    260264        printf("%s\n", *sep != '<' ? ">" : ""); 
     265    printf("Details:\n"); 
     266    for (i = 0; i < N(flags); i++) 
     267        printf("%12s %s 0x%08x - %s\n", flags[i].name, debug & flags[i].bit ? "+" : " ", flags[i].bit, flags[i].desc); 
    261268        return 0; 
    262269} 
    263270