Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[test] riscv64 failure in parallel/test-crypto-sign-verify #45065

Open
kapouer opened this issue Oct 19, 2022 · 2 comments
Open

[test] riscv64 failure in parallel/test-crypto-sign-verify #45065

kapouer opened this issue Oct 19, 2022 · 2 comments
Labels
test Issues and PRs related to the tests.

Comments

@kapouer
Copy link
Contributor

kapouer commented Oct 19, 2022

Version

v18.10.0

Platform

Linux 5.18.0-4-riscv64 #1 SMP Debian 5.18.16-1 (2022-08-10) riscv64 (riscv64)

Subsystem

test

What steps will reproduce the bug?

Run the test.

How often does it reproduce? Is there a required condition?

I don't know. At least once.

What is the expected behavior?

Test succeeds

What do you see instead?

Test fails

Additional information

What do these lines want to achieve exactly ?

const library = {
configurable: true,
set() {
throw new Error('bye, bye, library');
}
};
Object.defineProperty(Object.prototype, 'library', library);
assert.throws(() => {
crypto.createSign('sha1').sign(
`-----BEGIN RSA PRIVATE KEY-----
AAAAAAAAAAAA
-----END RSA PRIVATE KEY-----`);
}, { message: 'bye, bye, library' });

@bnoordhuis
Copy link
Member

See #27157 for background. In a nutshell, node shouldn't crash (as in: segfault, abort) when JS code tampers with object prototypes.

@kapouer
Copy link
Contributor Author

kapouer commented Oct 19, 2022

Oh, ok. It's just odd that it's not in its own test.
On the other hand, that test fails on riscv64 - as if the "library" property wasn't changed, and the initial openssl error is caught, not the 'bye, bye, library'.

@kapouer kapouer changed the title [test] oddity in parallel/test-crypto-sign-verify [test] riscv64 failure in parallel/test-crypto-sign-verify Oct 19, 2022
@VoltrexKeyva VoltrexKeyva added the test Issues and PRs related to the tests. label Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Issues and PRs related to the tests.
Projects
None yet
Development

No branches or pull requests

3 participants