Skip to content

Fix quoting error in SRP printf#64

Closed
radii wants to merge 1 commit intoopenssl:masterfrom
radii:master
Closed

Fix quoting error in SRP printf#64
radii wants to merge 1 commit intoopenssl:masterfrom
radii:master

Conversation

@radii
Copy link
Contributor

@radii radii commented Apr 12, 2014

The code is trying to interpolate the value of the BASE_SECTION macro,
but due to excess escaping, it instead prints the string "BASE_SECTION".

The code is trying to interpolate the value of the BASE_SECTION macro,
but due to excess escaping, it instead prints the string "BASE_SECTION".
@kroeckx
Copy link
Member

kroeckx commented Jan 22, 2016

+1

@richsalz
Copy link
Contributor

fixed in 44fdf1c thanks.

@richsalz richsalz closed this Jan 22, 2016
levitte pushed a commit to openssl/web that referenced this pull request Jul 26, 2018
Reviewed-by: Viktor Dukhovni <viktor@openssl.org>
(Merged from openssl/openssl#64)
@kroeckx kroeckx mentioned this pull request Apr 1, 2020
1 task
openssl-machine pushed a commit that referenced this pull request Nov 16, 2022
CLA: trivial

Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from #19670)
openssl-machine pushed a commit that referenced this pull request Nov 16, 2022
CLA: trivial

Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from #19670)

(cherry picked from commit ec76891)
openssl-machine pushed a commit that referenced this pull request Nov 16, 2022
CLA: trivial

Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from #19670)

(cherry picked from commit ec76891)
beldmit pushed a commit to beldmit/openssl that referenced this pull request Dec 26, 2022
CLA: trivial

Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from openssl#19670)
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
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.

3 participants