[src/trunk]: src/sbin/ifconfig ifconfig: Warn once more if media supported bu...

[roy <roy%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/e513a141c359
branches:  trunk
changeset: 955634:e513a141c359
user:      roy <roy%NetBSD.org@localhost>
date:      Mon Oct 05 16:17:05 2020 +0000

description:
ifconfig: Warn once more if media supported but no types

This reverts media.c -r1.7

diffstat:

 sbin/ifconfig/media.c |  36 ++++++++++++++++++------------------
 1 files changed, 18 insertions(+), 18 deletions(-)

diffs (54 lines):

diff -r a33c304f68de -r e513a141c359 sbin/ifconfig/media.c
--- a/sbin/ifconfig/media.c     Mon Oct 05 16:11:25 2020 +0000
+++ b/sbin/ifconfig/media.c     Mon Oct 05 16:17:05 2020 +0000
@@ -1,6 +1,6 @@
 #include <sys/cdefs.h>
 #ifndef lint
-__RCSID("$NetBSD: media.c,v 1.10 2020/09/22 14:14:17 roy Exp $");
+__RCSID("$NetBSD: media.c,v 1.11 2020/10/05 16:17:05 roy Exp $");
 #endif /* not lint */
 
 #include <assert.h>
@@ -426,25 +426,25 @@
 
        /* Interface link status is queried through SIOCGIFMEDIA.
         * Not all interfaces have actual media. */
-       if (ifmr.ifm_count != 0) {
-               media_list = calloc(ifmr.ifm_count, sizeof(int));
-               if (media_list == NULL)
-                       err(EXIT_FAILURE, "malloc");
-               ifmr.ifm_ulist = media_list;
+       if (ifmr.ifm_count == 0)
+               warnx("%s: no media types?", ifname);
 
-               if (prog_ioctl(s, SIOCGIFMEDIA, &ifmr) == -1)
-                       err(EXIT_FAILURE, "SIOCGIFMEDIA");
+       media_list = calloc(ifmr.ifm_count, sizeof(int));
+       if (media_list == NULL)
+               err(EXIT_FAILURE, "malloc");
+       ifmr.ifm_ulist = media_list;
 
-               printf("\tmedia: %s ", get_media_type_string(ifmr.ifm_current));
-               print_media_word(ifmr.ifm_current, " ");
-               if (ifmr.ifm_active != ifmr.ifm_current) {
-                       printf(" (");
-                       print_media_word(ifmr.ifm_active, " ");
-                       printf(")");
-               }
-               printf("\n");
-       } else
-               media_list = NULL;
+       if (prog_ioctl(s, SIOCGIFMEDIA, &ifmr) == -1)
+               err(EXIT_FAILURE, "SIOCGIFMEDIA");
+
+       printf("\tmedia: %s ", get_media_type_string(ifmr.ifm_current));
+       print_media_word(ifmr.ifm_current, " ");
+       if (ifmr.ifm_active != ifmr.ifm_current) {
+               printf(" (");
+               print_media_word(ifmr.ifm_active, " ");
+               printf(")");
+       }
+       printf("\n");
 
        if (ifmr.ifm_status & IFM_STATUS_VALID)
                print_media_status(IFM_TYPE(ifmr.ifm_current), ifmr.ifm_status);