diff options
| author |   angelos <angelos@openbsd.org> | 2002-06-25 00:20:38 +0000 |
|---|---|---|
| committer | angelos <angelos@openbsd.org> | 2002-06-25 00:20:38 +0000 |
| commit | f1dc91b50114a26deddfaabb9c960ba3f01a7871 (patch) | |
| tree | 6c244294d2a917a51ba80e68cb30baa7f66cc259 /sys | |
| parent | skip routing table lookup if multicasting/broadcasting and the outgoing (diff) | |
| download | wireguard-openbsd-f1dc91b50114a26deddfaabb9c960ba3f01a7871.tar.xz wireguard-openbsd-f1dc91b50114a26deddfaabb9c960ba3f01a7871.zip | |
Handle correctly return values from xf_input methods --- since the
return value was ignored anyway, this wasn't a problem so far. From
sam@errno.com
Diffstat (limited to 'sys')
| -rw-r--r-- | sys/netinet/ipsec_input.c | 18 |
1 files changed, 7 insertions, 11 deletions
diff --git a/sys/netinet/ipsec_input.c b/sys/netinet/ipsec_input.c index f6060b86072..adf42cd2db6 100644 --- a/sys/netinet/ipsec_input.c +++ b/sys/netinet/ipsec_input.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ipsec_input.c,v 1.58 2002/06/13 08:02:36 angelos Exp $ */ +/* $OpenBSD: ipsec_input.c,v 1.59 2002/06/25 00:20:38 angelos Exp $ */ /* * The authors of this code are John Ioannidis (ji@tla.org), * Angelos D. Keromytis (kermit@csd.uch.gr) and @@ -248,16 +248,12 @@ ipsec_common_input(struct mbuf *m, int skip, int protoff, int af, int sproto) } /* - * Call appropriate transform and return -- callback takes care of - * everything else. - */ - if ((*(tdbp->tdb_xform->xf_input))(m, tdbp, skip, protoff) == NULL) { - splx(s); - return EINVAL; - } else { - splx(s); - return 0; - } + * Call appropriate transform and return -- callback takes care of + * everything else. + */ + error = (*(tdbp->tdb_xform->xf_input))(m, tdbp, skip, protoff); + splx(s); + return error; } /* |