Skip to content

Commit

Permalink
* eval.c (rb_call0): wrong condition for $SAFE restoration.
Browse files Browse the repository at this point in the history
git-svn-id: http://svn.ruby-lang.org/repos/ruby/branches/ruby_1_6@9254 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
  • Loading branch information
matz committed Sep 20, 2005
1 parent fbe444d commit 46bf994
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 9 deletions.
22 changes: 17 additions & 5 deletions eval.c
Original file line number Diff line number Diff line change
Expand Up @@ -242,8 +242,10 @@ rb_clear_cache_by_class(klass)
}
}

#define NOEX_WITH_SAFE(n) ((n) | ruby_safe_level << 4)
#define NOEX_TAINTED 8
#define NOEX_SAFE(n) ((n) >> 4)
#define NOEX_WITH(n, v) ((n) | (v) << 4)
#define NOEX_WITH_SAFE(n) NOEX_WITH(n, ruby_safe_level)

void
rb_add_method(klass, mid, node, noex)
Expand Down Expand Up @@ -4468,12 +4470,16 @@ rb_call0(klass, recv, id, argc, argv, body, flags)
}
b2 = body = body->nd_next;

PUSH_VARS();
PUSH_TAG(PROT_FUNC);
if (NOEX_SAFE(flags) > ruby_safe_level) {
if (!(flags&NOEX_TAINTED) && ruby_safe_level == 0 && NOEX_SAFE(flags) > 2) {
rb_raise(rb_eSecurityError, "calling insecure method: %s",
rb_id2name(id));
}
safe = ruby_safe_level;
ruby_safe_level = NOEX_SAFE(flags);
}
PUSH_VARS();
PUSH_TAG(PROT_FUNC);
if ((state = EXEC_TAG()) == 0) {
NODE *node = 0;
int i;
Expand Down Expand Up @@ -6756,11 +6762,17 @@ method_call(argc, argv, method)
{
VALUE result;
struct METHOD *data;
int safe;

Data_Get_Struct(method, struct METHOD, data);
if (OBJ_TAINTED(method)) {
safe = NOEX_WITH(data->safe_level, 4)|NOEX_TAINTED;
}
else {
safe = data->safe_level;
}
PUSH_ITER(rb_block_given_p()?ITER_PRE:ITER_NOT);
result = rb_call0(data->klass,data->recv,data->id,argc,argv,data->body,
data->safe_level);
result = rb_call0(data->klass,data->recv,data->id,argc,argv,data->body,safe);
POP_ITER();
return result;
}
Expand Down
8 changes: 4 additions & 4 deletions version.h
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
#define RUBY_VERSION "1.6.8"
#define RUBY_RELEASE_DATE "2005-09-01"
#define RUBY_RELEASE_DATE "2005-09-21"
#define RUBY_VERSION_CODE 168
#define RUBY_RELEASE_CODE 20050901
#define RUBY_RELEASE_CODE 20050921

#define RUBY_VERSION_MAJOR 1
#define RUBY_VERSION_MINOR 6
#define RUBY_VERSION_TEENY 8
#define RUBY_RELEASE_YEAR 2005
#define RUBY_RELEASE_MONTH 9
#define RUBY_RELEASE_DAY 1
#define RUBY_RELEASE_MONTH 09
#define RUBY_RELEASE_DAY 21

0 comments on commit 46bf994

Please sign in to comment.