Skip to content

Parse non-v1 SCTs less awkwardly.#48

Closed
robstradling wants to merge 1 commit intoopenssl:masterfrom
robstradling:sct-viewer-master
Closed

Parse non-v1 SCTs less awkwardly.#48
robstradling wants to merge 1 commit intoopenssl:masterfrom
robstradling:sct-viewer-master

Conversation

@robstradling
Copy link
Contributor

No description provided.

@robstradling
Copy link
Contributor Author

Merged at 19f65dd

julian-klode added a commit to julian-klode/openssl that referenced this pull request Jan 18, 2025
Commit 1d1ca79 introduced
save and restore for the registers, saving them as

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

But the restore code was inadvertently typoed:

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

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

Fixes: openssl#26466
julian-klode added a commit to julian-klode/openssl that referenced this pull request Jan 18, 2025
Commit 1d1ca79 introduced
save and restore for the registers, saving them as

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

But the restore code was inadvertently typoed:

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

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

Fixes: openssl#26466
CLA: trivial
julian-klode added a commit to julian-klode/openssl that referenced this pull request Jan 18, 2025
Commit 1d1ca79 introduced
save and restore for the registers, saving them as

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

But the restore code was inadvertently typoed:

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

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

Fixes: openssl#26466

CLA: trivial
openssl-machine pushed a commit that referenced this pull request Jan 21, 2025
Commit 1d1ca79 introduced
save and restore for the registers, saving them as

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

But the restore code was inadvertently typoed:

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

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

Fixes: #26466

CLA: trivial

Reviewed-by: Kurt Roeckx <kurt@roeckx.be>
Reviewed-by: Paul Dale <ppzgs1@gmail.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #26469)
openssl-machine pushed a commit that referenced this pull request Jan 21, 2025
Commit 1d1ca79 introduced
save and restore for the registers, saving them as

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

But the restore code was inadvertently typoed:

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

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

Fixes: #26466

CLA: trivial

Reviewed-by: Kurt Roeckx <kurt@roeckx.be>
Reviewed-by: Paul Dale <ppzgs1@gmail.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #26469)

(cherry picked from commit 5261f3c)
openssl-machine pushed a commit that referenced this pull request Jan 21, 2025
Commit 1d1ca79 introduced
save and restore for the registers, saving them as

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

But the restore code was inadvertently typoed:

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

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

Fixes: #26466

CLA: trivial

Reviewed-by: Kurt Roeckx <kurt@roeckx.be>
Reviewed-by: Paul Dale <ppzgs1@gmail.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #26469)

(cherry picked from commit 5261f3c)
VladGud pushed a commit to VladGud/openssl that referenced this pull request Oct 9, 2025
 Add native Visual Studio 2015-2017 build support
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant