diff options
author | akr <akr@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-04-17 11:01:17 +0000 |
---|---|---|
committer | akr <akr@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-04-17 11:01:17 +0000 |
commit | ce6db8f286160fbf71c409416bd43c229b874601 (patch) | |
tree | e3ecd729b24a4746f8541954584b890404b7ecf0 /ext/socket | |
parent | cef6a377fcae99d062b2c7d9e0db2c07719c6975 (diff) | |
download | ruby-ce6db8f286160fbf71c409416bd43c229b874601.tar.gz |
* ext/socket/rubysocket.h (SOCKLEN_MAX): Defined.
* ext/socket/raddrinfo.c (ext/socket/raddrinfo.c): Reject too long
Linux abstract socket name.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@40335 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'ext/socket')
-rw-r--r-- | ext/socket/raddrinfo.c | 4 | ||||
-rw-r--r-- | ext/socket/rubysocket.h | 6 |
2 files changed, 9 insertions, 1 deletions
diff --git a/ext/socket/raddrinfo.c b/ext/socket/raddrinfo.c index d0d4d85c05..327218f222 100644 --- a/ext/socket/raddrinfo.c +++ b/ext/socket/raddrinfo.c @@ -450,8 +450,10 @@ rsock_unix_sockaddr_len(VALUE path) } else if (RSTRING_PTR(path)[0] == '\0') { /* abstract namespace; see unix(7) for details. */ + if (SOCKLEN_MAX - offsetof(struct sockaddr_un, sun_path) < (size_t)RSTRING_LEN(path)) + rb_raise(rb_eArgError, "Linux abstract socket too long"); return (socklen_t) offsetof(struct sockaddr_un, sun_path) + - RSTRING_LEN(path); + RSTRING_SOCKLEN(path); } else { #endif diff --git a/ext/socket/rubysocket.h b/ext/socket/rubysocket.h index 5369f566a4..1636a383cb 100644 --- a/ext/socket/rubysocket.h +++ b/ext/socket/rubysocket.h @@ -91,6 +91,12 @@ #ifndef HAVE_TYPE_SOCKLEN_T typedef int socklen_t; #endif + +#define SOCKLEN_MAX \ + (0 < (((socklen_t)0)-1) ? \ + ~(socklen_t)0 : \ + (((((socklen_t)1) << (sizeof(socklen_t) * CHAR_BIT - 2)) - 1) * 2 + 1)) + #ifndef RSTRING_SOCKLEN # define RSTRING_SOCKLEN (socklen_t)RSTRING_LENINT #endif |