test: use faster variant for rss in test-crypto-dh-leak · nodejs/node@cc37ff2 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 635
• Pull requests 755
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit cc37ff2

Browse filesBrowse the repository at this point in the historyBrowse files

PoojaDurgad

authored and

danielleadams

committed

test: use faster variant for rss in test-crypto-dh-leak

PR-URL: #36766 Refs: #34291 Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Michael Dawson <midawson@redhat.com> Reviewed-By: James M Snell <jasnell@gmail.com>

1 parent daad0ab commit cc37ff2
Copy full SHA for cc37ff2

1 file changed

+2-2Lines changed: 2 additions & 2 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-crypto-dh-leak.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-crypto-dh-leak.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-crypto-dh-leak.js

+2-2Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
@@ -10,7 +10,7 @@ if (process.config.variables.asan)
10	10	const assert = require('assert');
11	11	const crypto = require('crypto');
12	12
13		- const before = process.memoryUsage().rss;
	13	+ const before = process.memoryUsage.rss();
14	14	{
15	15	const dh = crypto.createDiffieHellman(common.hasFipsCrypto ? 1024 : 256);
16	16	const publicKey = dh.generateKeys();
@@ -21,7 +21,7 @@ const before = process.memoryUsage().rss;
21	21	}
22	22	}
23	23	global.gc();
24		- const after = process.memoryUsage().rss;
	24	+ const after = process.memoryUsage.rss();
25	25
26	26	// RSS should stay the same, ceteris paribus, but allow for
27	27	// some slop because V8 mallocs memory during execution.

0 commit comments

Comments

0 (0)