diff options
| author | Martin Willi <martin@revosec.ch> | 2014-10-17 15:44:06 +0200 |
|---|---|---|
| committer | Martin Willi <martin@revosec.ch> | 2015-02-20 13:34:48 +0100 |
| commit | 698ed656f7578fbef0769216579417b32d8eb57e (patch) | |
| tree | 271cb201c4235522f7193b225cd3cfb648280559 | |
| parent | d05d85fe658321aff8d859eaf50704a21299b683 (diff) | |
| download | strongswan-698ed656f7578fbef0769216579417b32d8eb57e.tar.bz2 strongswan-698ed656f7578fbef0769216579417b32d8eb57e.tar.xz | |
child-sa: Remove the obsolete update logic
The kernel backend uses an inbound parameter these days, where it makes
no sense to pass the update flag. The kernel backend decides itself how
it handles SA installation based on the inbound flag.
| -rw-r--r-- | src/libcharon/sa/child_sa.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/src/libcharon/sa/child_sa.c b/src/libcharon/sa/child_sa.c index ed9e9b087..6f5e046c5 100644 --- a/src/libcharon/sa/child_sa.c +++ b/src/libcharon/sa/child_sa.c @@ -639,7 +639,6 @@ METHOD(child_sa_t, install, status_t, u_int32_t tfc = 0; host_t *src, *dst; status_t status; - bool update = FALSE; /* now we have to decide which spi to use. Use self allocated, if "in", * or the one in the proposal, if not "in" (others). Additionally, @@ -648,10 +647,6 @@ METHOD(child_sa_t, install, status_t, { dst = this->my_addr; src = this->other_addr; - if (this->my_spi == spi) - { /* alloc_spi has been called, do an SA update */ - update = TRUE; - } this->my_spi = spi; this->my_cpi = cpi; } @@ -722,7 +717,7 @@ METHOD(child_sa_t, install, status_t, inbound ? this->mark_in : this->mark_out, tfc, lifetime, enc_alg, encr, int_alg, integ, this->mode, this->ipcomp, cpi, this->config->get_replay_window(this->config), - initiator, this->encap, esn, update, src_ts, dst_ts); + initiator, this->encap, esn, inbound, src_ts, dst_ts); free(lifetime); |