Skip to content

Buffer bug fix #6775

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.

Sign up for GitHub

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

Closed
jspri wants to merge 6 commits into from
Closed

Buffer bug fix #6775

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
f998347
Buffer: Fixes #6770, Single digit hex strings not raising TypeError o…
jspri May 15, 2016
61a4129
Adds test
jspri May 15, 2016
bada963
Fixed linting
jspri May 15, 2016
e4b8572
Refactored to improve performance
jspri May 18, 2016
c1d79eb
Cleaned up test comment and added regression test
jspri May 18, 2016
2db93e5
Changed base64 test to use length
jspri May 19, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Adds test
  • Loading branch information
@jspri
jspri committed May 15, 2016
commit 61a4129cd934cd6c79a6d6bd75586a84c3bb6158
5 changes: 5 additions & 0 deletions test/parallel/test-buffer.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -749,6 +749,11 @@ for (let i = 0; i < 256; i++) {
assert.equal(hexb2[i], hexb[i]);
}

//#6770 Test single hex character throws TypeError
Copy link
Copy Markdown
Member

@addaleax addaleax May 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can reference github issues in comments, but it is usually preferred to give the full URL for things like that so that it’s easier to look up and there is no ambiguity wrt the old issue tracker at https://github.com/nodejs/node-v0.x-archive/

assert.throws(function() {
var b = Buffer.from('A', 'hex');
}, TypeError)

{
// test an invalid slice end.
console.log('Try to slice off the end of the buffer');
Expand Down