Skip to content
This repository was archived by the owner on Apr 10, 2025. It is now read-only.

Conversation

@tmshort
Copy link
Member

@tmshort tmshort commented Nov 2, 2021

Will not work since we aren't registered with https://google.github.io/oss-fuzz/getting-started/accepting-new-projects/

Checklist
  • documentation is added or updated
  • tests are added or updated

@tmshort
Copy link
Member Author

tmshort commented Nov 3, 2021

This latest change just comments out the CIFuzz check.

The QuicTLS project is not registered with the oss-fuzz project.
Maybe, someday...

https://google.github.io/oss-fuzz/getting-started/accepting-new-projects/

This avoids the failure in the CI.
@tmshort tmshort changed the title WIP: Fix CIFuzz? QUIC: Remove CIFuzz Nov 8, 2021
@tmshort tmshort closed this Apr 26, 2022
@tmshort tmshort deleted the fuzzer branch April 26, 2022 13:47
tmshort pushed a commit that referenced this pull request Feb 7, 2023
CLA: trivial

Reviewed-by: Hugo Landau <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
(Merged from openssl#19670)

(cherry picked from commit ec76891)
wbl pushed a commit that referenced this pull request Mar 14, 2023
CLA: trivial

Reviewed-by: Hugo Landau <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
(Merged from openssl#19670)

(cherry picked from commit ec76891)
tsuyopon pushed a commit to tsuyopon/openssl that referenced this pull request Feb 13, 2025
Commit 1d1ca79 introduced
save and restore for the registers, saving them as

	stp		d8,d9,[sp, quictls#16]
	stp		d10,d11,[sp, quictls#32]
	stp		d12,d13,[sp, quictls#48]
	stp		d14,d15,[sp, quictls#64]

But the restore code was inadvertently typoed:

	ldp		d8,d9,[sp, quictls#16]
	ldp		d10,d11,[sp, quictls#32]
	ldp		d12,d13,[sp, quictls#48]
	ldp		d15,d16,[sp, quictls#64]

Restoring [sp, quictls#64] into d15,d16 instead of d14,d15.

Fixes: openssl#26466

CLA: trivial

Reviewed-by: Kurt Roeckx <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from openssl#26469)

(cherry picked from commit 5261f3c)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant