svn commit: r284886 - head/sys/dev/fe
Takahashi Yoshihiro
nyan at FreeBSD.org
Sat Jun 27 09:01:50 UTC 2015
Author: nyan
Date: Sat Jun 27 09:01:49 2015
New Revision: 284886
URL: https://svnweb.freebsd.org/changeset/base/284886
Log:
MFi386: r278165
Silence a coverity warning about ignoring a return value.
Modified:
head/sys/dev/fe/if_fe_cbus.c
Modified: head/sys/dev/fe/if_fe_cbus.c
==============================================================================
--- head/sys/dev/fe/if_fe_cbus.c Sat Jun 27 08:49:41 2015 (r284885)
+++ head/sys/dev/fe/if_fe_cbus.c Sat Jun 27 09:01:49 2015 (r284886)
@@ -157,10 +157,21 @@ static int
fe_isa_attach(device_t dev)
{
struct fe_softc *sc = device_get_softc(dev);
+ int error = 0;
- if (sc->port_used)
- fe98_alloc_port(dev, sc->type);
- fe_alloc_irq(dev, 0);
+ /*
+ * Note: these routines aren't expected to fail since we also call
+ * them in the probe routine. But coverity complains, so we'll honor
+ * that complaint since the intention here was never to ignore them..
+ */
+ if (sc->port_used) {
+ error = fe98_alloc_port(dev, sc->type);
+ if (error != 0)
+ return (error);
+ }
+ error = fe_alloc_irq(dev, 0);
+ if (error != 0)
+ return (error);
return fe_attach(dev);
}
More information about the svn-src-head
mailing list