The following reply was made to PR kern/164313; it has been noted by GNATS.

From: John Baldwin <[email protected]>
To: [email protected], [email protected]
Cc:  
Subject: Re: kern/164313: Fix pci_get_vpd_readonly_method
Date: Thu, 19 Jan 2012 14:41:45 -0500

 Huh.  Might be even simpler to just return instead of breaking out of 
 the loop once vptr is found.
 
 Index: pci.c
 ===================================================================
 --- pci.c      (revision 230331)
 +++ pci.c      (working copy)
 @@ -1136,11 +1136,9 @@ pci_get_vpd_readonly_method(device_t dev, device_t
                if (memcmp(kw, cfg->vpd.vpd_ros[i].keyword,
                    sizeof(cfg->vpd.vpd_ros[i].keyword)) == 0) {
                        *vptr = cfg->vpd.vpd_ros[i].value;
 +                      return (0);
                }
 
 -      if (i != cfg->vpd.vpd_rocnt)
 -              return (0);
 -
        *vptr = NULL;
        return (ENXIO);
   }
 
 Does this work for you as well?
 
 -- 
 John Baldwin
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "[email protected]"

Reply via email to