Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Commit

Permalink
Dgram: correctly report recvmsg errors
Browse files Browse the repository at this point in the history
  • Loading branch information
piscisaureus committed Nov 24, 2011
1 parent 86fba38 commit 83152d1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/dgram.js
Expand Up @@ -300,7 +300,7 @@ function onMessage(handle, nread, buf, rinfo) {
var self = handle.socket;

if (nread == -1) {
self.emit('error', errnoException('recvmsg'));
self.emit('error', errnoException(errno, 'recvmsg'));
}
else {
rinfo.size = buf.length; // compatibility
Expand Down

1 comment on commit 83152d1

@tshinnic
Copy link

Choose a reason for hiding this comment

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

Oh dear, you mean this isn't inspired by my pull request from 2.5 months ago?
#1689
Oh foo.

I see that the other typo referenced hasn't been fixed in lib/child_process.js ... that line has now moved to 443 ...

Please sign in to comment.