diff options
| author |   mikeb <mikeb@openbsd.org> | 2017-03-19 16:26:28 +0000 |
|---|---|---|
| committer | mikeb <mikeb@openbsd.org> | 2017-03-19 16:26:28 +0000 |
| commit | 7481bbc36d91e30a2ba85bdf239e93babf4d9bee (patch) | |
| tree | 19fbf5a9a039a92ffdd2b6befb6a86e2a29d763c | |
| parent | generate and show addresses from announced prefix (diff) | |
| download | wireguard-openbsd-7481bbc36d91e30a2ba85bdf239e93babf4d9bee.tar.xz wireguard-openbsd-7481bbc36d91e30a2ba85bdf239e93babf4d9bee.zip | |
Fixup return values to properly handle transfer submission errors
From Nathanael Rensen, thanks!
| -rw-r--r-- | sys/dev/pv/xbf.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sys/dev/pv/xbf.c b/sys/dev/pv/xbf.c index bedea7ab37d..cd15d64923b 100644 --- a/sys/dev/pv/xbf.c +++ b/sys/dev/pv/xbf.c @@ -1,4 +1,4 @@ -/* $OpenBSD: xbf.c,v 1.26 2017/03/13 01:10:03 mikeb Exp $ */ +/* $OpenBSD: xbf.c,v 1.27 2017/03/19 16:26:28 mikeb Exp $ */ /* * Copyright (c) 2016 Mike Belopuhov @@ -469,7 +469,7 @@ xbf_load_xs(struct scsi_xfer *xs, int desc) if (error) { DPRINTF("%s: failed to load %d bytes of data\n", sc->sc_dev.dv_xname, xs->datalen); - return (-1); + return (error); } for (i = 0; i < map->dm_nsegs; i++) { @@ -652,7 +652,7 @@ xbf_submit_cmd(struct scsi_xfer *xs) else error = xbf_bounce_xs(xs, desc); if (error) - return (error); + return (-1); } else { DPRINTF("%s: desc %d %s%s lba %llu\n", sc->sc_dev.dv_xname, desc, operation == XBF_OP_FLUSH ? "flush" : "barrier", @@ -675,7 +675,7 @@ xbf_submit_cmd(struct scsi_xfer *xs) xen_intr_signal(sc->sc_xih); - return desc; + return (desc); } int |