Skip to content

Comments

Call SSLfatal when the generate_ticket_cb returns 0#15487

Closed
tmshort wants to merge 1 commit intoopenssl:masterfrom
akamai:fix-ticket-cb-master
Closed

Call SSLfatal when the generate_ticket_cb returns 0#15487
tmshort wants to merge 1 commit intoopenssl:masterfrom
akamai:fix-ticket-cb-master

Conversation

@tmshort
Copy link
Contributor

@tmshort tmshort commented May 26, 2021

Otherwise, the state machine ends up being in a bad state:

SSL routines:write_state_machine:missing fatal:ssl/statem/statem.c:XXX:
Checklist
  • documentation is added or updated
  • tests are added or updated

Otherwise, the state machine ends up being in a bad state:
```
SSL routines:write_state_machine:missing fatal:ssl/statem/statem.c:XXX:
```
@kaduk kaduk added approval: otc review pending branch: master Applies to master branch triaged: bug The issue/pr is/fixes a bug labels May 26, 2021
@mattcaswell mattcaswell added approval: done This pull request has the required number of approvals and removed approval: otc review pending labels May 26, 2021
@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels May 27, 2021
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

openssl-machine pushed a commit that referenced this pull request May 27, 2021
Otherwise, the state machine ends up being in a bad state:
```
SSL routines:write_state_machine:missing fatal:ssl/statem/statem.c:XXX:
```

Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Ben Kaduk <[email protected]>
(Merged from #15487)
@kaduk
Copy link
Contributor

kaduk commented May 27, 2021

Merged to master; thanks!
Closing...

@kaduk kaduk closed this May 27, 2021
@tmshort tmshort deleted the fix-ticket-cb-master branch June 9, 2021 18:01
devnexen pushed a commit to devnexen/openssl that referenced this pull request Jul 7, 2021
Otherwise, the state machine ends up being in a bad state:
```
SSL routines:write_state_machine:missing fatal:ssl/statem/statem.c:XXX:
```

Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Ben Kaduk <[email protected]>
(Merged from openssl#15487)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Applies to master branch triaged: bug The issue/pr is/fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants