Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit ac11f88

Browse files
kyungrae2002aduh95
kyungrae2002
authored and
aduh95
committed
test,doc: cover and document multi-byte offset/size in randomFill
Signed-off-by: kyungrae <[email protected]> PR-URL: #64834 Reviewed-By: Daeyeon Jeong <[email protected]>
1 parent 1bf7737 commit ac11f88

3 files changed

Lines changed: 62 additions & 7 deletions

File tree

‎doc/api/crypto.md‎

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5584,9 +5584,12 @@ changes:
55845584

55855585
* `buffer` {ArrayBuffer|Buffer|TypedArray|DataView} Must be supplied. The
55865586
size of the provided `buffer` must not be larger than `2**31 - 1`.
5587-
* `offset` {number} **Default:** `0`
5588-
* `size` {number} **Default:** `buffer.length - offset`. The `size` must
5589-
not be larger than `2**31 - 1`.
5587+
* `offset` {number} The start position, in elements for a `TypedArray` and in
5588+
bytes for an `ArrayBuffer` or `DataView`. **Default:** `0`
5589+
* `size` {number} The amount to fill, in the same units as `offset`.
5590+
**Default:** `buffer.length - offset` for a `TypedArray`, or
5591+
`buffer.byteLength - offset` for an `ArrayBuffer` or `DataView`. The `size`
5592+
must not be larger than `2**31 - 1`.
55905593
* `callback` {Function} `function(err, buf) {}`.
55915594

55925595
This function is similar to [`crypto.randomBytes()`][] but requires the first
@@ -5721,9 +5724,12 @@ changes:
57215724

57225725
* `buffer` {ArrayBuffer|Buffer|TypedArray|DataView} Must be supplied. The
57235726
size of the provided `buffer` must not be larger than `2**31 - 1`.
5724-
* `offset` {number} **Default:** `0`
5725-
* `size` {number} **Default:** `buffer.length - offset`. The `size` must
5726-
not be larger than `2**31 - 1`.
5727+
* `offset` {number} The start position, in elements for a `TypedArray` and in
5728+
bytes for an `ArrayBuffer` or `DataView`. **Default:** `0`
5729+
* `size` {number} The amount to fill, in the same units as `offset`.
5730+
**Default:** `buffer.length - offset` for a `TypedArray`, or
5731+
`buffer.byteLength - offset` for an `ArrayBuffer` or `DataView`. The `size`
5732+
must not be larger than `2**31 - 1`.
57275733
* Returns: {ArrayBuffer|Buffer|TypedArray|DataView} The object passed as
57285734
`buffer` argument.
57295735

‎lib/internal/crypto/random.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,7 @@ function randomFill(buf, offset, size, callback) {
168168
size = buf.length;
169169
} else if (typeof size === 'function') {
170170
callback = size;
171-
size = buf.length - offset;
171+
size = (buf.length ?? buf.byteLength) - offset;
172172
} else {
173173
validateFunction(callback, 'callback');
174174
}

‎test/parallel/test-crypto-random.js‎

Lines changed: 49 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -218,6 +218,55 @@ common.expectWarning('DeprecationWarning',
218218
}));
219219
}
220220

221+
{
222+
const buf = new Uint16Array(10);
223+
const before = Buffer.from(buf.buffer).toString('hex');
224+
crypto.randomFillSync(buf, 1, 8);
225+
const after = Buffer.from(buf.buffer).toString('hex');
226+
assert.notStrictEqual(before, after);
227+
assert.deepStrictEqual(before.slice(0, 4), after.slice(0, 4));
228+
assert.deepStrictEqual(before.slice(-4), after.slice(-4));
229+
}
230+
231+
{
232+
const buf = new Uint32Array(10);
233+
const before = Buffer.from(buf.buffer).toString('hex');
234+
crypto.randomFillSync(buf, 1, 8);
235+
const after = Buffer.from(buf.buffer).toString('hex');
236+
assert.notStrictEqual(before, after);
237+
assert.deepStrictEqual(before.slice(0, 8), after.slice(0, 8));
238+
assert.deepStrictEqual(before.slice(-8), after.slice(-8));
239+
}
240+
241+
{
242+
const buf = new Uint16Array(10);
243+
const before = Buffer.from(buf.buffer).toString('hex');
244+
crypto.randomFill(buf, 1, 8, common.mustSucceed((buf) => {
245+
const after = Buffer.from(buf.buffer).toString('hex');
246+
assert.notStrictEqual(before, after);
247+
assert.deepStrictEqual(before.slice(0, 4), after.slice(0, 4));
248+
assert.deepStrictEqual(before.slice(-4), after.slice(-4));
249+
}));
250+
}
251+
252+
{
253+
const buf = new Uint32Array(10);
254+
const before = Buffer.from(buf.buffer).toString('hex');
255+
crypto.randomFill(buf, 1, 8, common.mustSucceed((buf) => {
256+
const after = Buffer.from(buf.buffer).toString('hex');
257+
assert.notStrictEqual(before, after);
258+
assert.deepStrictEqual(before.slice(0, 8), after.slice(0, 8));
259+
assert.deepStrictEqual(before.slice(-8), after.slice(-8));
260+
}));
261+
}
262+
263+
{
264+
// randomFill() with an offset and no size must not throw for types
265+
// without a .length property, matching randomFillSync().
266+
crypto.randomFill(new ArrayBuffer(10), 2, common.mustSucceed());
267+
crypto.randomFill(new DataView(new ArrayBuffer(10)), 2, common.mustSucceed());
268+
}
269+
221270
{
222271
[
223272
Buffer.alloc(10),

0 commit comments

Comments
 (0)