diff options
| author | Angelos D. Keromytis <angelos@cvs.openbsd.org> | 2002-06-25 00:20:39 +0000 |
|---|---|---|
| committer | Angelos D. Keromytis <angelos@cvs.openbsd.org> | 2002-06-25 00:20:39 +0000 |
| commit | 4052ea3919a763a084b4e76e7180fe68b6865a5b (patch) | |
| tree | 7b6f0a7a21c303e3a671f7d425457ec0033efe70 | |
| parent | 41e53fede2a32ca75b94b1d9d22c2db4dc1598f0 (diff) | |
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
| -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; } /* |