From 482bf1959a257813728cc7b13a9ff698590fec44 Mon Sep 17 00:00:00 2001 From: nobu Date: Sun, 10 May 2015 15:20:35 +0000 Subject: ruby.h: fix possible memory leak * include/ruby/ruby.h (Data_Make_Struct, TypedData_Make_Struct): allocate wrapper data object before allocating DATA_PTR to get rid of possible memory leak when the former failed. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@50464 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- gc.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) (limited to 'gc.c') diff --git a/gc.c b/gc.c index 3b2d5c9519..53c0ec5dd4 100644 --- a/gc.c +++ b/gc.c @@ -1812,6 +1812,14 @@ rb_data_object_alloc(VALUE klass, void *datap, RUBY_DATA_FUNC dmark, RUBY_DATA_F return newobj_of(klass, T_DATA, (VALUE)dmark, (VALUE)dfree, (VALUE)datap); } +VALUE +rb_data_object_zalloc(VALUE klass, size_t size, RUBY_DATA_FUNC dmark, RUBY_DATA_FUNC dfree) +{ + VALUE obj = rb_data_object_alloc(klass, 0, dmark, dfree); + DATA_PTR(obj) = xcalloc(1, size); + return obj; +} + VALUE rb_data_typed_object_alloc(VALUE klass, void *datap, const rb_data_type_t *type) { @@ -1819,6 +1827,14 @@ rb_data_typed_object_alloc(VALUE klass, void *datap, const rb_data_type_t *type) return newobj_of(klass, T_DATA | (type->flags & ~T_MASK), (VALUE)type, (VALUE)1, (VALUE)datap); } +VALUE +rb_data_typed_object_zalloc(VALUE klass, size_t size, const rb_data_type_t *type) +{ + VALUE obj = rb_data_typed_object_alloc(klass, 0, type); + DATA_PTR(obj) = xcalloc(1, size); + return obj; +} + size_t rb_objspace_data_type_memsize(VALUE obj) { -- cgit v1.2.3