Repository navigation
Commit d761a71
authored
fix(ext/node): read byob views at offset 0 in FileHandle.readableWebStream (#19)
The byob pull passed the view's byteOffset as the read offset. That
offset is relative to the view, so a view that starts after byte 0 of
its ArrayBuffer failed with ERR_OUT_OF_RANGE ("It must be <= 0"). Pass
0, as Node does since nodejs/node#58842 (v26.10.0).
Add a unit test that reads with a byob reader into 100-byte DataViews at
increasing offsets of one ArrayBuffer. It fails before this change.1 parent fba9b50 commit d761a71
2 files changed
Lines changed: 43 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
95 | 95 | | |
96 | 96 | | |
97 | 97 | | |
98 | | - | |
99 | 98 | | |
100 | 99 | | |
101 | 100 | | |
| |||
429 | 428 | | |
430 | 429 | | |
431 | 430 | | |
| 431 | + | |
432 | 432 | | |
433 | 433 | | |
434 | | - | |
| 434 | + | |
435 | 435 | | |
436 | 436 | | |
437 | 437 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
441 | 441 | | |
442 | 442 | | |
443 | 443 | | |
| 444 | + | |
| 445 | + | |
| 446 | + | |
| 447 | + | |
| 448 | + | |
| 449 | + | |
| 450 | + | |
| 451 | + | |
| 452 | + | |
| 453 | + | |
| 454 | + | |
| 455 | + | |
| 456 | + | |
| 457 | + | |
| 458 | + | |
| 459 | + | |
| 460 | + | |
| 461 | + | |
| 462 | + | |
| 463 | + | |
| 464 | + | |
| 465 | + | |
| 466 | + | |
| 467 | + | |
| 468 | + | |
| 469 | + | |
| 470 | + | |
| 471 | + | |
| 472 | + | |
| 473 | + | |
| 474 | + | |
| 475 | + | |
| 476 | + | |
| 477 | + | |
| 478 | + | |
| 479 | + | |
| 480 | + | |
| 481 | + | |
| 482 | + | |
| 483 | + | |
| 484 | + | |
444 | 485 | | |
445 | 486 | | |
446 | 487 | | |
| |||
0 commit comments