Delete SA

There is a small problem in find_phase2_state_to_delete().
[at least in freeswan-2.00-pre8]

The test for the bogus var must be
| if (pr->our_spi == spi)
|     *bogus = TRUE;
instead of
| if (pr->our_spi)
|     *bogus = TRUE;

The aim of the bogus var is to detect broken implementations
(like some freeswan 1.9x) which send its own spi in the delete
SA payload. The message will then explain why we ignore the
delete packet and avoid bugs reports.

| loglog(RC_LOG_SERIOUS
|     , "ignoring Delete SA payload: %s SA not found (%s)"
|     , enum_show(&protocol_names, d->isad_protoid)
|     , bogus ? "our SPI - bogus implementation" : "maybe expired");

--
Math.