From 8c6c73745ec6ace79e2fbb3655439acfb2d82f62 Mon Sep 17 00:00:00 2001 From: akr Date: Sat, 6 Apr 2013 04:14:57 +0000 Subject: * ext/socket/socket.c (rsock_sys_fail_path): Use rb_str_inspect only for String to avoid SEGV. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@40152 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ext/socket/socket.c | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) (limited to 'ext/socket') diff --git a/ext/socket/socket.c b/ext/socket/socket.c index be7bd62236..3ee725d8b5 100644 --- a/ext/socket/socket.c +++ b/ext/socket/socket.c @@ -29,16 +29,21 @@ void rsock_sys_fail_path(const char *mesg, VALUE path) { VALUE message; - if (memchr(RSTRING_PTR(path), '\0', RSTRING_LEN(path))) { - path = rb_str_inspect(path); - message = rb_sprintf("%s for %s", mesg, - StringValueCStr(path)); + if (RB_TYPE_P(path, T_STRING)) { + if (memchr(RSTRING_PTR(path), '\0', RSTRING_LEN(path))) { + path = rb_str_inspect(path); + message = rb_sprintf("%s for %s", mesg, + StringValueCStr(path)); + } + else { + message = rb_sprintf("%s for \"%s\"", mesg, + StringValueCStr(path)); + } + rb_sys_fail_str(message); } else { - message = rb_sprintf("%s for \"%s\"", mesg, - StringValueCStr(path)); + rb_sys_fail(mesg); } - rb_sys_fail_str(message); } void -- cgit v1.2.3