Add TooLarge error code for buffers (#1298)

In collaboration with @yushimatenjin
This commit is contained in:
binaryta 2018-12-10 05:38:30 +09:00 committed by Ryan Dahl
parent f2447f6a23
commit c427c2df42
3 changed files with 26 additions and 2 deletions

View file

@ -6,6 +6,7 @@
import { Reader, Writer, ReadResult } from "./io";
import { assert } from "./util";
import { TextDecoder } from "./text_encoding";
import { DenoError, ErrorKind } from "./errors";
// MIN_READ is the minimum ArrayBuffer size passed to a read call by
// buffer.ReadFrom. As long as the Buffer has at least MIN_READ bytes beyond
@ -170,7 +171,10 @@ export class Buffer implements Reader, Writer {
// don't spend all our time copying.
copyBytes(this.buf, this.buf.subarray(this.off));
} else if (c > MAX_SIZE - c - n) {
throw Error("ErrTooLarge"); // TODO DenoError(TooLarge)
throw new DenoError(
ErrorKind.TooLarge,
"The buffer cannot be grown beyond the maximum size."
);
} else {
// Not enough space anywhere, we need to allocate.
const buf = new Uint8Array(2 * c + n);

View file

@ -1,9 +1,9 @@
import { Buffer, readAll } from "deno";
import * as deno from "deno";
// This code has been ported almost directly from Go's src/bytes/buffer_test.go
// Copyright 2009 The Go Authors. All rights reserved. BSD license.
// https://github.com/golang/go/blob/master/LICENSE
import { assert, assertEqual, test } from "./test_util.ts";
// N controls how many iterations of certain checks are performed.
const N = 100;
let testBytes: Uint8Array | null;
@ -130,6 +130,25 @@ test(async function bufferLargeByteWrites() {
check(buf, "");
});
test(async function bufferTooLargeByteWrites() {
init();
const tmp = new Uint8Array(72);
const growLen = Number.MAX_VALUE;
const xBytes = repeat("x", 0);
const buf = new Buffer(xBytes.buffer as ArrayBuffer);
const { nread, eof } = await buf.read(tmp);
let err;
try {
buf.grow(growLen);
} catch (e) {
err = e;
}
assertEqual(err.kind, deno.ErrorKind.TooLarge);
assertEqual(err.name, "TooLarge");
});
test(async function bufferLargeByteReads() {
init();
const buf = new Buffer();

View file

@ -105,6 +105,7 @@ enum ErrorKind: byte {
HttpCanceled,
HttpParse,
HttpOther,
TooLarge,
}
table Cwd {}