| 90 | } |
| 91 | |
| 92 | struct htlc_in *htlc_in_check(const struct htlc_in *hin, const char *abortstr) |
| 93 | { |
| 94 | if (amount_msat_is_zero(hin->msat)) |
| 95 | return corrupt(abortstr, "zero msatoshi"); |
| 96 | else if (htlc_state_owner(hin->hstate) != REMOTE) |
| 97 | return corrupt(abortstr, "invalid state %s", |
| 98 | htlc_state_name(hin->hstate)); |
| 99 | else if (hin->failonion && hin->preimage) |
| 100 | return corrupt(abortstr, "Both failonion and succeeded"); |
| 101 | else if (hin->badonion != 0 && hin->preimage) |
| 102 | return corrupt(abortstr, "Both badonion and succeeded"); |
| 103 | else if (hin->failonion && hin->badonion) |
| 104 | return corrupt(abortstr, "Both failed and malformed"); |
| 105 | |
| 106 | /* Can't have a resolution while still being added. */ |
| 107 | if (hin->hstate >= RCVD_ADD_HTLC |
| 108 | && hin->hstate <= RCVD_ADD_ACK_REVOCATION) { |
| 109 | if (hin->preimage) |
| 110 | return corrupt(abortstr, "Still adding, has preimage"); |
| 111 | if (hin->failonion) |
| 112 | return corrupt(abortstr, "Still adding, has failmsg"); |
| 113 | if (hin->badonion) |
| 114 | return corrupt(abortstr, "Still adding, has badonion"); |
| 115 | } else if (hin->hstate >= SENT_REMOVE_HTLC |
| 116 | && hin->hstate <= SENT_REMOVE_ACK_REVOCATION) { |
| 117 | if (!hin->preimage && !hin->failonion && !hin->badonion) |
| 118 | return corrupt(abortstr, "Removing, no resolution"); |
| 119 | } else |
| 120 | return corrupt(abortstr, "Bad state %s", |
| 121 | htlc_state_name(hin->hstate)); |
| 122 | |
| 123 | return cast_const(struct htlc_in *, hin); |
| 124 | } |
| 125 | |
| 126 | struct htlc_in *new_htlc_in(const tal_t *ctx, |
| 127 | struct channel *channel, u64 id, |
no test coverage detected